[PATCH] D60031: Split tailcallelim into tailcallmark and tailcallelim

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: include/llvm/Transforms/Scalar/TailCallMarking.h:1
+#ifndef LLVM_TRANSFORMS_SCALAR_TAILCALLMARKING_H
+#define LLVM_TRANSFORMS_SCALAR_TAILCALLMARKING_H
----------------
License blurb missing


================
Comment at: lib/Transforms/Scalar/TailCallMarking.cpp:8
+//===----------------------------------------------------------------------===//
+
+#include "llvm/Transforms/Scalar/TailCallMarking.h"
----------------
Might be a good idea to add a comment here as to what this does


================
Comment at: lib/Transforms/Scalar/TailRecursionElimination.cpp:526
+static bool isAllCallsAreTailCalls(Function &F) {
+  // true if all call instruction which can be reachble entry blcok are marked
+  // as "tail"
----------------
block


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60031