[llvm] [RISCV] Add an option to enable CFIInstrInserter. (PR #164477)

Sam Elliott via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 17 11:01:09 PST 2025


lenary wrote:

I'm not actually sure the value of that test. What I'm trying to get an answer to is "why is this pass being added?", but I also haven't had the time to go and review what CFIInserter does and why - I know it's (by LLVM's standards) a fairly recent addition. (Your test shows CFI Inserter is being added, but not what effect that has, just the debug output from it being added)

If there are no (CodeGen) changes to existing tests, then I guess there's no problem and we can proceed, especially if it unblocks things we know are improvements.

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


More information about the llvm-commits mailing list