[PATCH] D105086: [NFC][RISCV] Add FrameSetup/FrameDestroy flag to prologue/epilog instructions.

[Hsiangkai Wang via Phabricator via llvm-commits]

HsiangKai added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVFrameLowering.cpp:528
         // move BP, SP
         TII->copyPhysReg(MBB, MBBI, DL, BPReg, SPReg, false);
       }
----------------
rogfer01 wrote:
> rogfer01 wrote:
> > I think this might still add unflagged instructions.
> > 
> > Not sure what is the best way to flag them (I imagine we could traverse MBBI to the end of the MBB and invoke `setFlag`).
> > (I imagine we could traverse MBBI to the end of the MBB and invoke setFlag).
> Oh no, that wouldn't work. Please ignore me.
Yeah, there is no way to distinguish the caller of copyPhysReg() comes from prologue/epilogue or not.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105086