[PATCH] D117014: AMDGPU: Use removeAllRegUnitsForPhysReg()

[Ruiling, Song via Phabricator via llvm-commits]

ruiling added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIWholeQuadMode.cpp:1594
   if (!KillInstrs.empty())
-    LIS->removeRegUnit(*MCRegUnitIterator(AMDGPU::EXEC, TRI));
+    LIS->removeAllRegUnitsForPhysReg(AMDGPU::EXEC);
 
----------------
arsenm wrote:
> EXEC is reserved, so does the liveness actually exist?
yes, it is a little bit strange, the EXEC_LO_LO16 live range appears after register coalescer, I have not figured out what happened.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117014