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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
