-
Notifications
You must be signed in to change notification settings - Fork 308
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 a data-flow analysis for local variables in uniformity #3298
Conversation
Previews, as seen when this build job started (262b841): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
Nearly there
Fixes gpuweb#2859 * Adds local variable value analysis to uniformity to track values * full assignments complete replace a value * partial assignments get connected to previous value * Describes function address space pointers as being desugared into variables and extra value return nodes
* Remove fallthrough references * Clarify what is uniform for pointer parameters (the value pointed to) * Fix edges for switch statement result * Fix typos
fd5b1bb
to
c78621e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Content looks right.
Two editorial nits left.
Will land this, as this implements the thing already agreed to and implemented in Tint. (hence is implementable). |
Fixes #2859
variables and extra value return nodes