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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 2 11:47:40 PDT 2017


efriedma added a comment.

When you're considering the overall complexity, there are a couple points to keep in mind:

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


https://reviews.llvm.org/D39415





More information about the llvm-commits mailing list