[PATCH] D137181: [clang-format] Don't use 'PPIndentWidth' inside multi-line macros
Noah Goldstein via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 8 21:55:47 PST 2022
goldstein.w.n added a comment.
In D137181#3916474 <https://reviews.llvm.org/D137181#3916474>, @owenpan wrote:
> You can still simply the changes to UnwrappedLineParser.cpp a lot. In fact, instead of adjusting `PPLevel` at various places, you only need to set it once when `InMacroBody` is set.
I don't think thats right, it needs to stay updated with the true macro depth (even though we only use `PPLevel` for formatting `InMacroBody` lines).
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