[PATCH] D16039: Use proper dataflow ordering to speed convergence.This will converge the testcase on bug 26055 in 2 iterations.(data structures speedups to come to make even that faster)

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 9 20:11:04 PST 2016


echristo accepted this revision.
echristo added a comment.
This revision is now accepted and ready to land.

LGTM, thanks. One silly nit.

-eric


================
Comment at: lib/CodeGen/LiveDebugValues.cpp:366
@@ +365,3 @@
+  // ranges converge. 
+  // Ranges have converged when both  worklist is empty.
+  while (!Worklist.empty() || !Pending.empty()) {
----------------
Nit: "when both worklists are empty"


http://reviews.llvm.org/D16039





More information about the llvm-commits mailing list