PATCH] ARM NEON Lowering: Merge extractelt, bitcast, sitofp sequence

[Jakob Stoklund Olesen]

On Feb 18, 2013, at 7:15 AM, Arnold Schwaighofer <aschwaighofer at apple.com> wrote:

> Is this okay to commit?
> 
> Jakob, Renato, Anton and David Peixotto had a look at it.

Yes, go ahead.

/jakob


> Thanks,
> Arnold
> 
> On Feb 15, 2013, at 1:20 PM, Jakob Stoklund Olesen <stoklund at 2pi.dk> wrote:
> 
>> 
>> On Feb 15, 2013, at 11:16 AM, Arnold Schwaighofer <aschwaighofer at apple.com> wrote:
>> 
>>> I think I am minimal now. I don't think I can get rid of the last remaining COPY_TO_REGCLASS. I think it needs to be there to assure the table gen type checker that we are all good copying from a (2xi32 DPRS:%src) to a (2f32 DPR:$src). But maybe there is a way. I am not sure what other instruction I could use that does not result in a real assembly instruction being generated (like a bitcast would).
>>> 
>>> def : Pat<(f32 (bitconvert (i32 (extractelt (v2i32 DPR:$src), imm:$lane)))),
>>>        (f32 (EXTRACT_SUBREG
>>>              (v2f32 (COPY_TO_REGCLASS (v2i32 DPR:$src), DPR)),    // <<< HERE
>>>              (SSubReg_f32_reg imm:$lane)))>;
>> 
>> Looks good.
>> 
>> Thanks,
>> /jakob
>> 
> <0001-ARM-NEON-Merge-a-f32-bitcast-of-a-v2i32-extractelt.patch>