Author: Anton Dukeman
Date: 2023-07-24T19:42:19Z
New Revision: 2f0630f8bc91e81fd5948e82164cf82ae595caf2
URL:
https://github.com/llvm/llvm-project/commit/2f0630f8bc91e81fd5948e82164cf82ae595caf2
DIFF:
https://github.com/llvm/llvm-project/commit/2f0630f8bc91e81fd5948e82164cf82ae595caf2.diff
LOG:
Author: Piotr Zegar
Date: 2023-07-24T19:43:47Z
New Revision: 992fa7be3438bce61cfba7c0e0b07d9d1ddcc05b
URL:
https://github.com/llvm/llvm-project/commit/992fa7be3438bce61cfba7c0e0b07d9d1ddcc05b
DIFF:
https://github.com/llvm/llvm-project/commit/992fa7be3438bce61cfba7c0e0b07d9d1ddcc05b.diff
LOG: [
Author: Piotr Zegar
Date: 2023-07-25T17:50:19Z
New Revision: 3397dae69e09d33301a7620171cabc301d1f3abb
URL:
https://github.com/llvm/llvm-project/commit/3397dae69e09d33301a7620171cabc301d1f3abb
DIFF:
https://github.com/llvm/llvm-project/commit/3397dae69e09d33301a7620171cabc301d1f3abb.diff
LOG: [
Author: Piotr Zegar
Date: 2023-07-25T18:31:17Z
New Revision: b530eeea5e7697d58af2adac66c8b86c79e649bb
URL:
https://github.com/llvm/llvm-project/commit/b530eeea5e7697d58af2adac66c8b86c79e649bb
DIFF:
https://github.com/llvm/llvm-project/commit/b530eeea5e7697d58af2adac66c8b86c79e649bb.diff
LOG: [
Author: Piotr Zegar
Date: 2023-07-27T15:49:43Z
New Revision: 315946c57da2bfff0c26e4fec42e21dac7bfbddc
URL:
https://github.com/llvm/llvm-project/commit/315946c57da2bfff0c26e4fec42e21dac7bfbddc
DIFF:
https://github.com/llvm/llvm-project/commit/315946c57da2bfff0c26e4fec42e21dac7bfbddc.diff
LOG: [
Author: Piotr Zegar
Date: 2023-07-27T17:50:59Z
New Revision: 2cdb843767529b2be641b93e43d70837900345c1
URL:
https://github.com/llvm/llvm-project/commit/2cdb843767529b2be641b93e43d70837900345c1
DIFF:
https://github.com/llvm/llvm-project/commit/2cdb843767529b2be641b93e43d70837900345c1.diff
LOG: [
Author: Piotr Zegar
Date: 2023-07-27T19:27:25Z
New Revision: a8eb7415a418d878be8a7d6e48f8dff0c401dfc6
URL:
https://github.com/llvm/llvm-project/commit/a8eb7415a418d878be8a7d6e48f8dff0c401dfc6
DIFF:
https://github.com/llvm/llvm-project/commit/a8eb7415a418d878be8a7d6e48f8dff0c401dfc6.diff
LOG: R
Author: Piotr Zegar
Date: 2023-07-27T19:47:05Z
New Revision: 50eff374855a68cd7ccf3c62945afc1575516d88
URL:
https://github.com/llvm/llvm-project/commit/50eff374855a68cd7ccf3c62945afc1575516d88
DIFF:
https://github.com/llvm/llvm-project/commit/50eff374855a68cd7ccf3c62945afc1575516d88.diff
LOG: [
Author: Piotr Zegar
Date: 2023-07-27T20:07:52Z
New Revision: 3d9a64f7d4d6995dab8abb8910fe82e0047f6277
URL:
https://github.com/llvm/llvm-project/commit/3d9a64f7d4d6995dab8abb8910fe82e0047f6277
DIFF:
https://github.com/llvm/llvm-project/commit/3d9a64f7d4d6995dab8abb8910fe82e0047f6277.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-10T19:51:09Z
New Revision: 7f29f14d025702a04ca3e5cdd31de40c392e2799
URL:
https://github.com/llvm/llvm-project/commit/7f29f14d025702a04ca3e5cdd31de40c392e2799
DIFF:
https://github.com/llvm/llvm-project/commit/7f29f14d025702a04ca3e5cdd31de40c392e2799.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-10T19:51:09Z
New Revision: 894140b4fdde10b6bff7fc99104b2292ade3c91c
URL:
https://github.com/llvm/llvm-project/commit/894140b4fdde10b6bff7fc99104b2292ade3c91c
DIFF:
https://github.com/llvm/llvm-project/commit/894140b4fdde10b6bff7fc99104b2292ade3c91c.diff
LOG: [
Author: Chris Cotter
Date: 2023-08-11T07:13:59Z
New Revision: a563ced78bb4bda31babb85e88a89b49ffd61a90
URL:
https://github.com/llvm/llvm-project/commit/a563ced78bb4bda31babb85e88a89b49ffd61a90
DIFF:
https://github.com/llvm/llvm-project/commit/a563ced78bb4bda31babb85e88a89b49ffd61a90.diff
LOG:
Author: Piotr Zegar
Date: 2023-08-11T07:29:06Z
New Revision: 30fed17a08cda324b557498a6db18dd4a0281daa
URL:
https://github.com/llvm/llvm-project/commit/30fed17a08cda324b557498a6db18dd4a0281daa
DIFF:
https://github.com/llvm/llvm-project/commit/30fed17a08cda324b557498a6db18dd4a0281daa.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-11T10:36:45Z
New Revision: 5568f28c868eb13033d64189560ee0e62d745285
URL:
https://github.com/llvm/llvm-project/commit/5568f28c868eb13033d64189560ee0e62d745285
DIFF:
https://github.com/llvm/llvm-project/commit/5568f28c868eb13033d64189560ee0e62d745285.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-11T11:02:28Z
New Revision: cd9e92309d1733219f439d663ec9a4c3d59e6650
URL:
https://github.com/llvm/llvm-project/commit/cd9e92309d1733219f439d663ec9a4c3d59e6650
DIFF:
https://github.com/llvm/llvm-project/commit/cd9e92309d1733219f439d663ec9a4c3d59e6650.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-11T16:04:15Z
New Revision: a3a66de37fc038d46d7146c9dfe55f26fd3a
URL:
https://github.com/llvm/llvm-project/commit/a3a66de37fc038d46d7146c9dfe55f26fd3a
DIFF:
https://github.com/llvm/llvm-project/commit/a3a66de37fc038d46d7146c9dfe55f26fd3a.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-11T16:04:15Z
New Revision: efd44f80a5a8194b9fe26ff3244ce702cd8dab73
URL:
https://github.com/llvm/llvm-project/commit/efd44f80a5a8194b9fe26ff3244ce702cd8dab73
DIFF:
https://github.com/llvm/llvm-project/commit/efd44f80a5a8194b9fe26ff3244ce702cd8dab73.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-11T16:06:45Z
New Revision: 887bece66c80fe2bc3f9010f34f4fccf0a2f964e
URL:
https://github.com/llvm/llvm-project/commit/887bece66c80fe2bc3f9010f34f4fccf0a2f964e
DIFF:
https://github.com/llvm/llvm-project/commit/887bece66c80fe2bc3f9010f34f4fccf0a2f964e.diff
LOG: [
Author: Julian Schmidt
Date: 2023-08-12T16:18:33Z
New Revision: c3da99275a520b73235d975017502876e07e3e8e
URL:
https://github.com/llvm/llvm-project/commit/c3da99275a520b73235d975017502876e07e3e8e
DIFF:
https://github.com/llvm/llvm-project/commit/c3da99275a520b73235d975017502876e07e3e8e.diff
LOG
Author: Piotr Zegar
Date: 2023-08-15T06:59:11Z
New Revision: 766dd7b80b575e723c8cb454532b614f9d5f2bc7
URL:
https://github.com/llvm/llvm-project/commit/766dd7b80b575e723c8cb454532b614f9d5f2bc7
DIFF:
https://github.com/llvm/llvm-project/commit/766dd7b80b575e723c8cb454532b614f9d5f2bc7.diff
LOG: [
Author: Piotr Zegar
Date: 2023-08-15T06:59:11Z
New Revision: 0c7d28f72e190ca387a14cf76d611c4e8b292557
URL:
https://github.com/llvm/llvm-project/commit/0c7d28f72e190ca387a14cf76d611c4e8b292557
DIFF:
https://github.com/llvm/llvm-project/commit/0c7d28f72e190ca387a14cf76d611c4e8b292557.diff
LOG: [
Author: Felix
Date: 2023-08-16T15:02:56Z
New Revision: a94c44cc0a5601e977b5cc1d2cfdee1488be62e9
URL:
https://github.com/llvm/llvm-project/commit/a94c44cc0a5601e977b5cc1d2cfdee1488be62e9
DIFF:
https://github.com/llvm/llvm-project/commit/a94c44cc0a5601e977b5cc1d2cfdee1488be62e9.diff
LOG: [clang-
Author: Piotr Zegar
Date: 2023-08-21T15:45:17Z
New Revision: 845d3b5489fc1ae7205968acb74486a93c5ca902
URL:
https://github.com/llvm/llvm-project/commit/845d3b5489fc1ae7205968acb74486a93c5ca902
DIFF:
https://github.com/llvm/llvm-project/commit/845d3b5489fc1ae7205968acb74486a93c5ca902.diff
LOG: [
Author: Chris Cotter
Date: 2023-08-21T17:38:30Z
New Revision: a7bdaff7cad93e96ee9aaf3dc8b2a46c84717361
URL:
https://github.com/llvm/llvm-project/commit/a7bdaff7cad93e96ee9aaf3dc8b2a46c84717361
DIFF:
https://github.com/llvm/llvm-project/commit/a7bdaff7cad93e96ee9aaf3dc8b2a46c84717361.diff
LOG:
Author: Felix
Date: 2023-08-21T17:47:14Z
New Revision: 4001ae175cbe351d496a6cd5481a554b346f706d
URL:
https://github.com/llvm/llvm-project/commit/4001ae175cbe351d496a6cd5481a554b346f706d
DIFF:
https://github.com/llvm/llvm-project/commit/4001ae175cbe351d496a6cd5481a554b346f706d.diff
LOG: [clang-
@@ -284,6 +284,12 @@ Changes in existing checks
` check to properly escape
single quotes.
+- Improved :doc:`performance-noexcept-move-constructor
+ ` and
+ :doc:`performance-noexcept-swap`
PiotrZSL wrote:
duplicate this change note, here leave one for m
https://github.com/PiotrZSL approved this pull request.
+- LGTM, just split in release notes is needed.
https://github.com/llvm/llvm-project/pull/68359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/PiotrZSL approved this pull request.
https://github.com/llvm/llvm-project/pull/68359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -269,6 +269,12 @@ Changes in existing checks
` to support for-loops with
iterators initialized by free functions like ``begin``, ``end``, or ``size``.
+- Improved :doc:`modernize-return-braced-init-list
+ ` check to ignore
+ false-positives when constructing the conta
@@ -17,11 +17,21 @@ using namespace clang::ast_matchers;
namespace clang::tidy::modernize {
void ReturnBracedInitListCheck::registerMatchers(MatchFinder *Finder) {
- // Skip list initialization and constructors with an initializer list.
+ auto SemanticallyDifferentContainer
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -17,11 +17,21 @@ using namespace clang::ast_matchers;
namespace clang::tidy::modernize {
void ReturnBracedInitListCheck::registerMatchers(MatchFinder *Finder) {
- // Skip list initialization and constructors with an initializer list.
+ auto SemanticallyDifferentContainer
@@ -17,11 +17,21 @@ using namespace clang::ast_matchers;
namespace clang::tidy::modernize {
void ReturnBracedInitListCheck::registerMatchers(MatchFinder *Finder) {
- // Skip list initialization and constructors with an initializer list.
+ auto SemanticallyDifferentContainer
https://github.com/PiotrZSL approved this pull request.
Add entry to check documentation that such constructors for such types are
excluded.
Consider fixing pointed out issues.
Except those, looks fine.
https://github.com/llvm/llvm-project/pull/68491
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -17,11 +17,21 @@ using namespace clang::ast_matchers;
namespace clang::tidy::modernize {
void ReturnBracedInitListCheck::registerMatchers(MatchFinder *Finder) {
- // Skip list initialization and constructors with an initializer list.
+ auto SemanticallyDifferentContainer
@@ -17,11 +17,21 @@ using namespace clang::ast_matchers;
namespace clang::tidy::modernize {
void ReturnBracedInitListCheck::registerMatchers(MatchFinder *Finder) {
- // Skip list initialization and constructors with an initializer list.
+ auto SemanticallyDifferentContainer
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/68359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Felix
Date: 2023-10-09T19:58:44Z
New Revision: 7a73da4c85a12341752a4573c55ebff46ba20db0
URL:
https://github.com/llvm/llvm-project/commit/7a73da4c85a12341752a4573c55ebff46ba20db0
DIFF:
https://github.com/llvm/llvm-project/commit/7a73da4c85a12341752a4573c55ebff46ba20db0.diff
LOG: [clang-
https://github.com/PiotrZSL approved this pull request.
LGTM,
When release notes could be nice (clang-tools-extra/doc/ReleaseNotest.rst) with
something like: `Improved bugprone-unchecked-optional-acces check to not crash
during handling of optional values` or `to not crash i certain situations`
@@ -202,6 +202,10 @@ Changes in existing checks
` check, so that it does not
warn on macros starting with underscore and lowercase letter.
+- Improved :doc:`bugprone-unchecked-optional-access
+ ` check, use
`cast_or_null`
+ instead `cast` so that it does not crash durin
Author: Piotr Zegar
Date: 2023-10-10T18:37:35Z
New Revision: 135f57d179977136f920f8c975bb4b6ca056ec66
URL:
https://github.com/llvm/llvm-project/commit/135f57d179977136f920f8c975bb4b6ca056ec66
DIFF:
https://github.com/llvm/llvm-project/commit/135f57d179977136f920f8c975bb4b6ca056ec66.diff
LOG: [
@@ -0,0 +1,48 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
PiotrZSL wrote:
first paragraph in documentation should be sam
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68738
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
@@ -0,0 +1,131 @@
+// RUN: %check_clang_tidy -std=c++20 %s misc-coroutine-hostile-raii %t \
+// RUN: -config="{CheckOptions: \
+// RUN: {misc-coroutine-hostile-raii.RAIIDenyList: \
+// RUN: 'my::Mutex; ::my::other::Mutex'}}"
+
+namespace std {
+
+templa
@@ -0,0 +1,48 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
+suspension point in a coroutine.
+
+Some objects require that they be destroyed
@@ -0,0 +1,48 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
+suspension point in a coroutine.
+
+Some objects require that they be destroyed
@@ -0,0 +1,48 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
+suspension point in a coroutine.
+
+Some objects require that they be destroyed
@@ -0,0 +1,48 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
+suspension point in a coroutine.
+
+Some objects require that they be destroyed
@@ -241,6 +241,7 @@ Clang-Tidy Checks
:doc:`llvmlibc-restrict-system-libc-headers
`, "Yes"
:doc:`misc-confusable-identifiers `,
:doc:`misc-const-correctness `, "Yes"
+ :doc:`misc-coroutine-hostile-raii `_,
"Yes"
PiotrZSL wrote:
Check does not prov
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
@@ -0,0 +1,131 @@
+// RUN: %check_clang_tidy -std=c++20 %s misc-coroutine-hostile-raii %t \
+// RUN: -config="{CheckOptions: \
+// RUN: {misc-coroutine-hostile-raii.RAIIDenyList: \
+// RUN: 'my::Mutex; ::my::other::Mutex'}}"
+
+namespace std {
+
+templa
@@ -0,0 +1,131 @@
+// RUN: %check_clang_tidy -std=c++20 %s misc-coroutine-hostile-raii %t \
+// RUN: -config="{CheckOptions: \
+// RUN: {misc-coroutine-hostile-raii.RAIIDenyList: \
+// RUN: 'my::Mutex; ::my::other::Mutex'}}"
+
+namespace std {
+
+templa
@@ -0,0 +1,131 @@
+// RUN: %check_clang_tidy -std=c++20 %s misc-coroutine-hostile-raii %t \
+// RUN: -config="{CheckOptions: \
+// RUN: {misc-coroutine-hostile-raii.RAIIDenyList: \
+// RUN: 'my::Mutex; ::my::other::Mutex'}}"
+
+namespace std {
+
+templa
https://github.com/PiotrZSL requested changes to this pull request.
>From functional point of view doesn't look do bad.
My main concerns is about that 3 nested for-loops and ifs, this need to be
refactored to show more clear that it's trying to match varDecl defined before
suspension point.
ht
@@ -333,3 +333,22 @@ void instantiate_a_class_template() {
AClassTemplate withObjRef(o);
withObjRef.never_moves(o);
}
+
+namespace gh68209
+{
+ void f1([[maybe_unused]] int&& x) {}
+
+ void f2(__attribute__((unused)) int&& x) {}
PiotrZSL wrote:
Add info
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/69045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL approved this pull request.
Looks fine, I don't thing that this needs to be configurable.
If someone put explicitly attribute, then there is assumption that such param
can be unused.
https://github.com/llvm/llvm-project/pull/69045
https://github.com/PiotrZSL approved this pull request.
https://github.com/llvm/llvm-project/pull/69045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL created
https://github.com/llvm/llvm-project/pull/69067
Add support for constructor conversion based functional
cast. Allows to detect issues like:
const std::string_view test1 = std::string(a);
>From 6d4c7297e19193781d97fe19c3ccf8e65226ad0b Mon Sep 17 00:00:00 2001
@@ -269,6 +269,10 @@ Changes in existing checks
` to support for-loops with
iterators initialized by free functions like ``begin``, ``end``, or ``size``.
+- Improved :doc:`modernize-loop-convert
PiotrZSL wrote:
Typo "dor" + leading spaces in second and th
@@ -939,4 +939,13 @@ void fundamentalTypesTest() {
// CHECK-FIXES: for (double Double : Doubles)
}
+template void test() {
+ unsigned int test[3][p];
+ // Initialize to zero
+ for (unsigned int i = 0; i < p; ++i)
+for (unsigned int j = 0; j < 3; ++j) // CHECK-MESSAGE
https://github.com/PiotrZSL closed
https://github.com/llvm/llvm-project/pull/69045
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
PiotrZSL wrote:
I got mixed feelings about this. Simply we can use 'using' within extern "C",
there is nothing wrong with this. using/typedef doesn't haven linkage, that
just type alias. And extern "C" does not impact it. In oryginal issue I think
someone mistook extern "C" into something wher
PiotrZSL wrote:
Missing release notes for this check, follow current style.
Put also some description of change into merge request description.
https://github.com/llvm/llvm-project/pull/69103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
https://github.com/PiotrZSL approved this pull request.
LGTM, please provide more information in pull request description before
merging instead of just title.
https://github.com/llvm/llvm-project/pull/69062
___
cfe-commits mailing list
cfe-commits@li
PiotrZSL wrote:
Please also fix CI first, because looks like it fails on windows.
https://github.com/llvm/llvm-project/pull/69062
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
PiotrZSL wrote:
> 2. where do I find the release notes.
clang-tools-extra/docs/ReleaseNotes.rst
https://github.com/llvm/llvm-project/pull/69102
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
https://github.com/PiotrZSL approved this pull request.
https://github.com/llvm/llvm-project/pull/69103
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,82 @@
+//===--- CoroutineSuspensionHostileCheck.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: Apache-
@@ -554,6 +555,8 @@ getLambdaProperties(const MatchFinder::MatchResult &Result)
{
LP.Callable.Materialization = getCallableMaterialization(Result);
LP.Callable.Decl =
getCallMethodDecl(Result, LP.Callable.Type, LP.Callable.Materialization);
+ if (LP.Callable.Decl)
+
https://github.com/PiotrZSL commented:
Release notes Except that looks +- fine for me.
https://github.com/llvm/llvm-project/pull/69207
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
@@ -122,6 +122,8 @@ Improvements to clang-tidy
if any :program:`clang-tidy` subprocess exits with a non-zero code or if
exporting fixes fails.
+- Do not emit a `return` for fixes of `modernize-avoid-bind` when the function
returns void.
PiotrZSL wrote:
w
@@ -554,6 +555,8 @@ getLambdaProperties(const MatchFinder::MatchResult &Result)
{
LP.Callable.Materialization = getCallableMaterialization(Result);
LP.Callable.Decl =
getCallMethodDecl(Result, LP.Callable.Type, LP.Callable.Materialization);
+ if (LP.Callable.Decl)
+
@@ -343,6 +343,10 @@ Changes in existing checks
` check to
identify calls to static member functions with out-of-class inline
definitions.
+- Improved :doc:`modernize-avoid-bind
PiotrZSL wrote:
put this in alphabetical order, use double `` for return, as
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/69207
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL approved this pull request.
Release notes still need some work, except that looks fine.
https://github.com/llvm/llvm-project/pull/69207
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
https://github.com/PiotrZSL created
https://github.com/llvm/llvm-project/pull/69242
Extended IgnoreConversionFromTypes option to include
types without a declaration, such as built-in types.
>From 56bf257baff400189651c71a1639fbad8faa3a19 Mon Sep 17 00:00:00 2001
From: Piotr Zegar
Date: Mon, 16
https://github.com/PiotrZSL updated
https://github.com/llvm/llvm-project/pull/69242
>From 56bf257baff400189651c71a1639fbad8faa3a19 Mon Sep 17 00:00:00 2001
From: Piotr Zegar
Date: Mon, 16 Oct 2023 19:43:21 +
Subject: [PATCH 1/2] [clang-tidy] Improved
cppcoreguidelines-narrowing-conversions
Author: Piotr Zegar
Date: 2023-10-16T20:50:38Z
New Revision: c5b617c5e53d7af81c621d200e2cd03324538541
URL:
https://github.com/llvm/llvm-project/commit/c5b617c5e53d7af81c621d200e2cd03324538541
DIFF:
https://github.com/llvm/llvm-project/commit/c5b617c5e53d7af81c621d200e2cd03324538541.diff
LOG: [
@@ -0,0 +1,117 @@
+//===--- CoroutineHostileRAII.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: Ap
@@ -0,0 +1,117 @@
+//===--- CoroutineHostileRAII.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: Ap
@@ -0,0 +1,117 @@
+//===--- CoroutineHostileRAII.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: Ap
@@ -0,0 +1,117 @@
+//===--- CoroutineHostileRAII.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: Ap
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/68738
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,117 @@
+//===--- CoroutineHostileRAII.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: Ap
https://github.com/PiotrZSL commented:
I would still like to see some cleanup in matchers.
Mainly hasHostileRAII, when it would be redesigned to get matches as an
argument, then it could be renamed to something more generic, and other AST
matchers woudn't be needed then.
https://github.com/ll
@@ -0,0 +1,49 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
PiotrZSL wrote:
this should start with: "Detects when objects
@@ -0,0 +1,49 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
+suspension point in a coroutine.
+
+Some objects require that they be destroyed
@@ -0,0 +1,49 @@
+.. title:: clang-tidy - misc-coroutine-hostile-raii
+
+misc-coroutine-hostile-raii
+
+
+This check detects hostile-RAII objects which should not persist across a
+suspension point in a coroutine.
+
+Some objects require that they be destroyed
@@ -0,0 +1,45 @@
+//===--- CoroutineHostileRAIICheck.h - clang-tidy *-
C++-*-===//
+//
+// 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: Apa
@@ -0,0 +1,45 @@
+//===--- CoroutineHostileRAIICheck.h - clang-tidy *-
C++-*-===//
+//
+// 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: Apa
1101 - 1200 of 2500 matches
Mail list logo
