[PATCH] D137181: [clang-format] Don't use 'PPIndentWidth' inside multi-line macros
Owen Pan via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 8 22:55:21 PST 2022
owenpan added a comment.
In D137181#3916521 <https://reviews.llvm.org/D137181#3916521>, @goldstein.w.n wrote:
> I think we can remove the places I set `InitialPPLevel` and `OldPPLevel` but anything else I remove causes at least one test to fail.
>
> What did you do? Just set `PPLevel = PPBranchLevel`?
Yes, if there is a header guard. Otherwise, set `PPLevel` to `PPBranchLevel + 1`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137181/new/
https://reviews.llvm.org/D137181
More information about the cfe-commits
mailing list