[PATCH] D43781: Make AddrSpaceCast noops on PPC

[Nemanja Ivanovic via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL327877: [PowerPC] Make AddrSpaceCast noop (authored by nemanjai, committed by ).

Repository:
  rL LLVM

https://reviews.llvm.org/D43781

Files:
  llvm/trunk/lib/Target/PowerPC/PPCISelLowering.h
  llvm/trunk/test/CodeGen/PowerPC/addrspacecast.ll


Index: llvm/trunk/test/CodeGen/PowerPC/addrspacecast.ll
===================================================================
--- llvm/trunk/test/CodeGen/PowerPC/addrspacecast.ll
+++ llvm/trunk/test/CodeGen/PowerPC/addrspacecast.ll
@@ -0,0 +1,22 @@
+; RUN: llc -mtriple=powerpc64le-unknown-linux-gnu %s -o - | FileCheck %s
+
+; Check that codegen for an addrspace cast succeeds without error.
+define <4 x i32 addrspace(1)*> @f (<4 x i32*> %x) {
+  %1 = addrspacecast <4 x i32*> %x to <4 x i32 addrspace(1)*>
+  ret <4 x i32 addrspace(1)*> %1
+  ; CHECK-LABEL: @f
+}
+
+; Check that fairly complicated addrspace cast and operations succeed without error.
+%struct = type opaque
+define void @g (%struct addrspace(10)** %x) {
+  %1 = load %struct addrspace(10)*, %struct addrspace(10)** %x
+  %2 = addrspacecast %struct addrspace(10)* %1 to %struct addrspace(11)*
+  %3 = bitcast %struct addrspace(11)* %2 to i8 addrspace(11)*
+  %4 = getelementptr i8, i8 addrspace(11)* %3, i64 16
+  %5 = bitcast i8 addrspace(11)* %4 to %struct addrspace(10)* addrspace(11)*
+  %6 = load %struct addrspace(10)*, %struct addrspace(10)* addrspace(11)* %5
+  store %struct addrspace(10)* %6, %struct addrspace(10)** undef
+  ret void
+  ; CHECK-LABEL: @g
+}
Index: llvm/trunk/lib/Target/PowerPC/PPCISelLowering.h
===================================================================
--- llvm/trunk/lib/Target/PowerPC/PPCISelLowering.h
+++ llvm/trunk/lib/Target/PowerPC/PPCISelLowering.h
@@ -884,6 +884,11 @@
       }
     };
 
+    bool isNoopAddrSpaceCast(unsigned SrcAS, unsigned DestAS) const override {
+      // Addrspacecasts are always noops.
+      return true;
+    }
+
     bool canReuseLoadAddress(SDValue Op, EVT MemVT, ReuseLoadInfo &RLI,
                              SelectionDAG &DAG,
                              ISD::LoadExtType ET = ISD::NON_EXTLOAD) const;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43781.138971.patch
Type: text/x-patch
Size: 1855 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180319/05b24838/attachment.bin>