[PATCH] D124956: [clang-format] Fix another bug in AlignConsecutiveAssignments
Owen Pan via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 4 17:24:24 PDT 2022
owenpan added a comment.
No, because the first part of that test case would cause non-consecutive assignments to be aligned. (See https://github.com/llvm/llvm-project/issues/55265.) Only then would the `if` block be treated as a braced list and get misindented. We should definitely use test cases with PP directives when fixing #55265 though.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124956/new/
https://reviews.llvm.org/D124956
More information about the cfe-commits
mailing list