这是indexloc提供的服务，不要输入任何密码
Skip to content

Allow inter-stage variable subsetting #3122

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 14, 2022
Merged

Allow inter-stage variable subsetting #3122

merged 1 commit into from
Jul 14, 2022

Conversation

kainino0x
Copy link
Contributor

@kainino0x kainino0x commented Jun 30, 2022 •
edited
Loading

Proposed less strict alternative to #3073

Fixes #2038

@kainino0x kainino0x added this to the V1.0 milestone Jun 30, 2022
@kainino0x kainino0x added the needs-cts-issue This change requires tests (or would need tests if accepted), but may not have a CTS issue filed yet label Jun 30, 2022
@github-actions
Copy link
Contributor

Previews, as seen when this build job started (d77cc10):
WebGPU webgpu.idl | Explainer | Correspondence Reference
WGSL grammar.js | wgsl.lalr.txt

@kainino0x kainino0x mentioned this pull request Jun 30, 2022
Closed
Kangz
Kangz approved these changes Jul 1, 2022
View reviewed changes
Copy link
Contributor

@Kangz Kangz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM assuming the group approves.

@kainino0x
Copy link
Contributor Author

Approach approved in CG meeting 2022-07-13

@kainino0x kainino0x added the copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.) label Jul 13, 2022
@kainino0x
Copy link
Contributor Author

Which I suppose means we land this.

@kainino0x kainino0x merged commit 025b94a into gpuweb:main Jul 14, 2022
@kainino0x kainino0x deleted the inter-stage-subset branch July 14, 2022 19:43
@kainino0x kainino0x removed the copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.) label Jul 14, 2022
jdarpinian pushed a commit to jdarpinian/gpuweb that referenced this pull request Aug 12, 2022
@kainino0x @jdarpinian
Allow inter-stage variable subsetting (gpuweb#3122)
c741dfe
@lokokung
Copy link
Contributor

Removing 'needs-cts-issue' since there is already an open CTS issue: gpuweb/cts#1554

@lokokung lokokung removed the needs-cts-issue This change requires tests (or would need tests if accepted), but may not have a CTS issue filed yet label Nov 30, 2022
@lokokung lokokung mentioned this pull request Nov 30, 2022
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kangz Kangz Kangz approved these changes

Assignees
No one assigned
Labels
proposal
Projects
None yet
Milestone
Milestone 0
Development

Successfully merging this pull request may close these issues.

Shader stage input-output subsetting rules
3 participants
@kainino0x @lokokung @Kangz