[llvm] M68k: Remove hasDebugInfo check when deciding to emit CFI (PR #99750)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 20 03:05:25 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-m68k
Author: Matt Arsenault (arsenm)
<details>
<summary>Changes</summary>
No other target checks this, and this is untested. I am trying
to remove the MachineModuleInfo reference from MachineFunction,
and this is the stickiest blocker.
---
Full diff: https://github.com/llvm/llvm-project/pull/99750.diff
1 Files Affected:
- (modified) llvm/lib/Target/M68k/M68kFrameLowering.cpp (+1-2)
``````````diff
diff --git a/llvm/lib/Target/M68k/M68kFrameLowering.cpp b/llvm/lib/Target/M68k/M68kFrameLowering.cpp
index 36443f9d33451..a609be08d3021 100644
--- a/llvm/lib/Target/M68k/M68kFrameLowering.cpp
+++ b/llvm/lib/Target/M68k/M68kFrameLowering.cpp
@@ -246,9 +246,8 @@ MachineBasicBlock::iterator M68kFrameLowering::eliminateCallFramePseudoInstr(
unsigned StackAlign = getStackAlignment();
Amount = alignTo(Amount, StackAlign);
- MachineModuleInfo &MMI = MF.getMMI();
const auto &Fn = MF.getFunction();
- bool DwarfCFI = MMI.hasDebugInfo() || Fn.needsUnwindTableEntry();
+ bool DwarfCFI = Fn.needsUnwindTableEntry();
// If we have any exception handlers in this function, and we adjust
// the SP before calls, we may need to indicate this to the unwinder
``````````
</details>
https://github.com/llvm/llvm-project/pull/99750
More information about the llvm-commits
mailing list