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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 3 12:30:05 PDT 2018


efriedma added a comment.

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".

I don't think it's necessary to write a test where the final assembly is actually affected by the incorrect memory operand.


Repository:
  rL LLVM

https://reviews.llvm.org/D52816





More information about the llvm-commits mailing list