diff options
author | Kanglei Fang <kanglei@uber.com> | 2020-04-06 12:45:38 -0700 |
---|---|---|
committer | Jin Lin <jinl@uber.com> | 2020-04-06 14:20:09 -0700 |
commit | 28ecd7f0b0865878a12d814a6d0de92cb7529d22 (patch) | |
tree | 0ba6209bfd5ff128b18ea4c53675243314827f38 /llvm/lib/CodeGen/MachineModuleInfo.cpp | |
parent | 836534f9970295ff559ef49d6c49958aac6171f9 (diff) | |
download | llvm-28ecd7f0b0865878a12d814a6d0de92cb7529d22.zip llvm-28ecd7f0b0865878a12d814a6d0de92cb7529d22.tar.gz llvm-28ecd7f0b0865878a12d814a6d0de92cb7529d22.tar.bz2 |
[clang-format] Don't break multi block parameters on ObjCBreakBeforeNestedBlockParam
Summary:
While [the original diff](https://reviews.llvm.org/D42493) makes a lot of sense, and multiple inline block parameter/trailing paramemter after inline block paramemter should be discouraged, the formatting result is different than what xcode does by default
For the exact same example provided in the original diff:
```
[object
blockArgument:^{
a = 42;
}
anotherArg:42];
```
The code is hard to read and not very visually pleasing
This diff uses `ObjCBreakBeforeNestedBlockParam` to shield from the formatting
When it's set to false, don't allign the inline block paramemters.
Reviewers: jolesiak, benhamilton, jinlin
Reviewed By: jolesiak
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D77039
Diffstat (limited to 'llvm/lib/CodeGen/MachineModuleInfo.cpp')
0 files changed, 0 insertions, 0 deletions