[llvm] r207565 - Add Simple return instruction to Mips fast-isel

reed kotler rkotler at mips.com
Tue Apr 29 13:40:29 PDT 2014


On 04/29/2014 01:37 PM, Eric Christopher wrote:
> On Tue, Apr 29, 2014 at 1:34 PM, reed kotler <rkotler at mips.com> wrote:
>> On 04/29/2014 12:54 PM, Eric Christopher wrote:
>>>> +    TargetSupported = ((Subtarget->getRelocationModel() == Reloc::PIC_)
>>>> &&
>>>> +                       (Subtarget->hasMips32r2() &&
>>>> (Subtarget->isABI_O32())));
>>>>
>>> Seems limiting?
>>>
>>> -eric
>> I will do the other cases after this.
>>
>> We have a customer that needs this to work for -fPIC, mips32, integrated
>> assembler so that is the first priority.
>>
> Sure, it was just that I can't imagine that the code as checked in
> would matter yet.
>
> -eric
The next checkin which is out for review internally now for sure needs 
that because you can
assign values to global symbols.

But you are right, I could have delayed that other part you pointed out 
till the next checkin.





More information about the llvm-commits mailing list