Same script as D95318. Test files are excluded.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
| openmp/libomptarget/deviceRTLs/interface.h | ||
|---|---|---|
| 182–183 | The latest version of clang-format can surprisingly handle this type of comments so well! | |
| openmp/runtime/doc/doxygen/libomp_interface.h | ||
| 4 | This file should be filtered out. | |
| openmp/runtime/src/thirdparty/ittnotify/disable_warnings.h | ||
| 15 | The tailing \ is unnecessary. Probably because clang-format takes it part of the pragma. | |
| openmp/runtime/src/thirdparty/ittnotify/ittnotify_static.cpp | ||
| 1049–1052 | Here as well | |
Comment Actions
I scrolled through all of it. 3 nits. LGTM otherwise, wait for @AndreyChurbanov though.
| openmp/runtime/src/thirdparty/ittnotify/ittnotify.h | ||
|---|---|---|
| 483 | This needs to be fixed. Change the comment style or sth | |
| 780 | same. | |
| 868 | same. | |
The latest version of clang-format can surprisingly handle this type of comments so well!