[PATCH] D31188: [AntiDepBreaker] Use liveins as well in StartBlock

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 26 17:29:26 PDT 2017


MatzeB added a comment.

In https://reviews.llvm.org/D31188#766326, @timshen wrote:

> In https://reviews.llvm.org/D31188#765178, @MatzeB wrote:
>
> > Ok, now that r303938 / https://reviews.llvm.org/D32464 has landed could you please double check and then revert this?
>
>
> https://reviews.llvm.org/D32464 doesn't seem to fix this test case - I ran pass post-RA-sched on this test case, and got bb.2.if.then not having X29 as live-in. Can you take a look?


The original testcase? The mir test in the repository already has the invalid livein lists from the earlier branch folding.


https://reviews.llvm.org/D31188





More information about the llvm-commits mailing list