https://github.com/cor3ntin approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/121854
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/123335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2025-01-24T08:40:49+01:00
New Revision: 886adf8fb3ba5ec358a9a4fbe87e4da6b3ae8973
URL:
https://github.com/llvm/llvm-project/commit/886adf8fb3ba5ec358a9a4fbe87e4da6b3ae8973
DIFF:
https://github.com/llvm/llvm-project/commit/886adf8fb3ba5ec358a9a4fbe87e4da6b3ae8973.diff
L
HighCommander4 wrote:
(Fixed brace style)
https://github.com/llvm/llvm-project/pull/124236
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/124236
>From 27d53f571de25bf102fe3b27a82edc9605911b50 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Fri, 24 Jan 2025 01:56:48 -0500
Subject: [PATCH] [clang][Sema] Handle undeduced auto types in
HeuristicRes
HighCommander4 wrote:
> Thanks, though having to nest five levels of 'if's seems a bit awkward
I'll follow this up with a patch to implement the `FIXME`, it will almost
certainly involve factoring out a new helper function so the nesting should be
reduced.
https://github.com/llvm/llvm-project
https://github.com/zyn0217 approved this pull request.
Thanks, though having to nest five levels of 'if's seems a bit awkward
https://github.com/llvm/llvm-project/pull/124236
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
https://github.com/frasercrmck edited
https://github.com/llvm/llvm-project/pull/124145
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
Thanks for this patch.
Sorry I did not see this sooner.
I am not convinced this the best way to resolve #120875 - and I suspect in
practice it would be a bit difficult to use/be under used.
I think the general desire is for users to have a way (maybe a file) where they
can lis
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Nathan Ridge (HighCommander4)
Changes
Fixes https://github.com/clangd/clangd/issues/897
---
Full diff: https://github.com/llvm/llvm-project/pull/124236.diff
2 Files Affected:
- (modified) clang/lib/Sema/HeuristicResolver.cpp (+17-1)
-
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/124236
Fixes https://github.com/clangd/clangd/issues/897
>From 94c375f40fa7e13c6c13c039893242f516072c6d Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Fri, 24 Jan 2025 01:56:48 -0500
Subject: [PATCH] [clang]
frasercrmck wrote:
> How does using isfpclass avoid scalarization here? I think it's somewhat
> preferably to use the named operations here, they are subtly different since
> they canonicalize the input unlike is.fpclass
The builtins we were using before, like `__builtin_isnan`, don't take vec
https://github.com/alexrp edited
https://github.com/llvm/llvm-project/pull/119997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/alexrp edited
https://github.com/llvm/llvm-project/pull/100714
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -243,14 +244,16 @@ class AnnotatingParser {
// operator that was misinterpreted because we are parsing template
// parameters.
// FIXME: This is getting out of hand, write a decent parser.
- if (InExpr && !Line.startsWith(tok::kw_template) &&
+ if
alexrp wrote:
ping
https://github.com/llvm/llvm-project/pull/119997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca closed
https://github.com/llvm/llvm-project/pull/124085
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Owen Pan
Date: 2025-01-23T21:44:29-08:00
New Revision: 6330f1e052dbec842e0d12e3d0cc44996e8663d2
URL:
https://github.com/llvm/llvm-project/commit/6330f1e052dbec842e0d12e3d0cc44996e8663d2
DIFF:
https://github.com/llvm/llvm-project/commit/6330f1e052dbec842e0d12e3d0cc44996e8663d2.diff
LOG:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
Changes
Note that PointerUnion::dyn_cast has been soft deprecated in
PointerUnion.h:
// FIXME: Replace the uses of is(), get() and dyn_cast() with
//isa, cast and the llvm::dyn_cast
Litera
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/124229
Note that PointerUnion::dyn_cast has been soft deprecated in
PointerUnion.h:
// FIXME: Replace the uses of is(), get() and dyn_cast() with
//isa, cast and the llvm::dyn_cast
Literal migration
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
Changes
Note that PointerUnion::dyn_cast has been soft deprecated in
PointerUnion.h:
// FIXME: Replace the uses of is(), get() and dyn_cast() with
//isa, cast and the llvm::dyn_cast
Litera
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/124228
Note that PointerUnion::dyn_cast has been soft deprecated in
PointerUnion.h:
// FIXME: Replace the uses of is(), get() and dyn_cast() with
//isa, cast and the llvm::dyn_cast
Literal migration
wzssyqa wrote:
> Do you get the same behavior when you use `-isystem` instead?
There is no warning then if I use command
```
clang -E -I build-x86/ -I sysdeps/x86 -I. -isystem ./include -I
sysdeps/unix/sysv/linux/ -I sysdeps/generic/ -include include/libc-symbols.h
-O2 -H xx.c
```
instead of
https://github.com/shafik edited
https://github.com/llvm/llvm-project/pull/121417
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shafik commented:
Thank you for the PR, small comments.
https://github.com/llvm/llvm-project/pull/121417
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,11 @@
+// RUN: %clang_cc1 -std=c++2c -verify -fsyntax-only %s
+
+template
+void decompose_array() {
+ int arr[4] = {1, 2, 3, 5};
+ auto [x, ... // #1
+rest, ...more_rest] = arr; // expected-error{{multiple packs in structured
binding declaration}}
+
@@ -0,0 +1,117 @@
+// RUN: %clang_cc1 -fsyntax-only -std=c++26 %s -verify
shafik wrote:
+1 we should always cover all the examples from the proposal.
https://github.com/llvm/llvm-project/pull/121417
___
cfe-commits mai
https://github.com/brad0 closed https://github.com/llvm/llvm-project/pull/65827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Lang Hames
Date: 2025-01-24T15:31:31+11:00
New Revision: fd174f0ff3e793fe96a6663b1488ed159cfe042f
URL:
https://github.com/llvm/llvm-project/commit/fd174f0ff3e793fe96a6663b1488ed159cfe042f
DIFF:
https://github.com/llvm/llvm-project/commit/fd174f0ff3e793fe96a6663b1488ed159cfe042f.diff
LO
@@ -253,12 +257,229 @@ static void validatePackoffset(Sema &S, HLSLBufferDecl
*BufDecl) {
}
}
+// Returns true if the array has a zero size = if any of the dimensions is 0
+static bool isZeroSizedArray(const ConstantArrayType *CAT) {
+ while (CAT && !CAT->isZeroSize())
+
Author: Lang Hames
Date: 2025-01-24T14:59:10+11:00
New Revision: c9bc242e387f4a4a3dfcd86561f3ec0ca8a72d62
URL:
https://github.com/llvm/llvm-project/commit/c9bc242e387f4a4a3dfcd86561f3ec0ca8a72d62
DIFF:
https://github.com/llvm/llvm-project/commit/c9bc242e387f4a4a3dfcd86561f3ec0ca8a72d62.diff
LO
https://github.com/Flandini edited
https://github.com/llvm/llvm-project/pull/123003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -437,7 +437,7 @@ bool SymbolReaper::isLiveRegion(const MemRegion *MR) {
// tell if anything still refers to this region. Unlike SymbolicRegions,
// AllocaRegions don't have associated symbols, though, so we don't actually
// have a way to track their liveness.
- retur
Author: Chuanqi Xu
Date: 2025-01-24T10:51:10+08:00
New Revision: 378dcf61014b787b3542b917f6296c9fb5ec490c
URL:
https://github.com/llvm/llvm-project/commit/378dcf61014b787b3542b917f6296c9fb5ec490c
DIFF:
https://github.com/llvm/llvm-project/commit/378dcf61014b787b3542b917f6296c9fb5ec490c.diff
LO
https://github.com/ChuanqiXu9 closed
https://github.com/llvm/llvm-project/pull/123931
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ChuanqiXu9 wrote:
I'd like to land this given the scale of the patch and I want to fix it before
20.x and I may not be available in the next few days.
https://github.com/llvm/llvm-project/pull/123931
___
cfe-commits mailing list
cfe-commits@lists.llv
Flandini wrote:
Working on some benchmarking. Posted some problems in the discord channel, but
working through it now, and benchmarking with the CodeChecker tool using only
clangsa. Is there an already established set of benchmarks y'all use somewhere?
IIRC, there was some github repo with som
Flandini wrote:
> Have you considered changing `MemRegion::getMemorySpace()` into
> `MemRegion::getMemorySpace(ProgramStateRef)`?
I thought about this, but I decided against it since I am thinking that
MemSpaces will eventually be their own separate thing, not part of the
MemRegion hierarchy,
https://github.com/shafik commented:
I feel like matching gcc's behavior makes sense for users but I am not sure
about the timing for clang-20. This seems like the change in behavior could be
impactful.
https://github.com/llvm/llvm-project/pull/122486
__
Author: Nathan Ridge
Date: 2025-01-23T20:21:11-05:00
New Revision: 00132643744237d2398fea1777114f3497629bd5
URL:
https://github.com/llvm/llvm-project/commit/00132643744237d2398fea1777114f3497629bd5
DIFF:
https://github.com/llvm/llvm-project/commit/00132643744237d2398fea1777114f3497629bd5.diff
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/124080
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5376,12 +5376,12 @@ let Features = "avx10.2-512", Attributes = [NoThrow,
Const, RequiredVectorWidth<
}
let Features = "avx10.2-256", Attributes = [NoThrow, Const,
RequiredVectorWidth<128>] in {
- def vcomsbf16eq : X86Builtin<"int(_Vector<8, __bf16>, _Vector<8, __bf16>)"
@@ -0,0 +1,785 @@
+//===-- Mustache.cpp
--===//
+//
+// 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/Artem-B approved this pull request.
https://github.com/llvm/llvm-project/pull/124080
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -707,6 +707,7 @@ ExprResult Sema::ImpCastExprToType(Expr *E, QualType Ty,
case CK_ToVoid:
case CK_NonAtomicToAtomic:
case CK_HLSLArrayRValue:
+case CK_HLSLAggregateCast:
llvm-beanz wrote:
Nope, this cast should not ever be implicit.
https:/
@@ -1461,6 +1552,7 @@ static bool castPreservesZero(const CastExpr *CE) {
case CK_NonAtomicToAtomic:
case CK_AtomicToNonAtomic:
case CK_HLSLVectorTruncation:
+ case CK_HLSLAggregateCast: // TODO does CK_HLSLAggregateCast preserve zero?
llvm-beanz wrote:
Author: Phoebe Wang
Date: 2025-01-24T08:37:29+08:00
New Revision: 24f177df61f673804a612dc48279c517bdecd696
URL:
https://github.com/llvm/llvm-project/commit/24f177df61f673804a612dc48279c517bdecd696
DIFF:
https://github.com/llvm/llvm-project/commit/24f177df61f673804a612dc48279c517bdecd696.diff
L
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/123307
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4213,8 +4226,35 @@ class DecompositionDecl final
static DecompositionDecl *CreateDeserialized(ASTContext &C, GlobalDeclID ID,
unsigned NumBindings);
- ArrayRef bindings() const {
-return llvm::ArrayRef(getTrailingObject
PeterChou1 wrote:
i've added a test to address this
https://github.com/llvm/llvm-project/pull/105893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/lenary closed
https://github.com/llvm/llvm-project/pull/124174
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Sam Elliott
Date: 2025-01-23T16:19:19-08:00
New Revision: 33c44074714d1d2f3d5f65c3fb842cddb6b689ac
URL:
https://github.com/llvm/llvm-project/commit/33c44074714d1d2f3d5f65c3fb842cddb6b689ac
DIFF:
https://github.com/llvm/llvm-project/commit/33c44074714d1d2f3d5f65c3fb842cddb6b689ac.diff
L
@@ -951,28 +959,124 @@ Sema::ActOnDecompositionDeclarator(Scope *S, Declarator
&D,
return New;
}
+// CheckBindingsCount
+// - Checks the arity of the structured bindings
+// - Creates the resolved pack expr if there is
+//one
+static bool CheckBindingsCount(Sema &S, D
@@ -951,28 +959,124 @@ Sema::ActOnDecompositionDeclarator(Scope *S, Declarator
&D,
return New;
}
+// CheckBindingsCount
zygoloid wrote:
We [don't start function comments with the function
name](https://llvm.org/docs/CodingStandards.html#commenting:~:text=
@@ -2316,6 +2316,20 @@ void
ASTDeclReader::VisitCXXConstructorDecl(CXXConstructorDecl *D) {
}
VisitCXXMethodDecl(D);
+
+ // Microsoft CXX ABI specific:
+ // Restore the RecordToCopyCtor sidecar LUT entry so that `throw` expressions
+ // find the correct copy constructo
https://github.com/arsenm commented:
How does using isfpclass avoid scalarization here? I think it's somewhat
preferably to use the named operations here, they are subtly different since
they canonicalize the input unlike is.fpclass
https://github.com/llvm/llvm-project/pull/124145
___
@@ -0,0 +1,114 @@
+//===--- Mustache.h -*- 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: Ap
@@ -0,0 +1,134 @@
+//=== ParseHLSLRootSignatureTest.cpp - Parse Root Signature tests
-===//
+//
+// 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/thurstond edited
https://github.com/llvm/llvm-project/pull/124211
___
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: Thurston Dang (thurstond)
Changes
This passes through the values of -fsanitize-skip-hot-cutoff (introduced patch
in https://github.com/llvm/llvm-project/pull/121619) to the
LowerAllowCheckPass, via the Options parameter (introduced in
ht
https://github.com/thurstond created
https://github.com/llvm/llvm-project/pull/124211
This passes through the values of -fsanitize-skip-hot-cutoff (introduced patch
in https://github.com/llvm/llvm-project/pull/121619) to the
LowerAllowCheckPass, via the Options parameter (introduced in
https:
@@ -0,0 +1,134 @@
+//=== ParseHLSLRootSignatureTest.cpp - Parse Root Signature tests
-===//
+//
+// 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/mizvekov closed
https://github.com/llvm/llvm-project/pull/124137
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -822,6 +822,7 @@ def NSobjectAttribute : DiagGroup<"NSObject-attribute">;
def NSConsumedMismatch : DiagGroup<"nsconsumed-mismatch">;
def NSReturnsMismatch : DiagGroup<"nsreturns-mismatch">;
+def UnknownAttributeNamespaces : DiagGroup<"unknown-attribute-namespaces">;
def In
aheejin wrote:
@llvm-ci The failure seems irrelevant.
https://github.com/llvm/llvm-project/pull/124042
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/120925
>From bce88b1bb464438828fc177c978ad2b99957530f Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Mon, 23 Dec 2024 02:35:07 +0200
Subject: [PATCH 1/9] [Clang] raise extension warning for unknown namespaced
att
@@ -0,0 +1,785 @@
+//===-- Mustache.cpp
--===//
+//
+// 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/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/120925
>From bce88b1bb464438828fc177c978ad2b99957530f Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Mon, 23 Dec 2024 02:35:07 +0200
Subject: [PATCH 1/8] [Clang] raise extension warning for unknown namespaced
att
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-aarch64-quick`
running on `linaro-clang-aarch64-quick` while building `clang` at step 5 "ninja
check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/65/builds/11149
Here is the relevant pie
cor3ntin wrote:
> I do not have a strong preference, but I kept `bindings()` the same since it
> represents the structure of the AST and that is how it is used in most
> places. (like 2x time more than `flat_bindings()`)
Lets not then
I'm inclined to say we should merge that once @zygoloid an
https://github.com/aheejin closed
https://github.com/llvm/llvm-project/pull/124042
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Heejin Ahn
Date: 2025-01-23T14:47:15-08:00
New Revision: 25825d4ac9683010b0b921924cd5e244a82db1a3
URL:
https://github.com/llvm/llvm-project/commit/25825d4ac9683010b0b921924cd5e244a82db1a3
DIFF:
https://github.com/llvm/llvm-project/commit/25825d4ac9683010b0b921924cd5e244a82db1a3.diff
LO
https://github.com/lenary approved this pull request.
Two tiny issues, but I think I'm happy for this to land.
https://github.com/llvm/llvm-project/pull/121394
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
lenary wrote:
You still have a whitespace change to this file?
https://github.com/llvm/llvm-project/pull/121394
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
@@ -238,3 +247,11 @@ void
RISCVSubtarget::overridePostRASchedPolicy(MachineSchedPolicy &Policy,
Policy.OnlyBottomUp = false;
}
}
+
+bool RISCVSubtarget::useLoadStorePairs() const {
+ return UseMIPSLoadStorePairsOpt && HasVendorXMIPSLSP;
+}
lenary wrote:
https://github.com/lenary edited
https://github.com/llvm/llvm-project/pull/121394
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,133 @@
+//=== ParseHLSLRootSignatureTest.cpp - Parse Root Signature tests
-===//
+//
+// 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
llvm-beanz wrote:
As I said in your other PR, I believe not including error reporting and the
appropriate testing makes it significantly more difficult to review the
subsequent commit since not all the appropriate code will even be in the PR.
https://github.c
@@ -0,0 +1,96 @@
+//===--- ParseHLSLRootSignature.h ---*- 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
https://github.com/glebov-andrey edited
https://github.com/llvm/llvm-project/pull/114075
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2316,6 +2316,20 @@ void
ASTDeclReader::VisitCXXConstructorDecl(CXXConstructorDecl *D) {
}
VisitCXXMethodDecl(D);
+
+ // Microsoft CXX ABI specific:
+ // Restore the RecordToCopyCtor sidecar LUT entry so that `throw` expressions
+ // find the correct copy constructo
@@ -0,0 +1,47 @@
+.. title:: clang-tidy - bugprone-smartptr-reset-ambiguous-call
+
+bugprone-smartptr-reset-ambiguous-call
+==
+
+Finds potentially erroneous calls to ``reset`` method on
+smart pointers when the pointee type also has a ``reset``
https://github.com/vbvictor updated
https://github.com/llvm/llvm-project/pull/121291
>From 37dce6a7ed0cca2e9819c24f4d176c43e3c9f2ac Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Sun, 29 Dec 2024 15:32:22 +0300
Subject: [PATCH 1/6] [clang-tidy] Add bugprone-reset-call check
---
.../bugpr
shafik wrote:
> > I see a lot of diagnostics were updated to `DefaultError` but I don't see
> > matching tests that demonstrate the now error. We should be testing each
> > diagnostic.
>
> #123464 also added `-Werror=return-type` to some tests; we can remove those
> now, which should cover at
@@ -2316,6 +2316,20 @@ void
ASTDeclReader::VisitCXXConstructorDecl(CXXConstructorDecl *D) {
}
VisitCXXMethodDecl(D);
+
+ // Microsoft CXX ABI specific:
+ // Restore the RecordToCopyCtor sidecar LUT entry so that `throw` expressions
+ // find the correct copy constructo
@@ -13464,6 +13464,14 @@ Decl *Sema::ActOnAliasDeclaration(Scope *S,
AccessSpecifier AS,
}
TemplateParameterList *TemplateParams = TemplateParamLists[0];
+// Check shadowing of a template parameter name
+for (NamedDecl *TP : TemplateParams->asArray()) {
+
https://github.com/dipeshs809 updated
https://github.com/llvm/llvm-project/pull/117149
>From 8d1758e9ddecd2ede29a3fb117404f03f6bdc1fb Mon Sep 17 00:00:00 2001
From: dipeshs809
Date: Thu, 21 Nov 2024 17:48:22 +0530
Subject: [PATCH] [clang] guard __STDC_NO_THREADS__ defination for MSVCVersion
20
GeorgeKA wrote:
Hi @Maetveis . This is ready for review.
There are a few things that I came across while making the script that I should
highlight :
1) clang -cc1 test directory
[test/Driver/lit.local.cfg](https://github.com/llvm/llvm-project/blob/e10d551aa482ee185a80216b2670a2947a8bdeb0/clan
https://github.com/hekota edited
https://github.com/llvm/llvm-project/pull/122820
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -253,12 +257,229 @@ static void validatePackoffset(Sema &S, HLSLBufferDecl
*BufDecl) {
}
}
+// Returns true if the array has a zero size = if any of the dimensions is 0
+static bool isZeroSizedArray(const ConstantArrayType *CAT) {
+ while (CAT && !CAT->isZeroSize())
+
Sirraide wrote:
> (to me) it's not really defensible in 2025 that falling off the end of a
> function with a non-void return type is anything but an error.
Yeah, agreed.
> Specifically, I wonder about `autoconf`
I hadn’t thought about that, that’s a good point.
> Is that something you've loo
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `flang-aarch64-dylib`
running on `linaro-flang-aarch64-dylib` while building `clang,llvm,utils` at
step 5 "build-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/50/builds/9302
Here is t
https://github.com/GeorgeKA ready_for_review
https://github.com/llvm/llvm-project/pull/120900
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -253,12 +257,229 @@ static void validatePackoffset(Sema &S, HLSLBufferDecl
*BufDecl) {
}
}
+// Returns true if the array has a zero size = if any of the dimensions is 0
+static bool isZeroSizedArray(const ConstantArrayType *CAT) {
+ while (CAT && !CAT->isZeroSize())
+
@@ -253,12 +257,229 @@ static void validatePackoffset(Sema &S, HLSLBufferDecl
*BufDecl) {
}
}
+// Returns true if the array has a zero size = if any of the dimensions is 0
+static bool isZeroSizedArray(const ConstantArrayType *CAT) {
+ while (CAT && !CAT->isZeroSize())
+
https://github.com/GeorgeKA edited
https://github.com/llvm/llvm-project/pull/120900
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -822,6 +822,7 @@ def NSobjectAttribute : DiagGroup<"NSObject-attribute">;
def NSConsumedMismatch : DiagGroup<"nsconsumed-mismatch">;
def NSReturnsMismatch : DiagGroup<"nsreturns-mismatch">;
+def UnknownAttributeNamespaces : DiagGroup<"unknown-attribute-namespaces">;
def In
@@ -606,6 +606,8 @@ Attribute Changes in Clang
- Clang now disallows the use of attributes after the namespace name.
(#GH121407)
+- Clang now raises warnings for unknown namespaced attributes only in pedantic
mode (#GH120875).
AaronBallman wrote:
This is i
pogo59 wrote:
Aha. I didn't know or had forgotten about the exception. The breakage isn't
great but is allowed.
https://github.com/llvm/llvm-project/pull/122928
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
https://github.com/a-tarasyuk closed
https://github.com/llvm/llvm-project/pull/106036
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 302 matches
Mail list logo
