[PATCH] R600: Make sign_extend_inreg legal.

[Jan Vesely]

Hi,

On Tue, 2014-04-01 at 11:34 -0700, Matt Arsenault wrote:
> I don't know why I didn't just do this before
> 
> http://llvm-reviews.chandlerc.com/D3250

I have two questions. I'm still learning llvm internals so pls bear with
me.

> +  if (!VT.isVector())
> +    return SDValue();

Is it possible to call the function on scalar types?
I assume the default is Legal, and the patch only changes it to expand for i1, i8, i16.

Also given that the LowerSIGN_EXTEND_INREG, how is it different from using setOperationAction(Expand)?
Is it kept around because of the EG TODO?

thank you,
Jan

PS: I apologize for not responding to that test request yet. These few weeks are rather busy for me.


> 
> Files:
>   lib/Target/R600/AMDGPUISelLowering.cpp
>   lib/Target/R600/EvergreenInstructions.td
>   lib/Target/R600/SIISelLowering.cpp
>   lib/Target/R600/SIInstructions.td
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-- 
Jan Vesely <jan.vesely at rutgers.edu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140402/a7ac6029/attachment.sig>