[PATCH] R600/SI: Implement shouldConvertConstantLoadToIntImm

Daenzer, Michel Michel.Daenzer at amd.com
Tue Mar 18 18:37:25 PDT 2014


On Die, 2014-03-18 at 09:59 -0700, Matt Arsenault wrote:
> On 03/17/2014 08:16 PM, Daenzer, Michel wrote:
> > What's the default behaviour without this? Not all source operands
> > support inline constants, but on the other hand even immediates which
> > cannot be encoded as inline constants can be encoded as literal
> > constants in some cases. So I wonder if this might not be either too
> > liberal or too strict.
> The default implementation is always false

Okay, thanks. I'm not sure about all the implications, but I think your
change should be fine then.


-- 
Earthling Michel Dänzer            |                  http://www.amd.com
Libre software enthusiast          |                Mesa and X developer




More information about the llvm-commits mailing list