[PATCH] D92400: [AMDGPU] Stop adding an implicit def of vcc_hi for wave32

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 1 11:12:27 PST 2020


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIISelLowering.cpp:11567
       for (auto &MI : MBB) {
         TII->fixImplicitOperands(MI);
       }
----------------
arsenm wrote:
> We should really fix the need for this loop too
How? Would you need separate wave32/wave64 versions of every pseudo that has an implicit VCC operand? There are quite a few of those.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92400



More information about the llvm-commits mailing list