[PATCH] D53887: [HotColdSplitting] [WIP] Outline more than once per function

[Aditya Kumar via Phabricator via llvm-commits]

hiraditya added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/HotColdSplitting.cpp:480
+
+    LLVM_DEBUG({
+      dbgs() << "Found cold block:\n";
----------------
Creating scope '{}' may not be required.


================
Comment at: llvm/lib/Transforms/IPO/HotColdSplitting.cpp:517
+      if (!isProfitableToOutline(SubRegion, TTI)) {
+        LLVM_DEBUG(dbgs() << "Skipping outlining; not profitable to outline\n");
+        continue;
----------------
Maybe print the region which didn't get outlined, or at least the root-block.


https://reviews.llvm.org/D53887