Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add shared-memory program #611

Merged
merged 2 commits into from
Oct 14, 2020
Merged

Conversation

jackcmay
Copy link
Contributor

@jackcmay jackcmay commented Oct 13, 2020

There's no good way to return data to the caller of a cross-program invocation as per:
solana-labs/solana#12601

This PR adds a shared-memory program that takes a single account and writes the instruction data into the account's data starting an offset

Because of the simplicity of the program, the minimal number of inputs, and the desire to not burden the cross-program invocations this program is optimized to pointedly deserialize the few program parameters it needs.

@jackcmay jackcmay added the WIP Work in progress label Oct 13, 2020
@jackcmay jackcmay force-pushed the shared-memory-program branch 3 times, most recently from ebc6bc9 to e035646 Compare October 14, 2020 07:37
@jackcmay jackcmay removed the WIP Work in progress label Oct 14, 2020
@jackcmay jackcmay force-pushed the shared-memory-program branch 2 times, most recently from 9261744 to c0c1ba5 Compare October 14, 2020 21:24
@jackcmay jackcmay force-pushed the shared-memory-program branch from c0c1ba5 to f5d8c97 Compare October 14, 2020 22:41
@jackcmay jackcmay added the automerge Merge this pull request automatically once CI and reviews are compelte label Oct 14, 2020
@mergify mergify bot merged commit 08d9999 into solana-labs:master Oct 14, 2020
@jackcmay jackcmay deleted the shared-memory-program branch October 14, 2020 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this pull request automatically once CI and reviews are compelte
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant