[PATCH] D99269: [AMDGPU] Unify spill code

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 24 08:07:10 PDT 2021


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIFrameLowering.cpp:708-709
+    // gets used.
+    auto SpillMI = MBBI;
+    (--SpillMI)->setFlag(MachineInstr::FrameSetup);
   }
----------------
I don't think you're supposed to rely on the frame setup flags this way


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99269



More information about the llvm-commits mailing list