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

Reply via email to