diff options
| author | Ben Hamilton <benhamilton@google.com> | 2018-03-30 15:38:45 +0000 |
|---|---|---|
| committer | Ben Hamilton <benhamilton@google.com> | 2018-03-30 15:38:45 +0000 |
| commit | 4dc1cdc5e1c1184f5ded058d5240532a54c96bc1 (patch) | |
| tree | e67765317c1e1194ae54c02868fd85bb2c5c8bf9 /llvm/lib/Transforms/Scalar/Float2Int.cpp | |
| parent | 4f99836a9e7228827bde6f205ec487cdd022c93f (diff) | |
| download | bcm5719-llvm-4dc1cdc5e1c1184f5ded058d5240532a54c96bc1.tar.gz bcm5719-llvm-4dc1cdc5e1c1184f5ded058d5240532a54c96bc1.zip | |
[clang-format] Ensure wrapped ObjC selectors with 1 arg obey IndentWrappedFunctionNames
Summary:
In D43121, @Typz introduced logic to avoid indenting 2-or-more
argument ObjC selectors too far to the right if the first component
of the selector was longer than the others.
This had a small side effect of causing wrapped ObjC selectors with
exactly 1 argument to not obey IndentWrappedFunctionNames:
```
- (aaaaaaaaaa)
aaaaaaaaaa;
```
This diff fixes the issue by ensuring we align wrapped 1-argument
ObjC selectors correctly:
```
- (aaaaaaaaaa)
aaaaaaaaaa;
```
Test Plan: New tests added. Test failed before change, passed
after change. Ran tests with:
% make -j12 FormatTests && ./tools/clang/unittests/Format/FormatTests
Reviewers: djasper, klimek, Typz, jolesiak
Reviewed By: djasper, jolesiak
Subscribers: cfe-commits, Typz
Differential Revision: https://reviews.llvm.org/D44994
llvm-svn: 328871
Diffstat (limited to 'llvm/lib/Transforms/Scalar/Float2Int.cpp')
0 files changed, 0 insertions, 0 deletions

