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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
