[compiler-rt] [llvm] [nfc]For InstrProfData.inc, clang-format functions and opt-out of formatting on the rest (PR #82057)
Mingming Liu via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 20 16:43:31 PST 2024
================
@@ -62,6 +62,7 @@
#define INSTR_PROF_VISIBILITY
#endif
+// clang-format off:re-enable clang-format after `issue #82426` is fixed
----------------
minglotus-6 wrote:
The main motivation is to provide a context why clang-format is turned off.
If clang-format could format macros in a readable way, re-enabling it gives automated code formats.
Meanwhile, clang-format 18 introduces an option `SkipMacroDefinitionBody` (https://clang.llvm.org/docs/ClangFormatStyleOptions.html#skipmacrodefinitionbody) which seems relevant. However pre-commit CI currently uses clang format version 17, and afaik would emit hard error if `.clang-format` contains unparsable fields.
https://github.com/llvm/llvm-project/pull/82057
More information about the llvm-commits
mailing list