This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] Add more tests for Objective-C 2.0 generic alignment
ClosedPublic

Authored by benhamilton on Feb 2 2018, 12:32 PM.

Details

Summary

In r236412, @djasper added a comment:

// FIXME: We likely want to do this for more combinations of brackets.
// Verify that it is wanted for ObjC, too.

In D42650, @stephanemoore asked me to confirm this.

This followup to D42650 adds more tests to verify the relative
alignment behavior for Objective-C 2.0 generics passed to functions
and removes the second half of the FIXME comment.

Test Plan:

make -j12 FormatTests && \
./tools/clang/unittests/Format/FormatTests --gtest_filter=FormatTestObjC.\*

Diff Detail

Repository
rC Clang

Event Timeline

benhamilton created this revision.Feb 2 2018, 12:32 PM
benhamilton edited the summary of this revision. (Show Details)
benhamilton edited the summary of this revision. (Show Details)
jolesiak accepted this revision.Feb 6 2018, 8:09 AM
This revision is now accepted and ready to land.Feb 6 2018, 8:09 AM
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.

Looks reasonable 👌