[PATCH] R600/SI: Implement shouldConvertConstantLoadToIntImm
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