[PATCH] D59534: Enable tail call in MemCpyOptimization

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 20 12:21:21 PDT 2019


efriedma added a comment.

Rerunning the analysis at some point after memcpyopt definitely makes sense.  I'm not sure where exactly you'd put it, though... I guess after memcpyopt, nothing is really likely to create new tail calls, so just sticking it after memcpyopt might be fine.   Maybe interesting to gather statistics for a few different placements, though.

Actually performing tail recursion elimination after loop optimizations is probably a bad idea, so we might want to split the pass.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59534





More information about the llvm-commits mailing list