[llvm] [RISCV] Set DebugLoc of epilogue (PR #74702)

Alex Bradbury via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 7 07:31:24 PST 2023


asb wrote:

> But is this the right way for current situation? The debugger behaves unproperly.

My standard assumption would be that following what X86/AArch64/Arm chose to do is correct and that remaining issues are likely caused by something else. I could be wrong of course, but I think you'd need to try to convince us as reviewers of why a different choice vs those targets is desirable here.

I really don't think there's been all that much critical analysis of the debuggability of LLVM generated RISC-V executables (though issues have been raised and fixed over the years), so I'm sure there are problems to be resolved like the one you raised in this PR.

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


More information about the llvm-commits mailing list