Same script as D95318. Test files are excluded.
Details
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 |
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. |
clang-tidy: error: 'common/allocator.h' file not found [clang-diagnostic-error]
not useful