[PATCH 1/2] R600: Promote i64 stores to v2i32

Tom Stellard tom at stellard.net
Thu Jun 19 08:53:30 PDT 2014


On Wed, Jun 18, 2014 at 04:16:46PM -0700, Matt Arsenault wrote:
> On 06/18/2014 04:10 PM, Tom Stellard wrote:
> >+  // These stores are legal.
> >+  if (Store->getAddressSpace() == AMDGPUAS::LOCAL_ADDRESS &&
> >+      VT.isVector() && VT.getVectorNumElements() == 2)
> >+    return SDValue();
> >+
> This won't work for the small type vectors. My (currently broken
> again) huge patch set for fixing all of the vector loads and stores
> handles this though so it probably doesn't matter for now

Attached is an updated patch that checks for an element type of i32.
This can be temporary until your fixes go in, this patch is just a
prerequisite for implementing global atomics.

-Tom

> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-R600-Promote-i64-stores-to-v2i32.patch
Type: text/x-diff
Size: 3871 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140619/9f71f046/attachment.patch>


More information about the llvm-commits mailing list