[CIR][CodeGen] Fix address space of result pointer type of array decay cast op (llvm#812)

There are two occurrences of `cir.cast(array_to_ptrdecay, ...)` that
drop address spaces unexpectedly for its result pointer type. This PR
fixes them with the source address space.

```mlir
// Before
%1 = cir.cast(array_to_ptrdecay, %0 : !cir.ptr<!cir.array<!s32i x 32>, addrspace(offload_local)>), !cir.ptr<!s32i>
// After
%1 = cir.cast(array_to_ptrdecay, %0 : !cir.ptr<!cir.array<!s32i x 32>, addrspace(offload_local)>), !cir.ptr<!s32i, addrspace(offload_local)>
```

Author: seven-mile

clang/lib/CIR/CodeGen/CIRGenBuilder.cpp

@@ -19,8 +19,10 @@ mlir::Value CIRGenBuilderTy::maybeBuildArrayDecay(mlir::Location loc,
       ::mlir::dyn_cast<::mlir::cir::ArrayType>(arrayPtrTy.getPointee());
 
   if (arrayTy) {
+    auto addrSpace = ::mlir::cast_if_present<::mlir::cir::AddressSpaceAttr>(
+        arrayPtrTy.getAddrSpace());
     mlir::cir::PointerType flatPtrTy =
-        mlir::cir::PointerType::get(getContext(), arrayTy.getEltType());
+        getPointerTo(arrayTy.getEltType(), addrSpace);
     return create<mlir::cir::CastOp>(
         loc, flatPtrTy, mlir::cir::CastKind::array_to_ptrdecay, arrayPtr);
   }

clang/lib/CIR/CodeGen/CIRGenExprAgg.cpp

@@ -503,8 +503,10 @@ void AggExprEmitter::buildArrayInit(Address DestPtr, mlir::cir::ArrayType AType,
   QualType elementPtrType = CGF.getContext().getPointerType(elementType);
 
   auto cirElementType = CGF.convertType(elementType);
-  auto cirElementPtrType = mlir::cir::PointerType::get(
-      CGF.getBuilder().getContext(), cirElementType);
+  auto cirAddrSpace = mlir::cast_if_present<mlir::cir::AddressSpaceAttr>(
+      DestPtr.getType().getAddrSpace());
+  auto cirElementPtrType =
+      CGF.getBuilder().getPointerTo(cirElementType, cirAddrSpace);
   auto loc = CGF.getLoc(ExprToVisit->getSourceRange());
 
   // Cast from cir.ptr<cir.array<elementType> to cir.ptr<elementType>

clang/lib/CIR/Dialect/IR/CIRDialect.cpp

@@ -495,6 +495,11 @@ LogicalResult CastOp::verify() {
     if (!arrayPtrTy || !flatPtrTy)
       return emitOpError() << "requires !cir.ptr type for source and result";
 
+    if (arrayPtrTy.getAddrSpace() != flatPtrTy.getAddrSpace()) {
+      return emitOpError()
+             << "requires same address space for source and result";
+    }
+
     auto arrayTy = llvm::dyn_cast<mlir::cir::ArrayType>(arrayPtrTy.getPointee());
     if (!arrayTy)
       return emitOpError() << "requires !cir.array pointee";

clang/test/CIR/CodeGen/OpenCL/array-decay.cl

@@ -0,0 +1,25 @@
+// RUN: %clang_cc1 -cl-std=CL3.0 -O0 -fclangir -emit-cir -triple spirv64-unknown-unknown %s -o %t.cir
+// RUN: FileCheck --input-file=%t.cir %s --check-prefix=CIR
+// RUN: %clang_cc1 -cl-std=CL3.0 -O0 -fclangir -emit-llvm -triple spirv64-unknown-unknown %s -o %t.ll
+// RUN: FileCheck --input-file=%t.ll %s --check-prefix=LLVM
+
+// CIR: @func1
+// LLVM: @func1
+kernel void func1(global int *data) {
+    local int arr[32];
+
+    local int *ptr = arr;
+    // CIR:      cir.cast(array_to_ptrdecay, %{{[0-9]+}} : !cir.ptr<!cir.array<!s32i x 32>, addrspace(offload_local)>), !cir.ptr<!s32i, addrspace(offload_local)>
+    // CIR-NEXT: cir.store %{{[0-9]+}}, %{{[0-9]+}} : !cir.ptr<!s32i, addrspace(offload_local)>, !cir.ptr<!cir.ptr<!s32i, addrspace(offload_local)>, addrspace(offload_private)>
+
+    // LLVM: store ptr addrspace(3) @func1.arr, ptr %{{[0-9]+}}
+}
+
+// CIR: @func2
+// LLVM: @func2
+kernel void func2(global int *data) {
+    private int arr[32] = {data[2]};
+    // CIR: %{{[0-9]+}} = cir.cast(array_to_ptrdecay, %{{[0-9]+}} : !cir.ptr<!cir.array<!s32i x 32>, addrspace(offload_private)>), !cir.ptr<!s32i, addrspace(offload_private)>
+
+    // LLVM: %{{[0-9]+}} = getelementptr i32, ptr %3, i32 0
+}

clang/test/CIR/IR/invalid.cir

@@ -1286,3 +1286,16 @@ module {
     cir.return
   }
 }
+
+// -----
+
+!s32i = !cir.int<s, 32>
+
+module {
+  cir.func @array_to_ptrdecay_addrspace() {
+    %0 = cir.alloca !cir.array<!s32i x 32>, !cir.ptr<!cir.array<!s32i x 32>, addrspace(offload_private)>, ["x", init]
+    // expected-error@+1 {{requires same address space for source and result}}
+    %1 = cir.cast(array_to_ptrdecay, %0 : !cir.ptr<!cir.array<!s32i x 32>, addrspace(offload_private)>), !cir.ptr<!s32i>
+    cir.return
+  }
+}