-
Notifications
You must be signed in to change notification settings - Fork 564
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
Handle more cases in dead member elim #3289
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Rewrite composite insert and extract operations on SpecConstnatOp. - Leaves assert for Access chain instructions, which are only allowed for kernels. - Other operations do not require any extra code will no longer cause an assert. Fixes KhronosGroup#3284.
kuhar
reviewed
Apr 9, 2020
kuhar
approved these changes
Apr 9, 2020
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.
LGTM
dneto0
pushed a commit
to dneto0/SPIRV-Tools
that referenced
this pull request
Sep 14, 2024
Roll third_party/glslang/ b5757b9..4d2298b (5 commits) KhronosGroup/glslang@b5757b9...4d2298b $ git log b5757b9..4d2298b --date=short --no-merges --format='%ad %ae %s' 2020-04-13 cnorthrop Support multiple swizzled out operands (KhronosGroup#2175) 2020-04-12 cepheus Fix KhronosGroup#2178: Allow specialization constants for texel offsets. 2020-04-10 h.baensch.92 Get rid of all warnings with MSVC and clang-cl (KhronosGroup#2177) 2020-04-08 40001162+alelenv Add support for EXT_ray_flags_primitive_culling. (KhronosGroup#2173) 2020-04-07 cepheus Error message: Finish addressing KhronosGroup#2097, better texture error message. Roll third_party/re2/ 58141dc9c..ac65d4531 (2 commits) google/re2@58141dc...ac65d45 $ git log 58141dc9c..ac65d4531 --date=short --no-merges --format='%ad %ae %s' 2020-04-09 junyer Remove deprecated APIs. Bump SONAME accordingly. 2020-04-06 junyer Go back to using __builtin_ctzll(). Sigh. Roll third_party/spirv-cross/ 6637610b1..54658d625 (3 commits) KhronosGroup/SPIRV-Cross@6637610...54658d6 $ git log 6637610b1..54658d625 --date=short --no-merges --format='%ad %ae %s' 2020-04-10 cdavis MSL: Add options to control emission of fragment outputs. 2020-04-09 h.baensch.92 Improve compatibility with clang-cl 2020-04-07 post MSL: Do not use base expression with PhysicalTypeID OpCompositeExtract. Roll third_party/spirv-headers/ f8bf11a..2ad0492 (1 commit) KhronosGroup/SPIRV-Headers@f8bf11a...2ad0492 $ git log f8bf11a..2ad0492 --date=short --no-merges --format='%ad %ae %s' 2020-04-13 cepheus Discuss generator magic number reservations. Roll third_party/spirv-tools/ e95fbfb..ca57515 (9 commits) KhronosGroup/SPIRV-Tools@e95fbfb...ca57515 $ git log e95fbfb..ca57515 --date=short --no-merges --format='%ad %ae %s' 2020-04-14 dneto If SPIRV-Headers is in our tree, include it as subproject (KhronosGroup#3299) 2020-04-13 stevenperron Struct CFG analysus and single block loop (KhronosGroup#3293) 2020-04-13 jaebaek Preserve debug info in eliminate-dead-functions (KhronosGroup#3251) 2020-04-13 stevenperron Update acorn version (KhronosGroup#3294) 2020-04-09 stevenperron Handle more cases in dead member elim (KhronosGroup#3289) 2020-04-09 h.baensch.92 Fix pch macro to ignore clang-cl (KhronosGroup#3283) 2020-04-07 afdx spirv-fuzz: Improve the handling of equation facts (KhronosGroup#3281) 2020-04-07 afdx spirv-fuzz: Handle more general SPIR-V in donation (KhronosGroup#3280) 2020-04-06 afdx spirv-fuzz: Improve support for compute shaders in donation (KhronosGroup#3277) Created with: roll-dep third_party/effcee third_party/glslang third_party/googletest third_party/re2 third_party/spirv-cross third_party/spirv-headers third_party/spirv-tools
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
for kernels.
assert.
Fixes #3284.
Fixes #3282.