[PATCH] D70584: [AutoFDO] Statistic for context sensitive profile guided inlining

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 11 21:41:19 PST 2019


wenlei marked an inline comment as done.
wenlei added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:879
   if (Cost.isNever()) {
-    ORE->emit(OptimizationRemark(DEBUG_TYPE, "Not inline", DLoc, BB)
+    ORE->emit(OptimizationRemark("sample-profile-inline", "NotInline", DLoc, BB)
               << "incompatible inlining");
----------------
wmi wrote:
> wenlei wrote:
> > wmi wrote:
> > > The first parameter in the declaration of OptimizationRemark is "const char *PassName", so why not use DEBUG_TYPE?  
> > > 
> > > I feel "NeverInline" may be more clear than "NotInline" in terms of showing it is illegal to inline. 
> > DEBUG_TYPE is "sample-profile" and I feel it's too broad for practical uses as it covers sample usages, weight propagation, and inlining. I wanted to have something that only gives me inlining remarks, thus using  "sample-profile-inline" instead here.
> > 
> > Good point about "NeverInline", will change. 
> Then maybe define a macro for it like #define CSINLINE DEBUG_TYPE "-inline".
thanks, macro added.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70584





More information about the llvm-commits mailing list