[PATCH] D64766: [AMDGPU] Enable merging m0 initializations.

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 15 14:03:12 PDT 2019


rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFixSGPRCopies.cpp:458-459
+getFirstNonProlog(MachineBasicBlock *MBB, const TargetInstrInfo *TII) {
+  MachineBasicBlock::iterator I = MBB->getFirstNonPHI();
+  while (I != MBB->end() && TII->isBasicBlockPrologue(*I))
+    ++I;
----------------
arsenm wrote:
> Why does this need to care about the prolog instructions? (FYI I'm planning on eliminating the rarely followed concept)
You actually cannot, it was created to prevent RA from insrting spills into endif/else block before we updated exec mask.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D64766





More information about the llvm-commits mailing list