[clang] [llvm] [win][aarch64] Always reserve frame pointers for Arm64 Windows (PR #146582)

[David Truby via llvm-commits]

================
@@ -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