[PATCH] D118879: [clang-format] Avoid merging macro definitions.

2022-02-03 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius added a comment. Thanks for the reviews! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118879/new/ https://reviews.llvm.org/D118879 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D118879: [clang-format] Avoid merging macro definitions.

2022-02-03 Thread Marek Kurdej via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. curdeius marked 2 inline comments as done. Closed by commit rGca0d97072e79: [clang-format] Avoid merging macro definitions. (authored by curdeius). Changed prior to co

[PATCH] D118879: [clang-format] Avoid merging macro definitions.

2022-02-03 Thread Owen Pan via Phabricator via cfe-commits
owenpan accepted this revision. owenpan added inline comments. Comment at: clang/unittests/Format/FormatTest.cpp:1812 + Style.BraceWrapping.AfterFunction = true; + // Test that a macro definition gets never merged with the following + // definition. ===

[PATCH] D118879: [clang-format] Avoid merging macro definitions.

2022-02-03 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius updated this revision to Diff 405539. curdeius added a comment. Force CI after changing parent revision. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118879/new/ https://reviews.llvm.org/D118879 Files: clang/lib/Format/UnwrappedLineFor

[PATCH] D118879: [clang-format] Avoid merging macro definitions.

2022-02-02 Thread Marek Kurdej via Phabricator via cfe-commits
curdeius created this revision. curdeius added reviewers: MyDeveloperDay, HazardyKnusperkeks, owenpan. curdeius requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes https://github.com/llvm/llvm-project/issues/42087. Repository: rG LLVM