[PATCH] R600: Check if a sextload should be used for parameter loads.

Tom Stellard tom at stellard.net
Fri Apr 11 12:33:36 PDT 2014


On Fri, Apr 11, 2014 at 11:05:29AM -0700, Matt Arsenault wrote:
> 
>   Ping
> 
> http://reviews.llvm.org/D3300
> 

Changing the intrinsics that are mapped to inputs from sext to zext
makes sense to me, but I don't understand why we need to do it for the
other scalar arguments.  Values less than 32-bit are extended prior to
being passed to the kernel so the ext type of the load shouldn't matter.

-Tom

> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



More information about the llvm-commits mailing list