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

[Eugene Leviant via Phabricator via llvm-commits]

evgeny777 added a comment.

> 1. DAGCombine calls CombineBaseUpdate for each relevant node in the DAG.
> 2. isPredecessorOf is O(N) in the size of the DAG.



1. That's true, I see no difference regarding this between old and new approaches.
2. I didn't know that, thanks! I'll check if it is possible to optimize


https://reviews.llvm.org/D39415