[PATCH] D52816: [AArch64] Create proper memoperand for multi-vector stores

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 5 11:23:01 PDT 2018


efriedma added inline comments.


================
Comment at: test/CodeGen/AArch64/multi-vector-store-size.ll:69
+  tail call void @llvm.aarch64.neon.st2lane.v4f32.p0f32(<4 x float> %ar, <4 x float> %br, i64 1, float* %res)
+; CHECK: ST2i32 {{.*}} :: (store 32 {{.*}})
+  tail call void @llvm.aarch64.neon.st3lane.v4f32.p0f32(<4 x float> %ar, <4 x float> %br, <4 x float> %cr, i64 1, float* %res)
----------------
I'm pretty sure these numbers aren't right.  You don't have to fix it here, necessarily; being overconservative isn't really a big deal.  But it would be nice to have a note in the testcase.


Repository:
  rL LLVM

https://reviews.llvm.org/D52816





More information about the llvm-commits mailing list