[PATCH] D32783: [PartialInlining] Add frequency based cost analysis

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 10 21:26:21 PDT 2017


dberris added a comment.

Drive-by-comment.



================
Comment at: lib/Transforms/IPO/PartialInlining.cpp:795-797
+  if (AnyInline && CalleeEntryCount) {
+    F->setEntryCount(CalleeEntryC);
+  }
----------------
nit: No need for {}'s in single-line bodies.


https://reviews.llvm.org/D32783





More information about the llvm-commits mailing list