majnemer added a subscriber: majnemer. majnemer requested changes to this revision. majnemer added a reviewer: majnemer. majnemer added a comment. This revision now requires changes to proceed.
This approach seems wrong to me. Instead, why not just make `ndrange_t` a typedef of a real struct in `Sema::Initialize`? ================ Comment at: lib/CodeGen/CGOpenCLRuntime.cpp:32-36 @@ +31,7 @@ + + llvm::SmallVector<llvm::Type*,4> EleTypes; + llvm::Type* ArrEleType = + llvm::ArrayType::get(llvm::IntegerType:: + get(CGM.getLLVMContext(), + CGM.PointerWidthInBits), 3); + ---------------- This formatting looks wrong. Repository: rL LLVM https://reviews.llvm.org/D23086 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
