[llvm] [RISCV] Guard CFI emission code with MF.needsFrameMoves() (PR #136060)

Sam Elliott via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 2 18:15:42 PDT 2025


https://github.com/lenary approved this pull request.

LGTM.

Sorry, going back over older LLVM PRs. I think the codegen is close enough right now that it's not an issue, but I would like the differences to be fixed.

This will likely need a rebase.

I did wonder if we should push the conditions into `CFIBuilder::insertCFIInst` but I don't think that's necessarily reasonable. It would be good if we didn't have to always remember the `if (NeedsDwarfCFI)` stuff inline.

https://github.com/llvm/llvm-project/pull/136060


More information about the llvm-commits mailing list