[PATCH] D99840: [clang-format] Correctly attach enum braces with ShortEnums disabled

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 21 03:25:00 PDT 2021


MyDeveloperDay added a comment.

> I don't think we should change the LLVM style

I'm not 100% convinced we are changing the LLVM style per say.

LLVM always "cuddles" the "{"  and Allow short Enums on a single line is true, this means if the "{" is on a new line then I think its actually a bug. @curdeius  what say you?

It be interested to know the impact on the LLVM tree.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99840



More information about the cfe-commits mailing list