Skip to content

[WIP] spv/lower: map conditional OpReturn in entry-points to ExitInvocation. - #40

Draft
eddyb wants to merge 1 commit into
eddyb/thunkfrom
eddyb/entry-ret-as-abort
Draft

eddyb wants to merge 1 commit into
eddyb/thunkfrom
eddyb/entry-ret-as-abort

Conversation

@eddyb

@eddyb eddyb commented Apr 23, 2026

Copy link
Copy Markdown
Member

Note: this PR is a draft to avoid accidental merging onto its "base" branch (used as a form of ad-hoc PR stacking), and will remain as such, until its "base" branch can be set to main, i.e. all prerequisite PRs will have landed, up to and including this PR (whose branch is the "base" of this one):


This, like #35, mostly serves to improve loading the output of SPIR-T -> SPIR-V (e.g. from Rust-GPU) back into SPIR-T, avoiding some amplification (in this case, deep if-else nesting from conditional panics).

@eddyb
eddyb force-pushed the eddyb/entry-ret-as-abort branch from a7b80bc to b39214a Compare October 9, 2026 16:29
@eddyb
eddyb force-pushed the eddyb/entry-ret-as-abort branch from b39214a to 80f2a7e Compare October 9, 2026 16:53
@eddyb
eddyb force-pushed the eddyb/entry-ret-as-abort branch 3 times, most recently from f29df1f to d32081a Compare October 12, 2026 14:41
@eddyb
eddyb force-pushed the eddyb/entry-ret-as-abort branch from d32081a to c037c76 Compare October 12, 2026 14:46
@eddyb
eddyb added this pull request to stack #53 October 12, 2026 15:08

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant