[PATCH] D60031: Split tailcallelim into tailcallmark and tailcallelim

[Nick Lewycky via Phabricator via llvm-commits]

nicholas added inline comments.


================
Comment at: lib/Transforms/Scalar/TailCallMarking.cpp:1
+//===- TailCallMarking.cpp - Mark Tail Calls ----------------===//
+//
----------------
Fix ruler, it's too short.


================
Comment at: lib/Transforms/Scalar/TailRecursionElimination.cpp:525
 }
-
+static bool isAllCallsAreTailCalls(Function &F) {
+  // true if all call instruction which can be reachble entry blcok are marked
----------------
"areAllCallsTailCalls"? "allCallsAreTail"? "allCallsHaveTail"? Or the inverse, "hasAnyNonTailCall"?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60031