[PATCH] D16124: Add to the split module utility an SCC based method which allows not to globalize any local variables.

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 15 15:46:30 PST 2016


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

LGTM with this one change. Thanks!


================
Comment at: lib/Transforms/Utils/SplitModule.cpp:160
@@ +159,3 @@
+    unsigned CurrentClusterSize = BalancinQueue.top().second;
+    BalancinQueue.pop();
+
----------------
Sorry, I meant what you wrote in your second paragraph. It isn't all that much work and makes the code slightly simpler.


http://reviews.llvm.org/D16124





More information about the llvm-commits mailing list