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

[Eugene Leviant via Phabricator via llvm-commits]

evgeny777 added a comment.

> After looking at your PDF, it sounds as though you don't need the while loop at all.

Can you please elaborate? Given this type of chain

  [ Store(Value, Addr) ]  ------------> [ VST1_UPD ]

The only user of Addr operand (VST1_UPD) is Store(Value, Addr). What should I do next?
Also sort (qsort) introduces additional O(N*logN) complexity in average case and O(N*N) in the worst case.

> Also, unless you can run the check-all

I **ran** check-llvm and also I ran LLVM test-suite, as described here:
http://llvm.org/docs/lnt/quickstart.html

However I don't have any linux ARM devices and running it on Android looks difficult. 
I'll try jailbroken iPad tomorrow which might probably be easier


https://reviews.llvm.org/D39415