[RISCV] Remove getOffsetOfLocalArea()

[wangpc-pp]

For RISC-V, it's always 0 and I don't see any reason we will
change it in the future.

[llvmbot]

@llvm/pr-subscribers-backend-risc-v

Author: Pengcheng Wang (wangpc-pp)

Changes

For RISC-V, it's always 0 and I don't see any reason we will
change it in the future.

---

Full diff: https://github.com/llvm/llvm-project/pull/93765.diff

1 Files Affected:

• (modified) llvm/lib/Target/RISCV/RISCVFrameLowering.cpp (+2-3)

diff --git a/llvm/lib/Target/RISCV/RISCVFrameLowering.cpp b/llvm/lib/Target/RISCV/RISCVFrameLowering.cpp
index 436bd4a38a319..4931e3a994e38 100644
--- a/llvm/lib/Target/RISCV/RISCVFrameLowering.cpp
+++ b/llvm/lib/Target/RISCV/RISCVFrameLowering.cpp
@@ -878,9 +878,8 @@ RISCVFrameLowering::getFrameIndexReference(const MachineFunction &MF, int FI,
           StackID == TargetStackID::ScalableVector) &&
          "Unexpected stack ID for the frame object.");
   if (StackID == TargetStackID::Default) {
-    Offset =
-        StackOffset::getFixed(MFI.getObjectOffset(FI) - getOffsetOfLocalArea() +
-                              MFI.getOffsetAdjustment());
+    Offset = StackOffset::getFixed(MFI.getObjectOffset(FI) +
+                                   MFI.getOffsetAdjustment());
   } else if (StackID == TargetStackID::ScalableVector) {
     Offset = StackOffset::getScalable(MFI.getObjectOffset(FI));
   }

[jrtc27]

If it exists I feel like we really should be including it, even if it is 0 in practice?

[wangpc-pp]

If it exists I feel like we really should be including it, even if it is 0 in practice?

Just a rough scan, only X86 has different LocalAreaOffset.

[francisvm]

Maybe we should assert it's zero before the if? Otherwise all in favor of simplifying all these offset calculations.