[PATCH] D35606: Move TailRecursionElimination to new OptimizationRemarkEmitter API

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 18 21:43:44 PDT 2017


davide added a comment.

In https://reviews.llvm.org/D35606#814098, @lenary wrote:

> I have just discovered that I have a failing test, in `test/Other/new-pm-thinlto-defaults.ll` Yours will probably fail there too.
>
> @davide I'll defer to you and your patch on this, given you have comments etc. No worries from me about my duplicated effort.


OK. FWIW, if you want to work on any other pass conversion you may want to assign the PR to you to avoid duplication (I was sloppy in this regard, but hey, I didn't think somebody else was interested otherwise I wouldn't have started :)
Also, you may want to subscribe to the `llvm-commits` mailing list because phabricator reviews mail are sent there.


https://reviews.llvm.org/D35606





More information about the llvm-commits mailing list