[PATCH] D14477: Remove unnecessary call to getAllocatableRegClass

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 11 17:12:38 PST 2015


> On Nov 11, 2015, at 5:10 PM, escha <escha at apple.com> wrote:
> 
> Yup, the assert does trigger.

> It’s a rather weird target, so I wouldn’t be surprised if it does nonintuitive things that don’t make any good sense.

Heh, feel free to revert, I’ll get back to you to try to understand why we end up in this situation.

Sorry for the breakage!

Q.
> 
> —escha
> 
>> On Nov 11, 2015, at 5:08 PM, Quentin Colombet <qcolombet at apple.com> wrote:
>> 
>> Hi Fiona,
>> 
>> Does the assert trigger?
>> 
>> Cheers,
>> -Quentin
>>> On Nov 11, 2015, at 4:01 PM, escha <escha at apple.com> wrote:
>>> 
>>> escha added a subscriber: escha.
>>> escha added a comment.
>>> 
>>> This broke one of our out-of-tree targets.
>>> 
>>> —escha
>>> 
>>> 
>>> http://reviews.llvm.org/D14477
>>> 
>>> 
>>> 
>> 
> 



More information about the llvm-commits mailing list