[PATCH] D129443: [clang-format] Add option for aligning requires clause body

2022-07-14 Thread Danil Sidoruk via Phabricator via cfe-commits
eoanermine updated this revision to Diff 444556. eoanermine edited the summary of this revision. eoanermine added a comment. - Rename AlignRequiresClauseBody to RequiresExpressionIndentation - Introduce RequiresExpressionIndentationKind as a type of RequiresExpressionIndentation - Add more tests

[clang] 52cb972 - [CommandLine] --help: print "-o " instead of "-o="

2022-07-14 Thread Fangrui Song via cfe-commits
Author: Fangrui Song Date: 2022-07-14T01:28:28-07:00 New Revision: 52cb972537dac7ac535b57ba9399d53549c027e8 URL: https://github.com/llvm/llvm-project/commit/52cb972537dac7ac535b57ba9399d53549c027e8 DIFF: https://github.com/llvm/llvm-project/commit/52cb972537dac7ac535b57ba9399d53549c027e8.diff

[clang-tools-extra] 52cb972 - [CommandLine] --help: print "-o " instead of "-o="

2022-07-14 Thread Fangrui Song via cfe-commits
Author: Fangrui Song Date: 2022-07-14T01:28:28-07:00 New Revision: 52cb972537dac7ac535b57ba9399d53549c027e8 URL: https://github.com/llvm/llvm-project/commit/52cb972537dac7ac535b57ba9399d53549c027e8 DIFF: https://github.com/llvm/llvm-project/commit/52cb972537dac7ac535b57ba9399d53549c027e8.diff

[PATCH] D129642: [Sema] Tweak diagnostic logic so suppress-in-hedaer logic works in tools too.

2022-07-14 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet added inline comments. Comment at: clang/lib/Sema/SemaDecl.cpp:1784 if (D->isInvalidDecl() || D->isUsed() || D->hasAttr()) return false; i think we can just bail out early here when the main file is a header file (i.e. lang opts have

[PATCH] D129742: [clang-format] Fix invalid-code-generation by RemoveBracesLLVM

2022-07-14 Thread Owen Pan via Phabricator via cfe-commits
owenpan created this revision. owenpan added reviewers: HazardyKnusperkeks, curdeius, MyDeveloperDay. owenpan added a project: clang-format. Herald added a project: All. owenpan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. When removing

[PATCH] D124447: [clang-tidy] Add infrastructure support for running on project-level information

2022-07-14 Thread Whisperity via Phabricator via cfe-commits
whisperity added a comment. In D124447#3645197 , @sammccall wrote: > In D124447#3608747 , @aaron.ballman > wrote: > >> In general, I think this is looking pretty close to good. Whether clang-tidy >> should get

<    1   2   3