================
@@ -518,6 +518,27 @@ bool AArch64FrameLowering::hasFPImpl(const MachineFunction
&MF) const {
return false;
}
+/// Should the Frame Pointer be reserved for the current function?
+bool AArch64FrameLowering::isFPReserved(const MachineFunction &MF) const {
+ const TargetMachine &TM = MF.getTarget();
+ const Triple &TT = TM.getTargetTriple();
+
+ // These OSes require the frame chain is valid, even if the current frame
does
+ // not use a frame pointer.
+ if (TT.isOSDarwin() || TT.isOSWindows())
----------------
DavidTruby wrote:
I think just removing the clang change will fix the fact that this fails for
flang, so it might be worth removing for that reason
https://github.com/llvm/llvm-project/pull/146582
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
