[PATCH] D145497: [AArch64] Allocate emergency spillslot when using fixed-length SVE.

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 7 07:46:43 PST 2023


sdesmalen added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/framelayout-sve-scavengingslot.mir:30
+---
+# This test verifies that an emergency scavenging slot is allocated when using fixed-length SVE
+name: RequiresScaveningSlotFixedLengthSVE
----------------
peterwaller-arm wrote:
> Since this test lacks check lines I presume it is meant to check only that llc exits successfully. Could you have checklines or a comment explaining what failure it is meant to catch?
Sorry I forgot to put in the CHECK lines. Thanks for catching this!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145497



More information about the llvm-commits mailing list