[PATCH] D60031: Split tailcallelim into tailcallmark and tailcallelim

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 1 11:20:47 PDT 2019


efriedma added inline comments.


================
Comment at: lib/Transforms/Scalar/TailRecursionElimination.cpp:531
+  Worklist.push_back(&F.getEntryBlock());
+  while (!Worklist.empty()) {
+    auto *BB = Worklist.pop_back_val();
----------------
There's a helper in llvm/ADT/DepthFirstIterator.h which can automatically handle all the boilerplate for iterating over all the reachable blocks in a function.


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