================
@@ -4694,12 +4790,8 @@ void
AArch64FrameLowering::processFunctionBeforeFrameFinalized(
assert(getStackGrowthDirection() == TargetFrameLowering::StackGrowsDown &&
"Upwards growing stack unsupported");
- int MinCSFrameIndex, MaxCSFrameIndex;
- int64_t SVEStackSize =
- assignSVEStackObjectOffsets(MFI, MinCSFrameIndex, MaxCSFrameIndex);
-
- AFI->setStackSizeSVE(alignTo(SVEStackSize, 16U));
- AFI->setMinMaxSVECSFrameIndex(MinCSFrameIndex, MaxCSFrameIndex);
+ auto [ZPRStackSize, PPRStackSize] = assignSVEStackObjectOffsets(MF);
----------------
sdesmalen-arm wrote:
nit: There's little value in returning from `assignSVEStackObjectOffsets` if
their only use is them being passed to `setStackSizeSVE`, so I think assignment
can now probably be moved to `determineSVEStackObjectOffsets` (predicated under
`AssignOffsets`), removing the need for the aliases that then just call
`determineSVEStackObjectOffsets`.
https://github.com/llvm/llvm-project/pull/142391
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
