Skip to content

Fix LifetimeIntrinsicTest issues#3609

Merged
tex3d merged 2 commits intomicrosoft:masterfrom
tex3d:fix-lifetime-test
Mar 21, 2021
Merged

Fix LifetimeIntrinsicTest issues#3609
tex3d merged 2 commits intomicrosoft:masterfrom
tex3d:fix-lifetime-test

Conversation

@tex3d
Copy link
Copy Markdown
Contributor

@tex3d tex3d commented Mar 19, 2021

  • was only run on SM 6.6 supporting devices
  • would only run SM 6.6 tests in experimental mode, compiling with -Vd
  • wasn't specifying options required to test lifetime fallback paths

- was only run on SM 6.6 supporting devices
- would only run SM 6.6 tests in experimental mode, compiling with -Vd
- wasn't specifying options required to test lifetime fallback paths
@tex3d tex3d requested a review from pow2clk March 19, 2021 22:06
Copy link
Copy Markdown
Collaborator

@pow2clk pow2clk left a comment

Choose a reason for hiding this comment

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

Extremely minor quibbles about naming. Looks good!

Comment thread tools/clang/unittests/HLSL/ExecutionTest.cpp Outdated
Comment thread tools/clang/unittests/HLSL/ExecutionTest.cpp
@AppVeyorBot
Copy link
Copy Markdown

@AppVeyorBot
Copy link
Copy Markdown

@tex3d tex3d merged commit 22fa209 into microsoft:master Mar 21, 2021
@tex3d tex3d deleted the fix-lifetime-test branch March 21, 2021 22:02
tex3d added a commit to tex3d/DirectXShaderCompiler that referenced this pull request Mar 22, 2021
* Fix LifetimeIntrinsicTest issues

- was only run on SM 6.6 supporting devices
- would only run SM 6.6 tests in experimental mode, compiling with -Vd
- wasn't specifying options required to test lifetime fallback paths

* Disable LifetimeIntrinsicTest and HelperLaneTestWave on warp

(cherry picked from commit 22fa209)
tex3d added a commit that referenced this pull request Mar 23, 2021
Cherry-pick changes to release branch:
cd3ef21 Roll back llvm::ArrayRef dependency in ExecutionTest (#3613)
2791c51 Generate descriptions for resources with no names (#3598)
22fa209 Fix LifetimeIntrinsicTest issues (#3609)
2039610 Fix Dxil validator compat and test issues (#3610)
220e884 Rename payload qualifier field to not match type (#3607)
0e89206 Correct exception handler sprintf for 32-bit (#3608)
9b475a7 Add dxc exception handler (#3604)
e8372b9 Fixed arg pairs not correct for old source in module pdbs (#3599)
2bda44f Add constant evaluation for clamp() (#3581)
640c9af Added way for caller to replace args in PDB utils (#3595)
de00b01 Fix const error check for object subscript operator (#3580)
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.

3 participants