[compiler-rt] [llvm] [TypeProf][InstrPGO] Introduce raw and instr profile format change for type profiling. (PR #81691)

via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 16 09:57:38 PST 2024


modiking wrote:

> > Also the CI clang-format is complaining so make sure the re-run that on your changes
> 
> clang-format formats existing code (specifically C++ macros) in `InstrProfData.inc` pretty aggressively and the indentations making code hard to read.
> 
> Taking https://github.com/llvm/llvm-project/actions/runs/7921397694/job/21626608711?pr=81691 as an example, reformatted code between `-/+` line 210 to line 267 are hard to read, same for code between line 284 to line 404, code between 780 to 970, etc
> 
> Given clang-format only complains about two `InstrProfData.inc`, I'm inclined not to run `git clang-format` to update these two files.

If that's the case we could ignore these files from `clang-format` via a `.clang-format-ignore` file if you're inclined to make that change.

https://github.com/llvm/llvm-project/pull/81691


More information about the llvm-commits mailing list