[Mesa-dev] R600 Patches: Add support for the local address space

[Tom Stellard]

On Wed, Jun 12, 2013 at 06:37:39PM -0700, Matt Arsenault wrote:
> On 06/12/2013 05:42 PM, Tom Stellard wrote:
> >Hi,
> >
> >The attached patches add support for local address space on
> >Evergreen / Northern Islands GPUs.
> >
> >Please Review.
> >
> >-Tom
> > +  def int_AMDGPU_barrier_local  : Intrinsic<[], [], []>;
> You probably want to mark this as IntrReadMem to try to avoid
> reordering stores around the barrier
>

I don't think the intrinsic as defined will have stores reordered around
it.  From include/llvm/IR/Intrinsics.td:

// Intr*Mem - Memory properties.  An intrinsic is allowed to have at most one of
// these properties set.  They are listed from the most aggressive (best to use
// if correct) to the least aggressive.  If no property is set, the worst case
// is assumed (it may read and write any memory it can get access to and
// it may have other side effects).

-Tom