@@ -0,0 +1,67 @@
+//===--- UseCppStyleCommentsCheck.cpp -
clang-tidy-===//
+
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
HerrCai0907 wrote:
Normally PR can be merged if someone approved PR. You can merge it by yourself.
If you don't have access, I'm glad to help you.
https://github.com/llvm/llvm-project/pull/93623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/HerrCai0907 approved this pull request.
re-think about this case. LGTM
https://github.com/llvm/llvm-project/pull/97969
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/HerrCai0907 requested changes to this pull request.
There are some other limitation for const variable to be internal linkage.
Please follow standard to exclude this case
https://eel.is/c++draft/basic.link#3.2.
https://github.com/llvm/llvm-project/pull/97969
HerrCai0907 wrote:
> What happens if you compile a file that really vectorizes reference types?
> The attached test doesn't seem to actually end up getting vectorized after
> all. Can you add a test that get vectorized?
Is it possible to vectorize external reference types? I don't think there
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/97080
>From b49737505ad3ad84e977787daf141b98fe3e6607 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 29 Jun 2024 00:24:30 +0800
Subject: [PATCH 1/2] [WebAssembly] support getVT from wasm externref and
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/97080
>From b49737505ad3ad84e977787daf141b98fe3e6607 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 29 Jun 2024 00:24:30 +0800
Subject: [PATCH 1/2] [WebAssembly] support getVT from wasm externref and
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/97080
>From b49737505ad3ad84e977787daf141b98fe3e6607 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 29 Jun 2024 00:24:30 +0800
Subject: [PATCH] [WebAssembly] support getVT from wasm externref and funcref
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/97080
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/97080
Fixes: #69894
>From b49737505ad3ad84e977787daf141b98fe3e6607 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 29 Jun 2024 00:24:30 +0800
Subject: [PATCH] [WebAssembly] support getVT from wasm
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/96917
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul
=��___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/96917
Fixes: #93157
>From 079e341092da1ce8b893887dc6f58a9986e5b6bd Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 27 Jun 2024 23:05:00 +0800
Subject: [PATCH] [clang-tidy] improve messages when auto-fix
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/96122
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/96779
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -25,3 +25,16 @@ Example:
}
// already declared as extern
extern int v2;
+
+Options
+---
+
+.. option:: FixMode
HerrCai0907 wrote:
#96779
https://github.com/llvm/llvm-project/pull/96203
___
cfe-commits
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/96203
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 approved this pull request.
LGTM. Thanks for your contribution.
https://github.com/llvm/llvm-project/pull/96617
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96203
>From 669205c0f659239c58a3bde3ddadabb0a8ecbad8 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 15:05:57 +
Subject: [PATCH 1/4] [clang-tidy] add fixhint for misc-use-internal-linkage
@@ -25,3 +25,16 @@ Example:
}
// already declared as extern
extern int v2;
+
+Options
+---
+
+.. option:: FixMode
+
+ Selects the fix mode when fixing automatically.
HerrCai0907 wrote:
```suggestion
Selects what kind of a fix the check should
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/96199
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/96506
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 approved this pull request.
lgtm
https://github.com/llvm/llvm-project/pull/96506
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96203
>From 669205c0f659239c58a3bde3ddadabb0a8ecbad8 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 15:05:57 +
Subject: [PATCH 1/3] [clang-tidy] add fixhint for misc-use-internal-linkage
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96199
>From 03bc40a343227cb839ec908492c0d904be9af654 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 14:36:51 +
Subject: [PATCH 1/3] [clang-tidy] align all help message in run-clang-tidy
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96203
>From 669205c0f659239c58a3bde3ddadabb0a8ecbad8 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 15:05:57 +
Subject: [PATCH 1/2] [clang-tidy] add fixhint for misc-use-internal-linkage
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96122
>From 41993ea6903668c41eef8a4477f5914c894f7109 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 19 Jun 2024 23:20:09 +
Subject: [PATCH 1/8] [clang-tidy] add option to avoid "no checks enabled"
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/96122
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HerrCai0907 wrote:
I found it. It named `clang-tidy-diff.py`
https://github.com/llvm/llvm-project/pull/96122
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
HerrCai0907 wrote:
Where is `diff-clang-tidy.py`?
https://github.com/llvm/llvm-project/pull/96122
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96122
>From 41993ea6903668c41eef8a4477f5914c894f7109 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 19 Jun 2024 23:20:09 +
Subject: [PATCH 1/6] [clang-tidy] add option to avoid "no checks enabled"
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96122
>From 41993ea6903668c41eef8a4477f5914c894f7109 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 19 Jun 2024 23:20:09 +
Subject: [PATCH 1/5] [clang-tidy] add option to avoid "no checks enabled"
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
@@ -342,31 +342,33 @@ void SizeofExpressionCheck::check(const
MatchFinder::MatchResult ) {
if (DenominatorSize > CharUnits::Zero() &&
!NumeratorSize.isMultipleOf(DenominatorSize)) {
-
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96199
>From 03bc40a343227cb839ec908492c0d904be9af654 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 14:36:51 +
Subject: [PATCH 1/2] [clang-tidy] align all help message in run-clang-tidy
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/96203
None
>From 669205c0f659239c58a3bde3ddadabb0a8ecbad8 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 15:05:57 +
Subject: [PATCH] [clang-tidy] add fixhint for misc-use-internal-linkage
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96199
>From 03bc40a343227cb839ec908492c0d904be9af654 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 14:36:51 +
Subject: [PATCH] [clang-tidy] align all help message in run-clang-tidy [NFC]
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/96199
None
>From 36194358315d1614cb6d802d30ffc7c651c9ff44 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 20 Jun 2024 14:36:51 +
Subject: [PATCH] [clang-tidy] align all help message in run-clang-tidy
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96122
>From 41993ea6903668c41eef8a4477f5914c894f7109 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 19 Jun 2024 23:20:09 +
Subject: [PATCH 1/4] [clang-tidy] add option to avoid "no checks enabled"
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96122
>From 41993ea6903668c41eef8a4477f5914c894f7109 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 19 Jun 2024 23:20:09 +
Subject: [PATCH 1/3] [clang-tidy] add option to avoid "no checks enabled"
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/96122
>From 41993ea6903668c41eef8a4477f5914c894f7109 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 19 Jun 2024 23:20:09 +
Subject: [PATCH 1/2] [clang-tidy] add option to avoid "no checks enabled"
HerrCai0907 wrote:
> if a reason for this parameter is not to fail wrapper scripts, then maybe
> run-clang-tidy.py / run-clang-tidy-diff.py should also be updated.
Both of them is fine because they only count the failed file and don't do any
check about failure.
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/96122
When clang-tidy get an empty checks, it will throw "no checks enabled" error
and exit with non-zero return value.
It make clang-tidy's wrapper program confused when in big project some files
don't want to
@@ -39,20 +40,24 @@ Hints declHints(const Decl *D) {
}
std::vector> locateDecl(const Decl ) {
- std::vector> Result;
- // FIXME: Should we also provide physical locations?
- if (auto SS = tooling::stdlib::Recognizer()()) {
-Result.push_back({*SS,
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/95797
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -39,20 +40,24 @@ Hints declHints(const Decl *D) {
}
std::vector> locateDecl(const Decl ) {
- std::vector> Result;
- // FIXME: Should we also provide physical locations?
- if (auto SS = tooling::stdlib::Recognizer()()) {
-Result.push_back({*SS,
HerrCai0907 wrote:
> Excellent quality. Thank for fixing this.
Should I fix the other type comparison in static analysis? It looks like not
consider this case at lots of position.
https://github.com/llvm/llvm-project/pull/95729
___
cfe-commits
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/95729
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HerrCai0907 wrote:
> Do you need us to merge this, or you have merge rights?
Yes.
https://github.com/llvm/llvm-project/pull/95729
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/95797
When a definition is being located, standard library function recognition is
skipped if the function has a definition with body in the main file
>From 71a2c0d29058edac2539fbbe406890e6a6dc875c Mon Sep 17
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/95729
>From 40f18f2be624ed2a5b4922e67e4ed6070d2d2400 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Mon, 17 Jun 2024 00:13:20 +
Subject: [PATCH 1/2] [clang][analyzer] use unqualified canonical type during
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/95729
Fixes: #95658
Unqualified canonical type should be used instead of normal QualType for type
equality comparison
>From 40f18f2be624ed2a5b4922e67e4ed6070d2d2400 Mon Sep 17 00:00:00 2001
From: Congcong Cai
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/95667
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/95667
>From 5a4a4aac26a2a4078f07977b5101d3a1e22a3e0c Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 15 Jun 2024 16:24:55 +
Subject: [PATCH 1/2] [clang-tidy] fix false negatives for
@@ -387,3 +387,38 @@ void foo(const StructWithFieldContainer ) {
B.push_back(Number);
}
}
+
+namespace gh95596 {
+
+void f(std::vector& t) {
+ {
+std::vector gh95596_0;
+// CHECK-FIXES: gh95596_0.reserve(10);
+for (unsigned i = 0; i < 10; ++i)
+
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/95667
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/95667
Fixes: #95596
Check will warn if the loop var type is not same as var init expr type
>From 5a4a4aac26a2a4078f07977b5101d3a1e22a3e0c Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 15 Jun 2024
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/93827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/95633
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/95633
>From 0022050f7b537914dff7adf937103def4c9c939a Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 15 Jun 2024 05:40:50 +
Subject: [PATCH 1/2] [clang-tidy]fix false positives of the result of
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/95633
Fixes: #86404
>From 0022050f7b537914dff7adf937103def4c9c939a Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Sat, 15 Jun 2024 05:40:50 +
Subject: [PATCH] [clang-tidy]fix false positives of the result
https://github.com/HerrCai0907 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/95550
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/95434
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/95434
>From ecbd726bb937361b243ea4433e8c597c8d30f857 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Fri, 14 Jun 2024 00:50:04 +0800
Subject: [PATCH 1/2] [clang-tidy] avoid false positive when overload for
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/95434
Fixes: #90274
>From ecbd726bb937361b243ea4433e8c597c8d30f857 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Fri, 14 Jun 2024 00:50:04 +0800
Subject: [PATCH] [clang-tidy] avoid false positive when
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/95384
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Congcong Cai
Date: 2024-06-13T18:24:12+08:00
New Revision: 3f9e2e179a52eb50a2bcff148a5f351a4eddcb37
URL:
https://github.com/llvm/llvm-project/commit/3f9e2e179a52eb50a2bcff148a5f351a4eddcb37
DIFF:
https://github.com/llvm/llvm-project/commit/3f9e2e179a52eb50a2bcff148a5f351a4eddcb37.diff
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/95384
- **[MLIR] Translate DIStringType. (#94480)**
- **[clangd] Fix crash with null check for Token at Loc (#94528)**
- **[flang][Transforms][NFC] Remove boilerplate from vscale range pass
(#94598)**
-
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/95265
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/95265
`#` and `##` preprocessing tokens cannot be replaced by constexpr function. It
should be ignored in check.
>From eaeb5b6493e0c3980a60f9acbc5572de0d619419 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date:
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?=
Message-ID:
In-Reply-To:
HerrCai0907 wrote:
> >
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?=
Message-ID:
In-Reply-To:
HerrCai0907 wrote:
> I
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/94923
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HerrCai0907 wrote:
> To be clear and reiterate my previous comment: this check should NOT require
> users to use at(). That behavior should be opt-in. It should only warn about
> using operator[]. It's up to the users to figure out what the best
> replacement is.
I don't think it is a good
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/94923
>From e467b03cc120eedc580c185232f000e0d8aa0cc7 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Mon, 10 Jun 2024 09:04:27 +0800
Subject: [PATCH 1/2] [clang-tidy] fix false positives for the functions with
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/94923
>From e467b03cc120eedc580c185232f000e0d8aa0cc7 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Mon, 10 Jun 2024 09:04:27 +0800
Subject: [PATCH 1/2] [clang-tidy] fix false positives for the functions with
@@ -15,3 +15,5 @@ std::string HelloString;
// CHECK-MESSAGES: :[[@LINE-1]]:6: warning: no header providing "std::string"
is directly included [misc-include-cleaner]
int FooBarResult = foobar();
// CHECK-MESSAGES: :[[@LINE-1]]:20: warning: no header providing "foobar" is
https://github.com/HerrCai0907 requested changes to this pull request.
`at` and `operator[]` in `std::map` have different meanings. `operator[]` will
insert value if key does not exist but `at` won't. Please limit this check only
for index containers instead of key-value containers.
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/94923
>From e467b03cc120eedc580c185232f000e0d8aa0cc7 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Mon, 10 Jun 2024 09:04:27 +0800
Subject: [PATCH] [clang-tidy] fix false positives for the functions with the
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/94923
Fixes: #93335
For decl with body, we should provide physical locations also. Because it may
be the function which have the same name as std library.
>From e467b03cc120eedc580c185232f000e0d8aa0cc7 Mon Sep 17
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/90830
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/94697
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/94697
1. merge valid check
2. use range base loop
>From 8104085685772f93bcc595858821f839636ada9e Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Fri, 7 Jun 2024 07:06:37 +0800
Subject: [PATCH] [clang-tidy]
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/94636
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/94636
Fixes: #94634
>From 28ef0936ee115720bbe6cc20cf255100d63780cc Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Fri, 7 Jun 2024 00:11:55 +0800
Subject: [PATCH] [clang-tidy]fix crashing when self include
@@ -24,12 +24,28 @@ namespace {
AST_MATCHER(Decl, isFirstDecl) { return Node.isFirstDecl(); }
-AST_MATCHER_P(Decl, isInMainFile, FileExtensionsSet, HeaderFileExtensions) {
+static bool isInMainFile(SourceLocation L, SourceManager ,
+ const
@@ -24,12 +24,28 @@ namespace {
AST_MATCHER(Decl, isFirstDecl) { return Node.isFirstDecl(); }
-AST_MATCHER_P(Decl, isInMainFile, FileExtensionsSet, HeaderFileExtensions) {
+static bool isInMainFile(SourceLocation L, SourceManager ,
+ const
@@ -0,0 +1,78 @@
+//===--- UseInternalLinkageCheck.cpp -
clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/94527
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,78 @@
+//===--- UseInternalLinkageCheck.cpp -
clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
@@ -0,0 +1,78 @@
+//===--- UseInternalLinkageCheck.cpp -
clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/90830
>From 24cbbd0c87ab2a06381d210da1dff5f966b72773 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 2 May 2024 15:44:45 +0800
Subject: [PATCH 1/7] reformat
---
.../clang-tidy/readability/CMakeLists.txt
@@ -0,0 +1,78 @@
+//===--- UseInternalLinkageCheck.cpp -
clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
@@ -306,6 +306,10 @@ Changes in existing checks
don't remove parentheses used in ``sizeof`` calls when they have array index
accesses as arguments.
+- Improved :doc:`modernize-use-constraints
+ ` check by fixing a crash that
+ occurred in some scenarios and excluded
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/92019
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/92025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,78 @@
+//===--- UseInternalLinkageCheck.cpp -
clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
@@ -0,0 +1,78 @@
+//===--- UseInternalLinkageCheck.cpp -
clang-tidy--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/90830
>From 24cbbd0c87ab2a06381d210da1dff5f966b72773 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Thu, 2 May 2024 15:44:45 +0800
Subject: [PATCH 1/7] reformat
---
.../clang-tidy/readability/CMakeLists.txt
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/91293
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 566 matches
Mail list logo