Index: clang/lib/CodeGen/CGExprScalar.cpp =================================================================== --- clang/lib/CodeGen/CGExprScalar.cpp +++ clang/lib/CodeGen/CGExprScalar.cpp @@ -4796,12 +4796,10 @@ // to vec4 if the original type is not vec4, then a shuffle vector to // get a vec3. if (NumElementsSrc != 3 && NumElementsDst == 3) { - if (!CGF.CGM.getCodeGenOpts().PreserveVec3Type) { - auto *Vec4Ty = llvm::FixedVectorType::get( - cast(DstTy)->getElementType(), 4); - Src = createCastsForTypeOfSameSize(Builder, CGF.CGM.getDataLayout(), Src, - Vec4Ty); - } + auto *Vec4Ty = llvm::FixedVectorType::get( + cast(DstTy)->getElementType(), 4); + Src = createCastsForTypeOfSameSize(Builder, CGF.CGM.getDataLayout(), Src, + Vec4Ty); Src = ConvertVec3AndVec4(Builder, CGF, Src, 3); Src->setName("astype"); Index: clang/test/CodeGenOpenCL/preserve_vec3.cl =================================================================== --- clang/test/CodeGenOpenCL/preserve_vec3.cl +++ clang/test/CodeGenOpenCL/preserve_vec3.cl @@ -53,3 +53,11 @@ // CHECK: store <4 x i16> %[[ASTYPE]], <4 x i16> addrspace(1)* %[[OUT_BC]] *out = __builtin_astype(a, long); } + +void to_char3(int a, global char3 *out) { + // CHECK-LABEL: void @to_char3 + // CHECK: %[[IN_BC:.*]] = bitcast i32 %a to <4 x i8> + // CHECK: %[[ASTYPE:.*]] = shufflevector <4 x i8> %[[IN_BC]], <4 x i8> poison, <3 x i32> + // CHECK: store <3 x i8> %[[ASTYPE]], <3 x i8> addrspace(1)* %out + *out = __builtin_astype(a, char3); +}