diff options
author | Galen Elias <galenelias@gmail.com> | 2023-08-31 14:13:06 -0700 |
---|---|---|
committer | Owen Pan <owenpiano@gmail.com> | 2023-08-31 14:21:19 -0700 |
commit | 58c67e724f718be06ea43c766871135a338c2d55 (patch) | |
tree | b0ef74d333362c965f5247a816a6edc5522d8d92 /llvm/lib/IR/Function.cpp | |
parent | c9ecaf32f6aa92f79a054478df55b4b8c3e53697 (diff) | |
download | llvm-58c67e724f718be06ea43c766871135a338c2d55.zip llvm-58c67e724f718be06ea43c766871135a338c2d55.tar.gz llvm-58c67e724f718be06ea43c766871135a338c2d55.tar.bz2 |
[clang-format] Fix AlignArrayOfStructures + Cpp11BracedListStyle=false
Currently AlignArrayOfStructures=Left is hard coding setting Spaces to
0 for the token following the initial opening brace, but not touching
Spaces for the subsequent lines, which leads to the array being
misaligned. Additionally, it's not adding a space before the trailing
} which is generally done when Cpp11BracedListStyle=false.
I'm not exactly sure why this function needs to override the Spaces as
it seems to generally already be set to either 0 or 1 according to
the other formatting settings, but I'm going with an explicit fix where
I just force the padding to 1 when Cpp11BracedListStyle=false.
AlignArrayOfStructures=Right doesn't have any alignment problems, but
isn't adding the expected padding around the braces either, so I'm
giving that the same treatment.
Fixes #57611.
Differential Revision: https://reviews.llvm.org/D158795
Diffstat (limited to 'llvm/lib/IR/Function.cpp')
0 files changed, 0 insertions, 0 deletions