[PATCH] D13378: [SCEV] Recognize simple br-phi patterns

Diego Novillo via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 6 11:03:04 PDT 2015


On Tue, Oct 6, 2015 at 1:56 PM, Diego Novillo <dnovillo at google.com> wrote:

>
>
> On Tue, Oct 6, 2015 at 1:52 PM, Sanjoy Das <sanjoy at playingwithpointers.com
> > wrote:
>
>>
>> If you're going by "<jokereph_> dnovillo: I just commited r249431 based
>> on sanjoyd commit, you’ll have to revert this one first." on IRC, I think
>> what he meant was that 249431 builds on the br-phi change, not that it
>> fixes the ICE. :)
>>
>
> Oh, drat.  Yeah, I misread.  I'm still having this issue.  I was at an
> earlier point than r249231, so I could just revert your patch.  I will try
> http://reviews.llvm.org/D13460.
>

This new patch fixes the problem.  Any idea when you'll be able to land
it?  While I can build my own compiler with this, I can't apply
out-of-trunk patches to our internal build.  I'd need to revert 249211 and
249431 unless you think D13460 will be committed soon.


Thanks.  Diego.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151006/5c32540b/attachment.html>


More information about the llvm-commits mailing list