Author: Timm Baeder
Date: 2025-09-08T08:56:46+02:00
New Revision: 8ba0d0f72f25229cfa2de9b7352866d7a33c1e54
URL:
https://github.com/llvm/llvm-project/commit/8ba0d0f72f25229cfa2de9b7352866d7a33c1e54
DIFF:
https://github.com/llvm/llvm-project/commit/8ba0d0f72f25229cfa2de9b7352866d7a33c1e54.diff
L
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/157328
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
Review ping
https://github.com/llvm/llvm-project/pull/153337
___
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
Author: Piyou Chen (BeMg)
Changes
Implement https://github.com/riscv-non-isa/riscv-elf-psabi-doc/pull/419.
This patch makes the type extension based on the variable type for BIGINT,
rather than using sign extension for all cases.
---
Full diff
https://github.com/BeMg updated https://github.com/llvm/llvm-project/pull/156592
>From 092700505e150a5de6e623bf12f0dbd557017e24 Mon Sep 17 00:00:00 2001
From: Piyou Chen
Date: Fri, 13 Jun 2025 03:11:08 -0700
Subject: [PATCH 1/4] [clang][RISCV] support BITINT with mixed-type
---
clang/lib/CodeG
Author: Nicolas van Kempen
Date: 2025-09-08T04:56:10Z
New Revision: c5ed3c6073aa6b22baff126b901a297fba7ecf64
URL:
https://github.com/llvm/llvm-project/commit/c5ed3c6073aa6b22baff126b901a297fba7ecf64
DIFF:
https://github.com/llvm/llvm-project/commit/c5ed3c6073aa6b22baff126b901a297fba7ecf64.diff
nicovank wrote:
Wow, I thought auto merge would wait for a reviewer approval. Oops. Good thing
this is harmless. Please let me know if there was an issue.
https://github.com/llvm/llvm-project/pull/157384
___
cfe-commits mailing list
cfe-commits@lists.
https://github.com/t-a-james updated
https://github.com/llvm/llvm-project/pull/154746
>From 39ad4945d05f3e88fd32dfdea585aa99c6ba985f Mon Sep 17 00:00:00 2001
From: Tom James
Date: Thu, 21 Aug 2025 13:14:24 +0100
Subject: [PATCH 01/30] [clang-tidy] New bugprone-method-hiding check
---
.../bugp
https://github.com/t-a-james updated
https://github.com/llvm/llvm-project/pull/154746
>From 39ad4945d05f3e88fd32dfdea585aa99c6ba985f Mon Sep 17 00:00:00 2001
From: Tom James
Date: Thu, 21 Aug 2025 13:14:24 +0100
Subject: [PATCH 01/30] [clang-tidy] New bugprone-method-hiding check
---
.../bugp
@@ -0,0 +1,124 @@
+// RUN: %check_clang_tidy %s bugprone-derived-method-shadowing-base-method %t
+
+class Base
+{
+void method();
+void methodWithArg(int I);
+
+virtual Base* getThis() = 0;
+};
+
+class A : public Base
+{
+public:
+void method();
+// CHECK-MESSAGE
@@ -0,0 +1,127 @@
+//===--===//
+//
+// 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
thesamesam wrote:
https://github.com/llvm/llvm-project/issues/74605
https://github.com/llvm/llvm-project/pull/157364
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,127 @@
+//===--===//
+//
+// 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
https://github.com/t-a-james updated
https://github.com/llvm/llvm-project/pull/154746
>From 39ad4945d05f3e88fd32dfdea585aa99c6ba985f Mon Sep 17 00:00:00 2001
From: Tom James
Date: Thu, 21 Aug 2025 13:14:24 +0100
Subject: [PATCH 01/25] [clang-tidy] New bugprone-method-hiding check
---
.../bugp
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-aarch64-windows`
running on `linaro-armv8-windows-msvc-05` while building
`bolt,clang,lldb,llvm,mlir` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/141/builds/11367
Here is t
https://github.com/nicovank closed
https://github.com/llvm/llvm-project/pull/157384
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nicovank wrote:
Feel free to ship immediately if OK.
https://github.com/llvm/llvm-project/pull/157384
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/nicovank auto_merge_enabled
https://github.com/llvm/llvm-project/pull/157384
___
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-tools-extra
Author: Nicolas van Kempen (nicovank)
Changes
Fix #157343.
---
Full diff: https://github.com/llvm/llvm-project/pull/157384.diff
1 Files Affected:
- (modified)
clang-tools-extra/docs/clang-tidy/checks/performance/noexcept-move-con
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Nicolas van Kempen (nicovank)
Changes
Fix #157343.
---
Full diff: https://github.com/llvm/llvm-project/pull/157384.diff
1 Files Affected:
- (modified)
clang-tools-extra/docs/clang-tidy/checks/performance/noexcept-move-constructo
https://github.com/nicovank created
https://github.com/llvm/llvm-project/pull/157384
Fix #157343.
>From 9d425a009d17385cc9bf1b9649084e68fde65c9f Mon Sep 17 00:00:00 2001
From: Nicolas van Kempen
Date: Mon, 8 Sep 2025 00:44:15 -0400
Subject: [PATCH] [clang-tidy] Minor documentation wording fo
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-remote-linux-ubuntu`
running on `as-builder-9` while building `bolt,clang,lldb,llvm,mlir` at step 16
"test-check-lldb-api".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/195/builds/14270
Here
https://github.com/arsenm closed
https://github.com/llvm/llvm-project/pull/157321
___
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
`sanitizer-x86_64-linux-fast` running on `sanitizer-buildbot4` while building
`clang-tools-extra` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/169/builds/14787
Here is the rel
https://github.com/itzexpoexpo updated
https://github.com/llvm/llvm-project/pull/154580
From 000ea7fb5703f9a409d64c2ba61e556e21187cee Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1=C5=A1=20Slanina?=
Date: Wed, 20 Aug 2025 19:28:23 +0200
Subject: [PATCH 01/15] [clang-format] Add option
Allo
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/157321
>From 5f2205d454e38e63ab6d9ed2a41ff8d8b674ec6b Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Sun, 7 Sep 2025 09:03:22 +0900
Subject: [PATCH 1/2] MC: Add Triple overloads for more MC constructors
Avoids mor
https://github.com/arsenm auto_merge_enabled
https://github.com/llvm/llvm-project/pull/157321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/155982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Yanzuo Liu
Date: 2025-09-08T10:19:52+08:00
New Revision: 2ec6b3bceda275b4146bb229668e38797d6afe62
URL:
https://github.com/llvm/llvm-project/commit/2ec6b3bceda275b4146bb229668e38797d6afe62
DIFF:
https://github.com/llvm/llvm-project/commit/2ec6b3bceda275b4146bb229668e38797d6afe62.diff
LO
zwuis wrote:
Thank you for your review!
https://github.com/llvm/llvm-project/pull/155982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/157321
>From 5f2205d454e38e63ab6d9ed2a41ff8d8b674ec6b Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Sun, 7 Sep 2025 09:03:22 +0900
Subject: [PATCH] MC: Add Triple overloads for more MC constructors
Avoids more Tr
https://github.com/zwuis updated
https://github.com/llvm/llvm-project/pull/155982
>From 426caa9f66cddd1deac23b397baf75f6809f6f52 Mon Sep 17 00:00:00 2001
From: Yanzuo Liu
Date: Fri, 29 Aug 2025 15:15:24 +0800
Subject: [PATCH 1/6] Handle nested-name-specifier in
"llvm-prefer-isa-or-dyn-cast-in-
itzexpoexpo wrote:
Nevermind, I missed that in this case `Never` should not apply since the record
is not considered as on a single line, only the block is, so your expected
behavior is indeed correct. I'll look into fixing it.
https://github.com/llvm/llvm-project/pull/154580
_
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/157321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/comex edited https://github.com/llvm/llvm-project/pull/154034
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
comex wrote:
Updated. Aside from adding a release note, I also discovered some related
issues that are fixed by adding a `SFINAETrap` in addition to the
`EnterExpressionEvaluationContext`. See the added test in
`non-trivial-c-union.m` (the `__builtin_addressof` call previously yielded a
com
https://github.com/comex updated
https://github.com/llvm/llvm-project/pull/154034
>From 5a0549b35cd5b3b4ecb8a25bcde1560cf85c094d Mon Sep 17 00:00:00 2001
From: comex
Date: Sun, 17 Aug 2025 12:41:44 -0700
Subject: [PATCH] [clang] Fix side effects resolving overloads of builtin
functions (#13865
itzexpoexpo wrote:
Wouldn't this be the correct behavior though? Since records are a subset of
blocks, it feels like it should override the blocks option, letting it be the
catch-all and `...Record...` an exception.
Although looking at `tryMergeSimpleBlock`, the option already overrides
`Allo
@@ -196,6 +196,16 @@ Changes in existing checks
adding an option to allow pointer arithmetic via prefix/postfix increment or
decrement operators.
+- Improved :doc:`llvm-prefer-isa-or-dyn-cast-in-conditionals
+ ` check:
+
+ - Fix-it handles Callees with nested-name-specif
@@ -22,105 +23,104 @@ AST_MATCHER(Expr, isMacroID) { return
Node.getExprLoc().isMacroID(); }
void PreferIsaOrDynCastInConditionalsCheck::registerMatchers(
MatchFinder *Finder) {
- auto Condition = hasCondition(implicitCastExpr(has(
- callExpr(unless(isMacroID()), un
https://github.com/aaronpuchert edited
https://github.com/llvm/llvm-project/pull/157171
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/157128
>From 97ee7e3b1cfc06b434b71987b9b0a9fd8b14a236 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Fri, 5 Sep 2025 10:42:07 -0500
Subject: [PATCH 1/2] [Clang] Rename elementwise builtins to `clzg` and `ctzg`
Summ
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= Slanina ,
=?utf-8?q?Tomáš?= S
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
henrybw wrote:
cc @RKSimon @rnk @efriedma-quic
https://github.com/llvm/llvm-project/pull/157365
___
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
Author: Henry Baba-Weiss (henrybw)
Changes
The MSVC ABI almost always returns vector types directly, but on x86 and
x86_64, there seems to be a special case for member functions, which return
vector types indirectly.
Fixes #104.
---
Full diff:
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Henry Baba-Weiss (henrybw)
Changes
The MSVC ABI almost always returns vector types directly, but on x86 and
x86_64, there seems to be a special case for member functions, which return
vector types indirectly.
Fixes #104.
---
Ful
jhuber6 wrote:
> I don't really mind what they're called. As I said in the original RFC and PR
> that introduced the builtins, I have no particular favourites.
>
> However, do note that the elementwise builtins are not _exactly_ like
> clzg/ctzg in that they don't have _target-specific_ zero-i
https://github.com/henrybw created
https://github.com/llvm/llvm-project/pull/157365
The MSVC ABI almost always returns vector types directly, but on x86 and
x86_64, there seems to be a special case for member functions, which return
vector types indirectly.
Fixes #104.
>From 8f42b86f8d487b9f
aaronpuchert wrote:
Commented on #156760: the issues is that we don't support scoped lockable
arguments for the attributes. (That is, we support a different syntax since
#110523, but we can't use that for `std::lock` which accepts both lockable and
scoped lockable arguments, possibly even mixe
aaronpuchert wrote:
The behavior of before/after is a bit weird in some cases, but I guess it's
mostly used within the same class where it should be fine. Unlike the rest of
the analysis, it doesn't work with expressions but strips them down to the bare
`FieldDecl` or `VarDecl`. This has some
@@ -0,0 +1,45 @@
+// RUN: %clang_cc1 -include %s -verify -fsyntax-only -Wthread-safety %s
+
+#ifndef HEADER
+#define HEADER
+
+#define LOCKABLE__attribute__ ((lockable))
+#define EXCLUSIVE_LOCK_FUNCTION(...) __attribute__
((exclusive_lock_function(__VA_ARGS__)))
+
+
https://github.com/llvm-beanz closed
https://github.com/llvm/llvm-project/pull/154179
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Chris B
Date: 2025-09-07T16:05:37-05:00
New Revision: 799d3466fa97e24082cb036e71a7a92f72597b4e
URL:
https://github.com/llvm/llvm-project/commit/799d3466fa97e24082cb036e71a7a92f72597b4e
DIFF:
https://github.com/llvm/llvm-project/commit/799d3466fa97e24082cb036e71a7a92f72597b4e.diff
LOG:
@@ -0,0 +1,45 @@
+// RUN: %clang_cc1 -include %s -verify -fsyntax-only -Wthread-safety %s
+
+#ifndef HEADER
+#define HEADER
+
+#define LOCKABLE__attribute__ ((lockable))
+#define EXCLUSIVE_LOCK_FUNCTION(...) __attribute__
((exclusive_lock_function(__VA_ARGS__)))
+
+
https://github.com/jeaye updated
https://github.com/llvm/llvm-project/pull/156977
>From 65d55e32ffa78138c016040c1344e929528a017b Mon Sep 17 00:00:00 2001
From: jeaye
Date: Thu, 4 Sep 2025 13:48:40 -0700
Subject: [PATCH 1/2] Allow for custom code model in clang::Interpreter
This is necessary wh
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-backend-x86
Author: None (Sirraide)
Changes
GCC 14 also made this an error by default, and I thought it’d make sense to
follow suit.
---
Patch is 146.31 KiB, truncated to 20.00 KiB below, full version:
https://github.com/ll
https://github.com/aaronpuchert requested changes to this pull request.
It looks like there is some confusion around the concepts here: `unique_lock`
is not a `lockable` but a
[`scoped_lockable`](https://clang.llvm.org/docs/ThreadSafetyAnalysis.html#scoped-capability).
The latter should be aut
https://github.com/jeaye updated
https://github.com/llvm/llvm-project/pull/157359
>From 73669c01825214c51d0ddb79dce6ec0235633d08 Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Thu, 30 May 2024 05:05:41 +
Subject: [PATCH 1/9] [clang-repl] Teach clang-repl how to load PCHs.
---
clang/
@@ -0,0 +1,45 @@
+// RUN: %clang_cc1 -include %s -verify -fsyntax-only -Wthread-safety %s
+
+#ifndef HEADER
+#define HEADER
+
+#define LOCKABLE__attribute__ ((lockable))
+#define EXCLUSIVE_LOCK_FUNCTION(...) __attribute__
((exclusive_lock_function(__VA_ARGS__)))
+
+
@@ -2443,6 +2443,22 @@ void ThreadSafetyAnalyzer::intersectAndWarn(FactSet
&EntrySet,
if (join(FactMan[*EntryIt], ExitFact, JoinLoc, EntryLEK))
*EntryIt = Fact;
} else if (!ExitFact.managed() || EntryLEK == LEK_LockedAtEndOfFunction) {
+ if (EntryLEK ==
stmuench wrote:
> I will land the patch at the end of the week
@vbvictor many thanks for taking care
https://github.com/llvm/llvm-project/pull/132924
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
jeaye wrote:
Looks like the previous run of the Linux job failed in an unrelated area:
https://productionresultssa6.blob.core.windows.net/actions-results/de1b4c8c-9dc9-4d5e-90f7-2ed4ede76dce/workflow-job-run-20b49bb8-304c-5069-9f6b-84a81382ac9a/logs/job/job-logs.txt?rsct=text%2Fplain&se=2025-09-
https://github.com/vbvictor approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/155982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
localspook wrote:
Re: adding it under an option
I'm struggling to imagine a user that wants this change:
```diff
- int i = 10;
+ int const i = 10;
```
but *doesn't* want this change:
```diff
- auto i = 10;
+ auto const i = 10;
```
The way I see it, this PR is more of a bugfix rather than a new f
https://github.com/localspook updated
https://github.com/llvm/llvm-project/pull/157319
>From e28e9e234b6165b49884cf254e1fb4efe44fe756 Mon Sep 17 00:00:00 2001
From: Victor Chernyakin
Date: Sat, 6 Sep 2025 17:35:36 -0700
Subject: [PATCH 1/3] [clang-tidy] make `misc-const-correctness` work with
https://github.com/jeaye updated
https://github.com/llvm/llvm-project/pull/157359
>From 73669c01825214c51d0ddb79dce6ec0235633d08 Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Thu, 30 May 2024 05:05:41 +
Subject: [PATCH 1/8] [clang-repl] Teach clang-repl how to load PCHs.
---
clang/
https://github.com/vbvictor approved this pull request.
https://github.com/llvm/llvm-project/pull/147553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
@stmuench 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 build
Author: St. Muench
Date: 2025-09-07T22:54:37+03:00
New Revision: 6fc32e93066c59a39b3c8c9be4e4f416653a917e
URL:
https://github.com/llvm/llvm-project/commit/6fc32e93066c59a39b3c8c9be4e4f416653a917e
DIFF:
https://github.com/llvm/llvm-project/commit/6fc32e93066c59a39b3c8c9be4e4f416653a917e.diff
LO
https://github.com/vbvictor closed
https://github.com/llvm/llvm-project/pull/132924
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
localspook wrote:
> I notice now there's no update in Release Notes for this.
There is though; did you maybe mean documentation?
https://github.com/llvm/llvm-project/pull/157319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.ll
vbvictor wrote:
Do you wish to have this merged or wait for your commit approval (it can take
2-3week if unlucky)?
https://github.com/llvm/llvm-project/pull/157123
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/
@@ -10,6 +10,12 @@ template
void type_dependent_variables() {
T value = 42;
auto &ref = value;
+ // CHECK-MESSAGES:[[@LINE-1]]:3: warning: variable 'ref' of type 'int &' can
be declared 'const'
+ // CHECK-FIXES: auto const&ref = value;
+ //
+ // FIXME: This is a fals
https://github.com/localspook updated
https://github.com/llvm/llvm-project/pull/157319
>From e28e9e234b6165b49884cf254e1fb4efe44fe756 Mon Sep 17 00:00:00 2001
From: Victor Chernyakin
Date: Sat, 6 Sep 2025 17:35:36 -0700
Subject: [PATCH 1/2] [clang-tidy] make `misc-const-correctness` work with
https://github.com/jeaye updated
https://github.com/llvm/llvm-project/pull/157359
>From 73669c01825214c51d0ddb79dce6ec0235633d08 Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Thu, 30 May 2024 05:05:41 +
Subject: [PATCH 1/8] [clang-repl] Teach clang-repl how to load PCHs.
---
clang/
@@ -10,6 +10,12 @@ template
void type_dependent_variables() {
T value = 42;
auto &ref = value;
+ // CHECK-MESSAGES:[[@LINE-1]]:3: warning: variable 'ref' of type 'int &' can
be declared 'const'
+ // CHECK-FIXES: auto const&ref = value;
localspook wrote
https://github.com/jeaye updated
https://github.com/llvm/llvm-project/pull/157359
>From 73669c01825214c51d0ddb79dce6ec0235633d08 Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Thu, 30 May 2024 05:05:41 +
Subject: [PATCH 1/8] [clang-repl] Teach clang-repl how to load PCHs.
---
clang/
vbvictor wrote:
> Perhaps having a section for "Breaking changes" or otherwise "Stuff you need
> to take action on" would help.
I support this, another solution could be bold text or preamble like "Important
⚠️".
But I think dedicated section is the best.
https://github.com/llvm/llvm-pr
https://github.com/jeaye edited https://github.com/llvm/llvm-project/pull/157359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kr-2003 updated
https://github.com/llvm/llvm-project/pull/157358
>From 4410aeb08dc14a4f29c9ec0e8730a1bde3386665 Mon Sep 17 00:00:00 2001
From: kr-2003
Date: Mon, 8 Sep 2025 00:19:44 +0530
Subject: [PATCH 1/4] [clang-repl] Adding custom lambda in launchExecutor
---
clang/inc
https://github.com/jeaye updated
https://github.com/llvm/llvm-project/pull/157359
>From 73669c01825214c51d0ddb79dce6ec0235633d08 Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Thu, 30 May 2024 05:05:41 +
Subject: [PATCH 1/8] [clang-repl] Teach clang-repl how to load PCHs.
---
clang/
https://github.com/jeaye edited https://github.com/llvm/llvm-project/pull/157359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kr-2003 updated
https://github.com/llvm/llvm-project/pull/157358
>From 4410aeb08dc14a4f29c9ec0e8730a1bde3386665 Mon Sep 17 00:00:00 2001
From: kr-2003
Date: Mon, 8 Sep 2025 00:19:44 +0530
Subject: [PATCH 1/3] [clang-repl] Adding custom lambda in launchExecutor
---
clang/inc
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 origin/main...HEAD clang/test/lit.cfg.py
``
:warning:
The reproduction instruction
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Jeaye Wilkerson (jeaye)
Changes
This is an updated version of @vgvassilev's PR from last year here:
https://github.com/llvm/llvm-project/pull/94166
In short, it includes:
1. The fix for a blocking issue where `clang::Interpreter`
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
https://github.com/jeaye created
https://github.com/llvm/llvm-project/pull/157359
This is an updated version of @vgvassilev's PR from last year here:
https://github.com/llvm/llvm-project/pull/94166
In short, it includes:
1. The fix for a blocking issue where `clang::Interpreter` cannot resolv
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Abhinav Kumar (kr-2003)
Changes
Introduce a custom lambda mechanism that allows injecting user-defined code
into the Remote JIT’s executor.
---
Full diff: https://github.com/llvm/llvm-project/pull/157358.diff
4 Files Affected:
- (modif
https://github.com/kr-2003 created
https://github.com/llvm/llvm-project/pull/157358
Introduce a custom lambda mechanism that allows injecting user-defined code
into the Remote JIT’s executor.
>From 4410aeb08dc14a4f29c9ec0e8730a1bde3386665 Mon Sep 17 00:00:00 2001
From: kr-2003
Date: Mon, 8 Se
https://github.com/carlosgalvezp requested changes to this pull request.
I notice now there's no update in Release Notes for this.
I also agree with previous reviewers that adding an option would be helpful to
ease in adopting to this new behavior. I would still make it on-by-default,
though.
https://github.com/carlosgalvezp approved this pull request.
LGTM, minor comments!
https://github.com/llvm/llvm-project/pull/157319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -10,6 +10,12 @@ template
void type_dependent_variables() {
T value = 42;
auto &ref = value;
+ // CHECK-MESSAGES:[[@LINE-1]]:3: warning: variable 'ref' of type 'int &' can
be declared 'const'
+ // CHECK-FIXES: auto const&ref = value;
+ //
+ // FIXME: This is a fals
@@ -10,6 +10,12 @@ template
void type_dependent_variables() {
T value = 42;
auto &ref = value;
+ // CHECK-MESSAGES:[[@LINE-1]]:3: warning: variable 'ref' of type 'int &' can
be declared 'const'
+ // CHECK-FIXES: auto const&ref = value;
carlosgalvezp wr
https://github.com/carlosgalvezp approved this pull request.
LGTM, thank you!
https://github.com/llvm/llvm-project/pull/157285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/carlosgalvezp approved this pull request.
LGTM, but I wonder if we need a 2-release cycle for this, after all people
might silently lose an important part of clang-tidy coverage and the note in
the release notes could be hard to spot in the middle of the other notes.
Perhaps
frasercrmck wrote:
I don't really mind what they're called. As I said in the original RFC and PR
that introduced the builtins, I have no particular favourites.
However, do note that the elementwise builtins are not *exactly* like clzg/ctzg
in that they don't have _target-specific_ zero-is-unde
vbvictor wrote:
I don't know how hard it is to achieve, but I'd like to have 2 distinct
options: one for "normal" local variables, another for lambdas
https://github.com/llvm/llvm-project/pull/157319
___
cfe-commits mailing list
cfe-commits@lists.llvm
vbvictor wrote:
I think Piotr's comment on the original review is still valid:
- Missing configuration option for this change (changing default behavior)
- Missing documentation update
- Tests for auto&&, auto* etc..
https://github.com/llvm/llvm-project/pull/157319
https://github.com/RKSimon updated
https://github.com/llvm/llvm-project/pull/156819
>From 1e8482846c5d623192bb5ce8e21dbc4eeb3fd591 Mon Sep 17 00:00:00 2001
From: SadiinsoSnowfall
Date: Thu, 4 Sep 2025 09:28:09 +0200
Subject: [PATCH] made the AVX512 _set macros into functions
---
clang/lib/Hea
1 - 100 of 313 matches
Mail list logo
