[PATCH] R600/SI: Consider adjacent offsets in getLdStBaseRegImmOfs

Tom Stellard tom at stellard.net
Tue Jul 29 17:43:54 PDT 2014


On Tue, Jul 29, 2014 at 05:30:18PM -0700, Matt Arsenault wrote:
> On 07/29/2014 05:20 PM, Tom Stellard wrote:
> > I think the offsets are always byte offsets.  Have you tested this
> > with piglit?
> >
> > -Tom
> 
> The offsets are in bytes for the single element loads, but read2 the 
> offsets are in elements. See table 12.7 in the CI manual. ds_read_b32 
> says it does [DSADDR + offset0 * 4] / [DSADDR + offset1 * 4]
> 

OK, I see now.  LGTM.

-Tom

>   It wouldn't matter to piglit right now since nothing is producing 
> read2s yet



More information about the llvm-commits mailing list