[PATCH] D80115: [clang-format] [PR45816] Add AlignConsecutiveBitFields

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 18 08:01:51 PDT 2020


MyDeveloperDay planned changes to this revision.
MyDeveloperDay marked an inline comment as done.
MyDeveloperDay added inline comments.


================
Comment at: clang/lib/Format/WhitespaceManager.cpp:588-590
+        return C.Tok->Previous->startsSequence(tok::identifier, tok::colon,
+                                               tok::numeric_constant);
+      },
----------------
JakeMerdichAMD wrote:
> TT_BitFieldColon should match this criteria exactly. 
I think using TT_BitFieldColon is a better idea


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80115/new/

https://reviews.llvm.org/D80115





More information about the cfe-commits mailing list