[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