r215629 - Fix a crash when compiling blocks in OpenCL with multiple

Pekka Jaaskelainen pekka.jaaskelainen at tut.fi
Thu Aug 14 02:37:51 PDT 2014


Author: pjaaskel
Date: Thu Aug 14 04:37:50 2014
New Revision: 215629

URL: http://llvm.org/viewvc/llvm-project?rev=215629&view=rev
Log:
Fix a crash when compiling blocks in OpenCL with multiple
address spaces.


Added:
    cfe/trunk/test/CodeGen/blocks-opencl.cl
Modified:
    cfe/trunk/lib/CodeGen/CGBlocks.cpp

Modified: cfe/trunk/lib/CodeGen/CGBlocks.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGBlocks.cpp?rev=215629&r1=215628&r2=215629&view=diff
==============================================================================
--- cfe/trunk/lib/CodeGen/CGBlocks.cpp (original)
+++ cfe/trunk/lib/CodeGen/CGBlocks.cpp Thu Aug 14 04:37:50 2014
@@ -78,7 +78,13 @@ static llvm::Constant *buildBlockDescrip
   ASTContext &C = CGM.getContext();
 
   llvm::Type *ulong = CGM.getTypes().ConvertType(C.UnsignedLongTy);
-  llvm::Type *i8p = CGM.getTypes().ConvertType(C.VoidPtrTy);
+  llvm::Type *i8p = NULL;
+  if (CGM.getLangOpts().OpenCL)
+    i8p = 
+      llvm::Type::getInt8PtrTy(
+           CGM.getLLVMContext(), C.getTargetAddressSpace(LangAS::opencl_constant));
+  else
+    i8p = CGM.getTypes().ConvertType(C.VoidPtrTy);
 
   SmallVector<llvm::Constant*, 6> elements;
 

Added: cfe/trunk/test/CodeGen/blocks-opencl.cl
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/blocks-opencl.cl?rev=215629&view=auto
==============================================================================
--- cfe/trunk/test/CodeGen/blocks-opencl.cl (added)
+++ cfe/trunk/test/CodeGen/blocks-opencl.cl Thu Aug 14 04:37:50 2014
@@ -0,0 +1,16 @@
+// RUN: %clang_cc1 -O0 %s -ffake-address-space-map -emit-llvm -o - -fblocks -triple x86_64-unknown-unknown | FileCheck %s
+// This used to crash due to trying to generate a bitcase from a cstring
+// in the constant address space to i8* in AS0.
+
+void dummy(float (^op)(float))
+{
+}
+
+// CHECK: i8 addrspace(3)* getelementptr inbounds ([9 x i8] addrspace(3)* @.str, i32 0, i32 0)
+
+kernel void test_block()
+{
+  float (^X)(float) = ^(float x) { return x + 42.0f; };
+  dummy(X);
+}
+





More information about the cfe-commits mailing list