[PATCH] D32563: Add LiveRangeShrink pass to shrink live range within BB.
Wei Mi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 1 16:58:22 PDT 2017
wmi added inline comments.
================
Comment at: lib/Transforms/Scalar/LiveRangeShrink.cpp:58-60
+ InstOrderMap[I] = InstOrderMap[Insert];
+ for (Instruction *U = Insert; U != I; U = U->getNextNode())
+ InstOrderMap[U]++;
----------------
wmi wrote:
> Can we remove the update loop of InstOrderMap? It leads to O(N^2) complexity.
>
> One way is just to let InstOrderMap[I] = InstOrderMap[Insert] and keep the rest the same.
Please ignore this for now. Sorry. It is an incomplete comment sent out accidently.
https://reviews.llvm.org/D32563
More information about the llvm-commits
mailing list