[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#911672, @evgeny777 wrote:

> **Samsung Galaxy Nexus (TI OMAP 4460 Dual-core Cortex-A9 1,2 GHz): **
>  **Huawei Honor 8 (Cortex A72, 2.3GHz):**


Right, those are good platforms. But we need more detailed benchmarks (which @eastig is going to run, but I encourage @evandro and @rovka to do the same on others) before we can settle the regression question.

>> From @javed.absar's comment, I'm not the only one finding this patch non-intuitive, even with the good amount of comments.
> 
> I'm preparing algorithm description, so I suggest returning to discussing this later.

Excellent, thank you!


Repository:
  rL LLVM

https://reviews.llvm.org/D39415