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

Brian Gesiak via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 19 13:55:52 PDT 2017


modocache 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)
----------------
Is this comment addition intentional?


https://reviews.llvm.org/D35570





More information about the llvm-commits mailing list