diff options
| author | mydeveloperday <mydeveloperday@gmail.com> | 2021-06-26 13:34:07 +0100 |
|---|---|---|
| committer | mydeveloperday <mydeveloperday@gmail.com> | 2021-06-26 13:34:07 +0100 |
| commit | 37c2233097ac44697b87228d86eef1fce10ea5c1 (patch) | |
| tree | dcf3b412eca89dec0291116a1158663c3c879ba4 /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | ee3b2c47ce41aeabede85d96e43bee33be73aa2f (diff) | |
[clang-format] [PR50702] Lamdba processing does not respect AfterClass and AfterNamespace
https://bugs.llvm.org/show_bug.cgi?id=50702
I believe {D44609} may be too aggressive with brace wrapping rules which doesn't always apply to Lamdbas
The introduction of BeforeLambdaBody and AllowShortLambdasOnASingleLine has impact on brace handling on other block types, which I suspect we didn't see before as people may not be using the BeforeLambdaBody style
From what I can tell this can be seen by the unit test I change as its not honouring the orginal LLVM brace wrapping style for the `Fct()` function
I added a unit test from PR50702 and have removed some of the code (which has zero impact on the unit test, which kind of suggests its unnecessary), some additional attempt has been made to try and ensure we'll only break on what is actually a LamdbaLBrace
Reviewed By: HazardyKnusperkeks
Differential Revision: https://reviews.llvm.org/D104222
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
0 files changed, 0 insertions, 0 deletions
