[PATCH] D29717: [LoopVectorize] Added address space check when analysing interleaved accesses

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 8 08:07:47 PST 2017


mssimpso added a comment.

Makes sense. I have comment about the test, though. Thanks!



================
Comment at: test/Transforms/LoopVectorize/AArch64/pr31900.ll:1
+; RUN: opt -mtriple=aarch64-apple-ios -loop-vectorize -enable-interleaved-mem-accesses -o /dev/null %s
+
----------------
I would prefer that we actually check the resulting IR rather than checking for "no crash". Maybe force vectorization (-force-vector-width) and then check that the loads are scalarized.


https://reviews.llvm.org/D29717





More information about the llvm-commits mailing list