[PATCH] D148239: [AMDGPU] Allow use of TTMP registers in AMDGPUResourceUsageAnalysis

[Christudasan Devadasan via Phabricator via llvm-commits]

cdevadas added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUResourceUsageAnalysis.cpp:359
-        } else if (AMDGPU::SReg_64RegClass.contains(Reg)) {
-          assert(!AMDGPU::TTMP_64RegClass.contains(Reg) &&
-                 "trap handler registers should not be used");
----------------
Shouldn't we retain the assertion for TTMP reg sizes bigger than 32-bit? 
We don't have any TTMP tuple usage, I guess.



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148239/new/

https://reviews.llvm.org/D148239