[PATCH] D52816: [AArch64] Create proper memoperand for multi-vector stores
David Greene via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 4 07:14:26 PDT 2018
greened added a comment.
In https://reviews.llvm.org/D52816#1254107, @efriedma wrote:
> You could just directly test that the computed memory operand is correct: write a test that runs "llc -stop-after=isel" and check the MIR. Without your patch, you should see something like "ST1Fourv2d killed %5, %4 :: (store 48 into %ir.addr, align 64)"; with your patch, that will be "store 64".
Ah yeah, that's a great idea. I'll do that. Thanks!
Repository:
rL LLVM
https://reviews.llvm.org/D52816
More information about the llvm-commits
mailing list