mantognini added a comment. Thanks for the update. Looks good overall. A minor question about backticks. I'm no native English speaker, so I would recommend a second review from someone else too.
================ Comment at: clang/docs/OpenCLSupport.rst:69 ++==============================+==============================================================+======================+===========================================================================+ +| Command line interface | New value for `-cl-std` flag | :good:`done` | https://reviews.llvm.org/D88300 | ++------------------------------+--------------------------------------------------------------+----------------------+---------------------------------------------------------------------------+ ---------------- Double backticks? ================ Comment at: clang/docs/UsersManual.rst:2877 + +Alternatively the internal header `opencl-c.h` containing the declarations +can be included manually using ``-include`` or ``-I`` followed by the path ---------------- Do you also need to use double back-ticks here? I don't remember. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93942/new/ https://reviews.llvm.org/D93942 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits