[PATCH] AMDGPU: don't match vgpr loads for constant loads

Matt Arsenault arsenm2 at gmail.com
Fri Jul 24 16:34:39 PDT 2015


> On Jul 24, 2015, at 4:15 PM, Marek Olšák <maraeo at gmail.com> wrote:
> 
> Hi Matt,
> 
> Aren't scalar loads converted to vector loads when the operands are legalized?
> 
> Marek

They should be. There are a handful of tests already for this in salu-to-valu.ll, although their check lines aren’t strict enough to show the suboptimal addressing mode matching caused by doing this. Adding one that uses a large maximum mubuf immediate offset (i.e <4096 bytes) but larger than the SMRD offset can handle (256 dwords) that isn’t folded into the mubuf offset would be useful to track this defect.

-Matt



More information about the llvm-commits mailing list