bader added a comment. In http://reviews.llvm.org/D12855#246004, @pekka.jaaskelainen wrote:
> The patch seems straightforward enough. BTW does someone know if anyone has > worked on the 'pipe' qualifier? I'm currently supporting Intel's implementation of 'pipe' qualifier and I'd like to contribute this implementation to the community if there is any interest. ================ Comment at: lib/Serialization/ASTCommon.cpp:70 @@ -69,1 +69,3 @@ case BuiltinType::OCLImage2dArray: ID = PREDEF_TYPE_IMAGE2D_ARR_ID; break; + case BuiltinType::OCLImage2dDepth: + ID = PREDEF_TYPE_IMAGE2D_DEP_ID; ---------------- pekka.jaaskelainen wrote: > A nitpick, but why deviate from the style where each case has been stuffed to > a single line? My guess it's effect of clang-format. Original patch sent by Pedro matched the style, but violated 80-character limit, so he was asked to run clang-format over the patch. Do you want me to run clang-format on surrounding code to unify the style? http://reviews.llvm.org/D12855 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
