[PATCH] D84538: [AArch64][SVE] Don't support fixedStack for SVE objects.

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 27 04:11:18 PDT 2020


paulwalker-arm accepted this revision.
paulwalker-arm added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AArch64/AArch64FrameLowering.cpp:2599
   // First process all fixed stack objects.
-  int64_t Offset = 0;
   for (int I = MFI.getObjectIndexBegin(); I != 0; ++I)
+    assert(MFI.getStackID(I) != TargetStackID::SVEVector &&
----------------
Given the for loop is just for the assert should it be protected by debug or something?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84538





More information about the llvm-commits mailing list