[llvm] r300878 - ARM: handle post-indexed NEON ops where the offset isn't the access width.

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 13:54:10 PDT 2017


> We might want to consider keeping this check; even if the transform is legal, it's likely not profitable given we have to materialize the immediate.

I was a bit ambivalent about that too. The main reason it's there is that otherwise the whole thing is virtually untestable -- the kind of IR that only becomes obviously constant on the 3rd run of combine isn't the kind of IR you want in your regression tests (see the attachment to https://bugs.llvm.org/show_bug.cgi?id=32658).

I'm still willing to try and come up with something if we really want (I might be able to tweak that example to cover the other kinds of load/store, though it'll always be fragile).

Cheers.

Tim.


More information about the llvm-commits mailing list