[PATCH] D39415: [ARMISelLowering] Better handling of NEON load/store for sequential memory regions

[Renato Golin via Phabricator via llvm-commits]

rengolin added a comment.

In https://reviews.llvm.org/D39415#917654, @evgeny777 wrote:

> CTMark benchmarks (Baseline corresponds to non-patched version)
>  F5473128: Selection_001.png <https://reviews.llvm.org/F5473128>
> CFLAGS:
>
>   -target armv7-unknown-linux-gnueabihf  -O3 -mcpu=cortex-a15 -fomit-frame-pointer


Thanks! Unfortunately, ClamAV is dominated by I/O, so probably just noise.

The ones that probably responded are lencode and kc, but they responded on opposite directions, equally well/badly.

So far, this change has been good for code-size, but not much else. If/when this change is likely to be accepted, it will probably have to rely on size-opts profile being chosen.


https://reviews.llvm.org/D39415