[PATCH] D58399: [Inliner] Don't initialize ComputeFullInlineCost to be always true because of ORE
    Chandler Carruth via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Feb 19 15:31:54 PST 2019
    
    
  
chandlerc added inline comments.
================
Comment at: lib/Analysis/InlineCost.cpp:2008-2012
+  OptimizationRemarkEmitter NewORE(Callee);
+  if (!ORE &&
+      Callee->getContext().getDiagHandlerPtr()->isMissedOptRemarkEnabled(
+          DEBUG_TYPE))
+    ORE = &NewORE;
----------------
you can just set `ORE` to `nullptr` here?
Repository:
  rL LLVM
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58399/new/
https://reviews.llvm.org/D58399
    
    
More information about the llvm-commits
mailing list