bcm5719-llvm/clang/unittests, branch meklort-10.0.1

[Syntax] Merge overlapping top-level macros in TokenBuffer

2020-06-10T12:21:44+00:00

Summary: Our previous definition of "top-level" was too informal, and didn't allow for overlapping macros that each directly produce expanded tokens. See D77507 for previous discussion. Fixes http://bugs.llvm.org/show_bug.cgi?id=45428 Reviewers: kadircet, vabridgers Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D77615 (cherry picked from commit d66afd6dde542dc373f87e07fe764c071fe20d76)

[clang-format] [PR45357] Fix issue found with operator spacing

2020-05-07T18:06:57+00:00

Summary: This is a tentative fix for https://bugs.llvm.org/show_bug.cgi?id=45357 Spaces seem to be introduced between * and * due to changes brought in for {D69573} Reviewers: sylvestre.ledru, mitchell-stellar, sammccall, Abpostelnicu, krasimir, jbcoe Reviewed By: Abpostelnicu Subscribers: tstellar, hans, Abpostelnicu, cfe-commits Tags: #clang, #clang-format Differential Revision: https://reviews.llvm.org/D78879 (cherry picked from commit b01dca50085768f1f1a5ad21a685906d48c38816)

clang-format: Fix pointer alignment for overloaded operators (PR45107)

2020-05-07T18:06:57+00:00

This fixes a regression from D69573 which broke the following example: $ echo 'operator C*();' | bin/clang-format --style=Chromium operator C *(); (There should be no space before the asterisk.) It seems the problem is in TokenAnnotator::spaceRequiredBetween(), which only looked at the token to the left of the * to see if it was a type or not. That code only handled simple types or identifiers, not templates or qualified types. This patch addresses that. Differential revision: https://reviews.llvm.org/D76850 (cherry picked from commit eb85e90350e93a64279139e7eca9ca40c8fbf5eb)

clang-format: fix spacing in `operator const char*()`

2020-03-02T10:19:33+00:00

Summary: Revision a75f8d98d7ac9e557b238a229a9a2647c71feed1 fixed spacing for operators, but caused the const and non-const versions to diverge: ``` // With Style.PointerAlignment = FormatStyle::PAS_Left: struct A { operator char*() { return ""; } operator const char *() const { return ""; } }; ``` The code was checking if the type specifier was directly preceded by `operator`. However there could be comments and `const/volatile` in between. Reviewers: mprobst Reviewed By: mprobst Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D72911 (cherry picked from commit 33463cfba2be7c8d6c08e666123cc34f114a1f3e)

[ASTMatchers] HasNameMatcher handles `extern "C"`

2020-02-27T12:06:12+00:00

Summary: Fixes [[ https://bugs.llvm.org/show_bug.cgi?id=42193 | hasName AST matcher is confused by extern "C" in namespace. ]] Reviewers: klimek, aaron.ballman, gribozavr2 Reviewed By: aaron.ballman Subscribers: cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D75202 (cherry picked from commit 16cabf278fc8c14d415e677ce0bc40d46a6de30d)

[clang-format] Expand the SpacesAroundConditions option to include catch statements

2020-01-21T17:53:05+00:00

Summary: This diff expands the SpacesAroundConditions option added in D68346 to include adding spaces to catch statements. Reviewed By: MyDeveloperDay Patch by: timwoj Differential Revision: https://reviews.llvm.org/D72793 (cherry picked from commit ea2be452542c81b04621e26c0d5e83be565f07e2)

Removed PointerUnion3 and PointerUnion4 aliases in favor of the variadic template

2020-01-14T17:56:29+00:00

[Syntax] Unset IsOriginal flag on nodes removed from the tree

2020-01-14T16:00:33+00:00

And add a corresponding test. Only nodes inside the TranslationUnit subtree can be marked as original, computeReplacements() relies on this.

[Syntax] Mark synthesized nodes as modifiable

2020-01-14T15:41:09+00:00

This was an oversight in the original patch. Also add corresponding tests.

[clang] Remove raw string literals in macros

2020-01-13T12:38:58+00:00

Older (but still supported) versions of GCC don't handle C++11 raw string literals in macro parameters correctly.