[PATCH] D89911: [PartialInliner]: Handle code regions in a switch stmt cases

Ettore Tiotto via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 30 11:42:33 PDT 2020


etiotto marked 3 inline comments as done.
etiotto added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/PartialInlining.cpp:537
+
+        LLVM_DEBUG(dbgs() << "ABORT: Outline region cost is smaller than "
+                          << MinOutlineRegionCost << "\n";);
----------------
fhahn wrote:
> I am not sure why we say `ABORT: ` here but not at the other, similar continues?
I'll add similar debug messages in the other 'continue' conditions


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89911/new/

https://reviews.llvm.org/D89911



More information about the llvm-commits mailing list