Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

jimblandy
Copy link
Contributor

Change references to GPUProgrammableStage.entryPoint, which is no longer required (#4387), to uses of the "get the entry point" algorithm.

Rather than talking about resources "statically used" by a GPUProgrammableStage, talk about resources "statically used" by a specific entry point. Use "get the entry point" as necessary to determine the relevant entry point.

@jimblandy jimblandy added the copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.) label Jan 31, 2024
Copy link
Contributor

github-actions bot commented Jan 31, 2024

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

Copy link
Contributor

@kainino0x kainino0x left a comment

Choose a reason for hiding this comment

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

Content LGTM, copyediting comments

@kainino0x kainino0x added this to the Milestone 1 milestone Feb 1, 2024
@jimblandy jimblandy force-pushed the use-get-the-entry-point branch 2 times, most recently from e073e6d to d934032 Compare February 2, 2024 02:34
Change references to `GPUProgrammableStage.entryPoint`, which is no
longer required (gpuweb#4387), to uses of the "get the entry point"
algorithm.

Rather than talking about resources "statically used" by a
GPUProgrammableStage, talk about resources "statically used" by a
specific entry point. Use "get the entry point" as necessary to
determine the relevant entry point.
@jimblandy jimblandy force-pushed the use-get-the-entry-point branch from d934032 to e01c5b2 Compare February 2, 2024 02:39
@jimblandy jimblandy enabled auto-merge (squash) February 2, 2024 02:40
@jimblandy jimblandy merged commit af6e545 into gpuweb:main Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
copyediting Pure editorial stuff (copyediting, *.bs file syntax, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants