[PATCH] D60031: Split tailcallelim into tailcallmark and tailcallelim

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 2 15:56:48 PDT 2019


jdoerfert added a comment.

I might have missed some earlier discussion but how does this solve the problem? Also, I assume the problem is still PR41047, correct?



================
Comment at: lib/Transforms/Scalar/TailCallMarking.cpp:13
+//
+//===----------------------------------------------------------------------===//
+
----------------
I would have "just" created the second pass in the original TailRecursionElimination.cpp file but separating it is fine too. (One benefit would have been a much smaller diff ;))


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60031





More information about the llvm-commits mailing list