[PATCH] R600/SI: Implement isLegalAddressingMode

Matt Arsenault Matthew.Arsenault at amd.com
Fri Aug 8 21:02:54 PDT 2014


The default assumes that a 16-bit signed offset is used.
LDS instruction use a 16-bit unsigned offset, so it wasn't
being used in some cases where it was assumed a negative offset
could be used.
    
More should be done here, but first isLegalAddressingMode needs
to gain an addressing mode argument. For now, copy most of the rest
of the default implementation with the immediate offset change.

http://reviews.llvm.org/D4836

Files:
  lib/Target/R600/SIISelLowering.cpp
  lib/Target/R600/SIISelLowering.h
  test/CodeGen/R600/ds-negative-offset-addressing-mode-loop.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D4836.12324.patch
Type: text/x-patch
Size: 5525 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140809/e0655fbc/attachment.bin>


More information about the llvm-commits mailing list