[PATCH] D64867: [OpenCL] Update comments/diagnostics to refer to C++ for OpenCL mode

[Neil Hickey via Phabricator via cfe-commits]

neil.hickey added inline comments.


================
Comment at: lib/Frontend/InitPreprocessor.cpp:414
       else
-        llvm_unreachable("Unsupported OpenCL C++ version");
+        llvm_unreachable("Unsupported C++ version for OpenCL");
       Builder.defineMacro("__CL_CPP_VERSION_1_0__", "100");
----------------
Shouldn't this be C++  for OpenCL version as in other places you use it as a noun


================
Comment at: lib/Parse/ParseDecl.cpp:3562
     case tok::kw_virtual:
-      // OpenCL C++ v1.0 s2.9: the virtual function qualifier is not supported.
+      // C++ for OpenCL does not allow virtual function qualifier to avoid
+      // function pointers restricted in OpenCL v2.0 s6.9.a.
----------------
need a comma between qualifier and to to remove ambiguity


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64867/new/

https://reviews.llvm.org/D64867