@@ -18,6 +18,12 @@ using namespace clang::ast_matchers;
namespace clang::tidy::cppcoreguidelines {
+namespace {
+AST_MATCHER(CXXRecordDecl, isInMacro) {
carlosgalvezp wrote:
(For another commit) This comes up often, maybe we can create an AST matcher
for th
https://github.com/HaohaiWen updated
https://github.com/llvm/llvm-project/pull/123870
>From 7c531922f20bf284a5cd83b20e94cb624ef031d2 Mon Sep 17 00:00:00 2001
From: Haohai Wen
Date: Thu, 9 Jan 2025 15:29:33 +0800
Subject: [PATCH 1/4] [COFF] Preserve UniqueID used to create MCSectionCOFF
This Un
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions h,cpp --
clang/lib/AST/ByteCode/IntegralAP.h clang/lib/AST/B
https://github.com/wangpc-pp approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/145267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ziqingluo-90 updated
https://github.com/llvm/llvm-project/pull/143487
>From 5825b324e47c78a939d8e62d1101e1574fd04638 Mon Sep 17 00:00:00 2001
From: Ziqing Luo
Date: Tue, 10 Jun 2025 15:50:10 +0800
Subject: [PATCH 1/7] [-Wunterminated-string-initialization] Handle C string
li
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/145260
This removes the workaround introduced in 3e1a9cf3b8 and 1ba7dc38d.
The issues should have been already resolved elsewhere, at least removing these
lines doesn't break any existing tests.
The workaround overwr
https://github.com/ChuanqiXu9 commented:
I don't feel bad but I'd like to leave the formal approvals to @jansvoboda11
and @Bigcheese here.
https://github.com/llvm/llvm-project/pull/145221
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
ranapratap55 wrote:
@arsenm ping.
https://github.com/llvm/llvm-project/pull/140210
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/143739
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Vladimir Vuksanovic
Date: 2025-06-22T19:00:11-07:00
New Revision: a17b5bce8c9bfa7767827df392855cff9f2af372
URL:
https://github.com/llvm/llvm-project/commit/a17b5bce8c9bfa7767827df392855cff9f2af372
DIFF:
https://github.com/llvm/llvm-project/commit/a17b5bce8c9bfa7767827df392855cff9f2af372
github-actions[bot] wrote:
@vvuksanovic Congratulations on having your first Pull Request (PR) merged into
the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a
problem with a bu
https://github.com/alexander-shaposhnikov closed
https://github.com/llvm/llvm-project/pull/142149
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yronglin wrote:
> Compile-time:
> https://llvm-compile-time-tracker.com/compare.php?from=40eee8ec7fb64dd1c6f15e8cd6e087f5d9f9d37e&to=2c67921274cfa31ccf7c59789df153fabc601816&stat=instructions:u
Thanks! This seems to relieve the performance regression.
https://github.com/llvm/llvm-project/pull/
https://github.com/naveen-seth edited
https://github.com/llvm/llvm-project/pull/145243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `arc-builder` running on
`arc-worker` while building `clang,llvm` at step 6
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/3/builds/17896
Here is the relevant pie
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-s390x-linux-lnt`
running on `systemz-1` while building `clang,llvm` at step 7 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/136/builds/4356
Here is the relevant piece of the b
@@ -0,0 +1,14 @@
+// RUN: grep -Ev "// *[A-Z-]+:" %s \
naveen-seth wrote:
Sorry, I misunderstood that! Implemented as unit tests now.
https://github.com/llvm/llvm-project/pull/145243
___
cfe-commits mailing list
cfe-co
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Rahul Joshi (jurahul)
Changes
---
Patch is 80.35 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/145207.diff
46 Files Affected:
- (modified) clang/include/clang/AST/AbstractBasicReader.h (+1
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Baranov Victor (vbvictor)
Changes
Fixed crush caused by types mismatch of expected `getline` and `getdelim`
signatures and actual user-defined signatures.
There are 2 ways to fix the issue:
1. Make sure that all paramet
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Baranov Victor (vbvictor)
Changes
Fixed crush caused by types mismatch of expected `getline` and `getdelim`
signatures and actual user-defined signatures.
There are 2 ways to fix the issue:
1. Make sure that all parameters of `CallEvent`
yronglin wrote:
The only difference between https://github.com/llvm/llvm-project/pull/143898
and this PR is
https://github.com/llvm/llvm-project/pull/145244/commits/6fe2cd4c723695c0ce6a4d2ced7a2b2628cdaf5d.
https://github.com/llvm/llvm-project/pull/145244
__
https://github.com/jurahul ready_for_review
https://github.com/llvm/llvm-project/pull/145207
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shiltian approved this pull request.
https://github.com/llvm/llvm-project/pull/145152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/miguelcsx updated
https://github.com/llvm/llvm-project/pull/145200
From be78d6dab41a9af13b884449c3e10e41937f02dc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miguel=20C=C3=A1rdenas?=
Date: Sun, 22 Jun 2025 03:32:24 +0200
Subject: [PATCH 1/3] [clang][nvlink-wrapper] Add support fo
nikic wrote:
Compile-time:
https://llvm-compile-time-tracker.com/compare.php?from=40eee8ec7fb64dd1c6f15e8cd6e087f5d9f9d37e&to=2c67921274cfa31ccf7c59789df153fabc601816&stat=instructions:u
https://github.com/llvm/llvm-project/pull/145244
___
cfe-commits
github-actions[bot] wrote:
:warning: Python code formatter, darker found issues in your code. :warning:
You can test this locally with the following command:
``bash
darker --check --diff -r HEAD~1...HEAD clang/bindings/python/clang/cindex.py
clang/test/bindings/python/tests/__ini
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From 704d36bdd22efd4645feaf8988e918ccfdf641a5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/4] [clang][python][test] Move python binding tests to lit
fr
denzor200 wrote:
@HerrCai0907 @PiotrZSL could you please approve again?
https://github.com/llvm/llvm-project/pull/142324
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/145243
>From 8cee6b33c54aca0ea69b3ad291287464523a3d7a Mon Sep 17 00:00:00 2001
From: Naveen Seth Hanig
Date: Sun, 22 Jun 2025 18:06:07 +0200
Subject: [PATCH 1/3] [clang-format] Handle Trailing Whitespace After Line
https://github.com/wenju-he updated
https://github.com/llvm/llvm-project/pull/144686
>From b5c9a4834accbac18664f1431acc5204c0527fa8 Mon Sep 17 00:00:00 2001
From: Wenju He
Date: Wed, 18 Jun 2025 14:21:09 +0200
Subject: [PATCH 1/7] [InstCombine] Don't folder select to or if value argument
is us
@@ -0,0 +1,25 @@
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
UTC_ARGS: --version 5
+; RUN: opt -S -mtriple=amdgcn-amd-amdhsa -passes=instcombine %s | FileCheck %s
+
+; Check that `select B, true, C` isn't optimized to `or B, C`,
+; because the inval
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/142324
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,219 @@
+//===--- BoolBitwiseOperationCheck.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/HerrCai0907 approved this pull request.
lgtm
https://github.com/llvm/llvm-project/pull/142324
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From e57e53c7e5abdb4c390a04b4ce9084dec9e71dd5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/5] [clang][python][test] Move python binding tests to lit
fr
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From e57e53c7e5abdb4c390a04b4ce9084dec9e71dd5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/6] [clang][python][test] Move python binding tests to lit
fr
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-s390x-linux` running
on `systemz-1` while building `clang,llvm` at step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/42/builds/5087
Here is the relevant piece of the build
https://github.com/qiongsiwu edited
https://github.com/llvm/llvm-project/pull/144105
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-x64-windows-msvc`
running on `windows-gcebot2` while building `clang,llvm` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/63/builds/7334
Here is the relevant piece of the
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From 704d36bdd22efd4645feaf8988e918ccfdf641a5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/5] [clang][python][test] Move python binding tests to lit
fr
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From 704d36bdd22efd4645feaf8988e918ccfdf641a5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/3] [clang][python][test] Move python binding tests to lit
fr
https://github.com/dingxiangfei2009 updated
https://github.com/llvm/llvm-project/pull/143424
>From aed0d6e3ead7e691b34f16b7cd8e830860758f2f Mon Sep 17 00:00:00 2001
From: Xiangfei Ding
Date: Mon, 9 Jun 2025 13:57:18 +
Subject: [PATCH 1/2] [clang-c] introduce queries on GCC-style inline asse
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From 704d36bdd22efd4645feaf8988e918ccfdf641a5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/2] [clang][python][test] Move python binding tests to lit
fr
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From e57e53c7e5abdb4c390a04b4ce9084dec9e71dd5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/7] [clang][python][test] Move python binding tests to lit
fr
https://github.com/DeinAlptraum ready_for_review
https://github.com/llvm/llvm-project/pull/144873
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DeinAlptraum updated
https://github.com/llvm/llvm-project/pull/144873
>From e57e53c7e5abdb4c390a04b4ce9084dec9e71dd5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/6] [clang][python][test] Move python binding tests to lit
fr
@@ -2753,10 +2753,18 @@ static bool resolveAllocationOverloadInterior(
if (Diagnose) {
// If this is an allocation of the form 'new (p) X' for some object
// pointer p (or an expression that will decay to such a pointer),
- // diagnose the missing inclusion
https://github.com/vbvictor updated
https://github.com/llvm/llvm-project/pull/144270
>From 07392e16740d3d89700da43e22b4d90197bb4d4a Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Sun, 15 Jun 2025 19:56:05 +0300
Subject: [PATCH 1/3] [Clang] Improve diagnostics when 'placement new' was
call
@@ -0,0 +1,14 @@
+// RUN: grep -Ev "// *[A-Z-]+:" %s \
HazardyKnusperkeks wrote:
We test those things in the unit tests.
https://github.com/llvm/llvm-project/pull/145243
___
cfe-commits mailing list
cfe-commits@lists.l
@@ -0,0 +1,14 @@
+// RUN: grep -Ev "// *[A-Z-]+:" %s \
HazardyKnusperkeks wrote:
That's not what I meant. We test in `clang/unittests/Format/FormatTest.cpp`.
https://github.com/llvm/llvm-project/pull/145243
___
cfe-com
Author: Stanislav Mekhanoshin
Date: 2025-06-22T12:52:05-07:00
New Revision: 40eee8ec7fb64dd1c6f15e8cd6e087f5d9f9d37e
URL:
https://github.com/llvm/llvm-project/commit/40eee8ec7fb64dd1c6f15e8cd6e087f5d9f9d37e
DIFF:
https://github.com/llvm/llvm-project/commit/40eee8ec7fb64dd1c6f15e8cd6e087f5d9f9d3
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: None (yronglin)
Changes
Depends on [[clang][Preprocessor] Add peekNextPPToken, makes look ahead next
token without side-effects](https://github.com/llvm/llvm-project/pull/143898).
This PR fix the performance regression that introd
https://github.com/yronglin created
https://github.com/llvm/llvm-project/pull/145244
Depends on [[clang][Preprocessor] Add peekNextPPToken, makes look ahead next
token without side-effects](https://github.com/llvm/llvm-project/pull/143898).
This PR fix the performance regression that introduce
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (yronglin)
Changes
Depends on [[clang][Preprocessor] Add peekNextPPToken, makes look ahead next
token without side-effects](https://github.com/llvm/llvm-project/pull/143898).
This PR fix the performance regression that introduced in
https://github.com/AmrDeveloper created
https://github.com/llvm/llvm-project/pull/145238
- Merge all vector-type invalid tests into one file.
- Update the names of the functions in the invalid test
Issue https://github.com/llvm/llvm-project/issues/136487
>From 02121b86a2ee7cef7e2fddde059e4efc9
topperc wrote:
> Since https://github.com/llvm/llvm-project/pull/144848 has landed, I reverted
> this reverted commit by
> https://github.com/llvm/llvm-project/commit/f78819aeef32e50ac3fec9a175b70a971b7c10e5.
In the future use "Recommit" in the title instead of "Revert Revert".
https://github
yronglin wrote:
> > 1. IIUC, `42935M | 43035M (+0.23%)`does this mean the compiler output
> > binary size increased by 0.23%?
>
> This is the number of instructions retired. It's a metric for compilation
> time, not binary size.
Thanks, I reproduced this issue, and created an new PR to fix is
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/145229
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,14 @@
+// RUN: grep -Ev "// *[A-Z-]+:" %s \
naveen-seth wrote:
Fixed.
https://github.com/llvm/llvm-project/pull/145243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/145243
>From 8cee6b33c54aca0ea69b3ad291287464523a3d7a Mon Sep 17 00:00:00 2001
From: Naveen Seth Hanig
Date: Sun, 22 Jun 2025 18:06:07 +0200
Subject: [PATCH 1/2] [clang-format] Handle Trailing Whitespace After Line
vbvictor wrote:
Also, writing "Fixes https://github.com/llvm/llvm-project/issues/144642."; at
the end of the PR description automatically links the issue to the PR so
everyone easily know what problem you solved.
https://github.com/llvm/llvm-project/pull/144828
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-format
Author: Naveen Seth Hanig (naveen-seth)
Changes
Fixes #145226.
Implement
[P2223R2](https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2021/p2223r2.pdf)
in clang-format to correctly handle cases where a backslas
https://github.com/naveen-seth edited
https://github.com/llvm/llvm-project/pull/145243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/naveen-seth edited
https://github.com/llvm/llvm-project/pull/145243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/naveen-seth created
https://github.com/llvm/llvm-project/pull/145243
…223R2)
Fixes #145226.
Implement
[P2223R2](https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2021/p2223r2.pdf)
in clang-format to correctly handle cases where a backslash '\\' is followed by
trailing w
https://github.com/miguelcsx updated
https://github.com/llvm/llvm-project/pull/145200
From 883345d337df982a87c55b11b3374cf661b7b49e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miguel=20C=C3=A1rdenas?=
Date: Sun, 22 Jun 2025 03:32:24 +0200
Subject: [PATCH 1/3] [clang][nvlink-wrapper] Add support fo
Miguel =?utf-8?q?Cárdenas?=
Message-ID:
In-Reply-To:
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp -
https://github.com/miguelcsx updated
https://github.com/llvm/llvm-project/pull/145200
From 883345d337df982a87c55b11b3374cf661b7b49e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miguel=20C=C3=A1rdenas?=
Date: Sun, 22 Jun 2025 03:32:24 +0200
Subject: [PATCH 1/2] [clang][nvlink-wrapper] Add support fo
Miguel =?utf-8?q?Cárdenas?=
Message-ID:
In-Reply-To:
https://github.com/jhuber6 approved this pull request.
So, this is probably a little weird since we now have two ways this is parsed.
One through the LLVM commandline interface and another here as the command
options interface. The reason
https://github.com/miguelcsx updated
https://github.com/llvm/llvm-project/pull/145200
From 883345d337df982a87c55b11b3374cf661b7b49e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miguel=20C=C3=A1rdenas?=
Date: Sun, 22 Jun 2025 03:32:24 +0200
Subject: [PATCH 1/2] [clang][nvlink-wrapper] Add support fo
https://github.com/vbvictor created
https://github.com/llvm/llvm-project/pull/145229
Fixed crush caused by types mismatch of expected `getline` and `getdelim`
signatures and actual user-defined signatures.
There are 2 ways to fix the issue:
1. Make sure that all parameters of `CallEvent` unde
https://github.com/AmrDeveloper edited
https://github.com/llvm/llvm-project/pull/145238
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clangir
@llvm/pr-subscribers-clang
Author: Amr Hesham (AmrDeveloper)
Changes
- Merge all vector-type invalid tests into one file.
- Update the names of the functions in the invalid test
Issue https://github.com/llvm/llvm-project/issues/136487
---
Full
@@ -0,0 +1,146 @@
+//===--- QueryCheck.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
steakhal wrote:
Ideally, we could reuse the signature matching of the
`StdLibraryFunctionsChecker`. It shouldn't be particularly difficult to hoist
it, but also a considerable work by the size of the change.
I think the `Checkers` library component (or directory) is suitable for it. We
could c
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: Rahul Joshi (jurahul)
Changes
---
Patch is 80.35 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/145207.diff
46 Files Affected:
- (modified) clang/include/clang/AST/AbstractBasicRead
evelez7 wrote:
### Merge activity
* **Jun 21, 6:55 PM UTC**: A user started a stack merge that includes this pull
request via
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/145069).
https://github.com/llvm/llvm-project/pull/145069
yronglin wrote:
> A couple of quick notes:
>
> * This is my first LLVM PR, so if there are any issues with code style or
> conventions, please let me know!
> * I'm not entirely satisfied with the `VD->getName().starts_with("__range")`
> check, but it was the most reliable approach I found. Wal
@@ -0,0 +1,79 @@
+// RUN: %clangxx -std=c++23 -fsyntax-only -Xclang -verify %s
+
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
yronglin wrote:
We can construct some simple fake std dependencies.
Eg.
```cpp
namespace std {
@@ -0,0 +1,79 @@
+// RUN: %clangxx -std=c++23 -fsyntax-only -Xclang -verify %s
zwuis wrote:
Use `%clang_cc1` for frontend tests.
`-Xclang` can be removed.
https://github.com/llvm/llvm-project/pull/145164
___
cfe-commi
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/145164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,79 @@
+// RUN: %clangxx -std=c++23 -fsyntax-only -Xclang -verify %s
+
+#include
+#include
+#include
+#include
+#include
+#include
+#include
+#include
zwuis wrote:
Do not include files outside test folders. You can search `namespace std` to
see
@@ -57,6 +57,31 @@ enum LifetimeKind {
};
using LifetimeResult =
llvm::PointerIntPair;
+
+/// Returns true if the given entity is part of a range-based for loop and
+/// should trigger lifetime extension under C++23 rules.
+///
+/// This handles both explicit range loop var
https://github.com/zwuis commented:
Thank you for the patch!
Please add a release note in `clang/docs/ReleaseNotes.rst` so that users can
know the improvement.
https://github.com/llvm/llvm-project/pull/145164
___
cfe-commits mailing list
cfe-commits@
HighCommander4 wrote:
@kleinesfilmroellchen (or anyone else reading this thread), any objection to
switching to the approach suggested by Kadir in the previous comment, of
matching against resolved paths?
https://github.com/llvm/llvm-project/pull/140594
https://github.com/xlauko edited
https://github.com/llvm/llvm-project/pull/145163
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tclin914 wrote:
Since https://github.com/llvm/llvm-project/pull/144848 has landed, I reverted
this reverted commit by
https://github.com/llvm/llvm-project/commit/f78819aeef32e50ac3fec9a175b70a971b7c10e5.
https://github.com/llvm/llvm-project/pull/144402
_
Author: Jim Lin
Date: 2025-06-22T17:54:37+08:00
New Revision: f78819aeef32e50ac3fec9a175b70a971b7c10e5
URL:
https://github.com/llvm/llvm-project/commit/f78819aeef32e50ac3fec9a175b70a971b7c10e5
DIFF:
https://github.com/llvm/llvm-project/commit/f78819aeef32e50ac3fec9a175b70a971b7c10e5.diff
LOG:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Marco Vitale (mrcvtl)
Changes
C++23 mandates that temporaries used in range-based for loops are
lifetime-extended
to cover the full loop. This patch adds a check for loop variables and compiler-
generated `__range` bindings to apply the co
https://github.com/mrcvtl ready_for_review
https://github.com/llvm/llvm-project/pull/145164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mrcvtl wrote:
A couple of quick notes:
- This is my first LLVM PR, so if there are any issues with code style or
conventions, please let me know!
- I'm not entirely satisfied with the `VD->getName().starts_with("__range")`
check, but it was the most reliable approach I found. Walking up the AS
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/145220
>From cc92d075db7e9b0cd4c8187e9dc8e20bf4151fdc Mon Sep 17 00:00:00 2001
From: Naveen Seth Hanig
Date: Fri, 13 Jun 2025 09:19:21 +0200
Subject: [PATCH] [clang][modules-driver] Add scanner to detect C++20 modu
https://github.com/naveen-seth created
https://github.com/llvm/llvm-project/pull/145220
This PR is part of a series aimed at implementing native support for explicit
module builds from the Clang driver.
This introduces a new scanner that detects C++20 module usage in source files
without using
@@ -118,8 +138,8 @@ class Document {
BulletList &addBulletList();
/// Doesn't contain any trailing newlines.
- /// We try to make the markdown human-readable, e.g. avoid extra escaping.
- /// At least one client (coc.nvim) displays the markdown verbatim!
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Naveen Seth Hanig (naveen-seth)
Changes
This adds standard C++ named modules to the scanning output format
'experimental-full'.
This is motivated by an effort to provide native support for explicit module
builds from the Clang driver.
Th
https://github.com/naveen-seth created
https://github.com/llvm/llvm-project/pull/145221
This adds standard C++ named modules to the scanning output format
'experimental-full'.
This is motivated by an effort to provide native support for explicit module
builds from the Clang driver.
The driver
https://github.com/xlauko approved this pull request.
lgtm
https://github.com/llvm/llvm-project/pull/145163
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-clang
Author: Naveen Seth Hanig (naveen-seth)
Changes
This PR is part of a series aimed at implementing native support for explicit
module builds from the Clang driver.
This introduces a new scanner that detects C++20 m
nikic wrote:
> 1. IIUC, `42935M | 43035M (+0.23%)`does this mean the compiler output binary
> size increased by 0.23%?
This is the number of instructions retired. It's a metric for compilation time,
not binary size.
https://github.com/llvm/llvm-project/pull/144233
https://github.com/NewSigma updated
https://github.com/llvm/llvm-project/pull/144319
>From 584d47295f7719f96ee77d32a6b4329f82be Mon Sep 17 00:00:00 2001
From: NewSigma
Date: Mon, 16 Jun 2025 15:36:53 +0800
Subject: [PATCH 1/9] Add LifetimeMovePass
---
clang/test/CodeGenCoroutines/pr56919.
1 - 100 of 103 matches
Mail list logo
