[PATCH] D41468: AMDGPU: Implement getTgtMemIntrinsic for images

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 20 15:43:04 PST 2017

arsenm created this revision.
arsenm added reviewers: rampitec, mareko.
Herald added subscribers: t-tye, tpr, dstuttard, yaxunl, nhaehnle, wdng, kzhuravl.

Currently all images are lowered to have a single
image PseudoSourceValue. Image stores happen to have
overly strict mayLoad/mayStore/hasSideEffects flags
set on them, so this happens to work. When these
are fixed to be correct, the scheduler breaks
this because the identical PSVs are assumed to
be the same address. These need to be unique
to the image resource value.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41468.127791.patch
Type: text/x-patch
Size: 10995 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171220/87b06564/attachment.bin>

More information about the llvm-commits mailing list