[PATCH] D47949: [callsitesplit] Limit the # of predecessors walk when recording condition

Xin Tong via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 11 09:07:11 PST 2019


Hi Florian

I saw the patch (https://reviews.llvm.org/D44627) was landed and
reverted the patch on Nov 9, so I waited.

I see it got recommitted on Nov 14. I will measure it if I can find
the test case we had before.

Thanks
-Xin

On Fri, Jan 11, 2019 at 8:58 AM Florian Hahn via Phabricator via
llvm-commits <llvm-commits at lists.llvm.org> wrote:
>
> fhahn added a comment.
>
> In D47949#1292403 <https://reviews.llvm.org/D47949#1292403>, @trentxintong wrote:
>
> > Thanks for the change @fhahn. I will dig up my test case and run it.
>
>
> @trentxintong did you manage to get any numbers for the test case?
>
>
> Repository:
>   rL LLVM
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D47949/new/
>
> https://reviews.llvm.org/D47949
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits


More information about the llvm-commits mailing list