Anastasia added a comment.

Btw, is there any reason to add testing specifically for half? Is there 
anything specific to half in the implementation of this?



================
Comment at: test/CodeGenOpenCL/kernel-arg-info.cl:79
+kernel void foo6_2(global const volatile half * restrict 
globalconstvolatilehalfrestrictp,
+                   local half*localhalfp,
+                   local half *restrict localhalfrestrictp,
----------------
could you add space b/w half and *. Also could we format consistently below?


https://reviews.llvm.org/D35000



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to