[Openmp-commits] [PATCH] D97088: [OpenMP][NFC] clang-format the whole openmp project

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Feb 19 14:34:35 PST 2021


jdoerfert added a comment.

I scrolled through all of it. 3 nits. LGTM otherwise, wait for @AndreyChurbanov though.



================
Comment at: openmp/runtime/src/thirdparty/ittnotify/ittnotify.h:480
+                                                                     *tools
+                                                                     *********************************************************************/
 /** @{ */
----------------
This needs to be fixed. Change the comment style or sth


================
Comment at: openmp/runtime/src/thirdparty/ittnotify/ittnotify.h:783
+                                                                     *tools
+                                                                     *********************************************************************/
 /** @{ */
----------------
same.


================
Comment at: openmp/runtime/src/thirdparty/ittnotify/ittnotify.h:876
+                                                                  *tools
+                                                                  ******************************************************************/
 /** @{ */
----------------
same.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97088



More information about the Openmp-commits mailing list