[PATCH] D35570: [ORE] Port TailRecursionElimination to the new API

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 19 14:02:29 PDT 2017


davide added inline comments.


================
Comment at: lib/Transforms/Scalar/TailRecursionElimination.cpp:304
       // escape point wouldn't have been put into DeferredTails.
-      emitOptimizationRemark(F.getContext(), "tailcallelim", F,
-                             CI->getDebugLoc(),
-                             "marked this call a tail call candidate");
+      // using namespace ore;
+      ORE->emit(OptimizationRemark(DEBUG_TYPE, "tailcallelim", CI)
----------------
modocache wrote:
> Is this comment addition intentional?
No, slipped through the cracks. I'll remove.


https://reviews.llvm.org/D35570





More information about the llvm-commits mailing list