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 validation support for SPV_NV_shader_invocation_reorder. #4979

Merged
merged 6 commits into from
Nov 24, 2022

Conversation

alelenv
Copy link
Contributor

@alelenv alelenv commented Oct 31, 2022

@alelenv
Copy link
Contributor Author

alelenv commented Nov 1, 2022

cc : @dgkoch

@johnkslang or @dneto0 to review
Task list issue contains all related collateral

@alan-baker
Copy link
Contributor

This PR needs rebased.

@alelenv
Copy link
Contributor Author

alelenv commented Nov 14, 2022

@alan-baker : Rebased, please re-review

@alelenv
Copy link
Contributor Author

alelenv commented Nov 17, 2022

Ping @alan-baker , can you take a look
Thanks

Copy link
Contributor

@alan-baker alan-baker left a comment

Choose a reason for hiding this comment

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

Looks basically good, just need to fix the bots.

@alelenv
Copy link
Contributor Author

alelenv commented Nov 22, 2022

@alan-baker : I have fixed the build bot failures. Can you retrigger?

@alelenv
Copy link
Contributor Author

alelenv commented Nov 23, 2022

Had missed a clang format, everything is now green except the ndk-build which complains about undefined references

@alan-baker : Can you assist? I am not clear why it would break only on android/ndk build

@alan-baker
Copy link
Contributor

You might have forgotten to update other build systems. It looks like only cmakelists are updated. The android.mk needs updated I expect. Bazel is probably glob'ing all the files already.

@alan-baker alan-baker merged commit f33d152 into KhronosGroup:master Nov 24, 2022
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.

6 participants