[PATCH] D94401: [AArch64][SVE] Allow accesses to SVE stack objects to use frame pointer

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64MachineFunctionInfo.h:251-253
+  bool isCalleeSavedStackSizeComputed() const {
+    return HasCalleeSavedStackSize;
+  }
----------------
bsmith wrote:
> sdesmalen wrote:
> > You should be able to use `MFI.isCalleeSavedInfoValid()` (instead of having to add this function).
> CalleeSavedInfoValid is set before the callee saved stack size is set, hence unfortunately this isn't sufficient. Doing this triggers an assertion: 
> 
>   unsigned int llvm::AArch64FunctionInfo::getCalleeSavedStackSize() const: Assertion `HasCalleeSavedStackSize && "CalleeSavedStackSize has not been calculated"' failed.
> 
> Given that the callee saved stack size being set is protected by `HasCalleeSavedStackSize` I think it makes sense to expose that value.
Are you sure? `MFI.setCalleeSavedInfoValid(true)` is called in PEI after calling `TFI->determineCalleeSaves()` which sets `HasCalleeSavedStackSize`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94401/new/

https://reviews.llvm.org/D94401