[PATCH] D43303: [Format] Fix for bug 35641
Daniel Jasper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 15 04:32:50 PST 2018
djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Thanks for the fix.
================
Comment at: unittests/Format/FormatTest.cpp:9453
+
+ // Bug 35641
+ Alignment.AlignConsecutiveDeclarations = true;
----------------
Make this "See llvm.org/PR35641".
================
Comment at: unittests/Format/FormatTest.cpp:9457
+ " int b;\n"
+ " int c;\n"
+ "}",
----------------
Might be useful to use different types here to verify that alignment is actually happening, e.g. "int" and "unsigned".
Repository:
rC Clang
https://reviews.llvm.org/D43303
More information about the cfe-commits
mailing list