[PATCH] R600/SI: Remove isel mubuf legalization

Matt Arsenault arsenm2 at gmail.com
Thu Feb 19 12:29:22 PST 2015


> On Feb 19, 2015, at 8:59 AM, Tom Stellard <thomas.stellard at amd.com> wrote:
> 
> We legalize mubuf instructions post-instruction selection, so this
> code is no longer needed.
> ---
> lib/Target/R600/SIISelLowering.cpp               | 124 -----------------------
> lib/Target/R600/SIISelLowering.h                 |   6 --
> test/CodeGen/R600/si-triv-disjoint-mem-access.ll |   4 +-
> 3 files changed, 1 insertion(+), 133 deletions(-)


LGTM

> 
> -; XXX: Should be able to reorder this, but the laods count as ordered
> -
> ; FUNC-LABEL: @reorder_constant_load_local_store_constant_load
> ; CI: buffer_load_dword
> -; CI: ds_write_b32
> ; CI: buffer_load_dword
> +; CI: ds_write_b32
> ; CI: buffer_store_dword
> define void @reorder_constant_load_local_store_constant_load(i32 addrspace(1)* %out, i32 addrspace(3)* %lptr) #0 {
>   %ptr0 = load i32 addrspace(2)* addrspace(3)* @stored_constant_ptr, align 8
> -- 
> 2.0.4

Not sure why this changed from this
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150219/8d9b4e6a/attachment.html>


More information about the llvm-commits mailing list