[PATCH] D112311: [AArch64] Handle ST1iN instructions in isAArch64FrameOffsetLegal

[Oliver Stannard (Linaro) via Phabricator via llvm-commits]

ostannard added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:4293
+  case AArch64::ST1i16:
+  case AArch64::ST1i32:
   case AArch64::IRG:
----------------
Should this also handle `ST1i64`, and the `LD1` opcodes?


================
Comment at: llvm/test/CodeGen/AArch64/aarch64st1.ll:2
+; Check that it doesn't crash with unhandled opcode error, see pr52249
+; RUN: llc < %s -o -| FileCheck %s
+
----------------
This would be better as an MIR test, so it's not dependent on unrelated details of codegen. If I run this test with `-stop-before=localstackalloc` to generate a MIR file, then run that back through llc with `-run-pass=localstackalloc`, I get the same same assertion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112311