[PATCH] D16876: [OpenCL] Refine pipe builtin support

[Anastasia Stulova via cfe-commits]

Anastasia added inline comments.

================
Comment at: lib/Sema/SemaChecking.cpp:294
@@ +293,3 @@
+             diag::err_opencl_builtin_pipe_invalid_access_modifier)
+          << "read_only" << Arg0->getSourceRange();
+      return true;
----------------
Anastasia wrote:
> Could we use getName() instead?
> 
> We could then also move this statement after the switch and just set an error flag here.
Just to be clear getName() of the attr instead of passing the string (i.e. "read_only").


http://reviews.llvm.org/D16876