@@ -0,0 +1,16 @@
+// RUN: %clang_cc1 %s -fsyntax-only -verify
+// expected-no-diagnostics
+
+// This example uncovered a bug in Sema::BuiltinVectorMath, where we should be
+// using ASTContext::hasSameUnqualifiedType().
+
+typedef float vec3 __attribute__((ext_vector_type(3)));
+
@@ -388,6 +388,76 @@ bool LoongArchTargetInfo::handleTargetFeatures(
return true;
}
+enum class AttrFeatureKind { Arch, Tune, NoFeature, Feature, Invalid };
wangleiat wrote:
Is Invalid unused?
https://github.com/llvm/llvm-project/pull/140700
__
https://github.com/zyn0217 approved this pull request.
Thanks!
https://github.com/llvm/llvm-project/pull/141890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/marcogmaia edited
https://github.com/llvm/llvm-project/pull/139348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Owen Pan
Date: 2025-05-28T19:10:20-07:00
New Revision: 7e1a88b9d1431e263258e3ff0f729c1fdce342d3
URL:
https://github.com/llvm/llvm-project/commit/7e1a88b9d1431e263258e3ff0f729c1fdce342d3
DIFF:
https://github.com/llvm/llvm-project/commit/7e1a88b9d1431e263258e3ff0f729c1fdce342d3.diff
LOG:
https://github.com/owenca closed
https://github.com/llvm/llvm-project/pull/141714
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
marcogmaia wrote:
@kadircet I've added a description about the tweak at the beginning of the file
as requested.
https://github.com/llvm/llvm-project/pull/139348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
Author: Trung Nguyen
Date: 2025-05-28T22:18:55-04:00
New Revision: 1d9ef8211f399a57849d0bc47cb86bf594bac364
URL:
https://github.com/llvm/llvm-project/commit/1d9ef8211f399a57849d0bc47cb86bf594bac364
DIFF:
https://github.com/llvm/llvm-project/commit/1d9ef8211f399a57849d0bc47cb86bf594bac364.diff
https://github.com/brad0 closed https://github.com/llvm/llvm-project/pull/135367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sarnex wrote:
It took me all day but I managed to reproduce the issue with Chromium,
investigating more.
https://github.com/llvm/llvm-project/pull/138205
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
https://github.com/ChuanqiXu9 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangleiat edited
https://github.com/llvm/llvm-project/pull/140700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wangleiat approved this pull request.
LGTM with some minor nits, thanks.
https://github.com/llvm/llvm-project/pull/140700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
@@ -34,6 +34,19 @@ bool LoongArch::isValidArchName(StringRef Arch) {
return false;
}
+bool LoongArch::isValidFeatureName(StringRef Feature) {
+ if (Feature.starts_with("+") || Feature.starts_with("-")) {
+return false;
+ }
+ for (const auto F : AllFeatures) {
+Str
https://github.com/QiYueFeiXue updated
https://github.com/llvm/llvm-project/pull/141485
>From 4566e35db95001270dc84fb854fae8081c1e13a4 Mon Sep 17 00:00:00 2001
From: QiYue
Date: Mon, 26 May 2025 19:51:24 +0800
Subject: [PATCH 1/2] [Sema] built-in args type checking using
hasSameUnqualifiedType
Author: Matheus Izvekov
Date: 2025-05-29T02:08:11-03:00
New Revision: 61314076f78327ffd5e1463c35373c7f4e52d30f
URL:
https://github.com/llvm/llvm-project/commit/61314076f78327ffd5e1463c35373c7f4e52d30f
DIFF:
https://github.com/llvm/llvm-project/commit/61314076f78327ffd5e1463c35373c7f4e52d30f.dif
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
@@ -1741,57 +1741,65 @@ Sema::ActOnFinishSwitchStmt(SourceLocation SwitchLoc,
Stmt *Switch,
void
Sema::DiagnoseAssignmentEnum(QualType DstType, QualType SrcType,
Expr *SrcExpr) {
+
+ const auto *E
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/shafik commented:
Very nice change.
How are you coming up w/ the target for your improvements? Is this something
other folks could try and try and find some inspiration?
https://github.com/llvm/llvm-project/pull/141471
_
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/shafik edited
https://github.com/llvm/llvm-project/pull/141471
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Ami-zhang updated
https://github.com/llvm/llvm-project/pull/140700
>From 1f521b6105a032d4722041d179e96cee28ab4169 Mon Sep 17 00:00:00 2001
From: Ami-zhang
Date: Wed, 14 May 2025 15:21:54 +0800
Subject: [PATCH 1/2] [Clang][LoongArch] Support target attribute for function
This
@@ -34,6 +34,19 @@ bool LoongArch::isValidArchName(StringRef Arch) {
return false;
}
+bool LoongArch::isValidFeatureName(StringRef Feature) {
+ if (Feature.starts_with("+") || Feature.starts_with("-")) {
+return false;
+ }
wangleiat wrote:
The braces
@@ -0,0 +1,16 @@
+// RUN: %clang_cc1 %s -fsyntax-only -verify
+// expected-no-diagnostics
+
+// This example uncovered a bug in Sema::BuiltinVectorMath, where we should be
+// using ASTContext::hasSameUnqualifiedType().
+
+typedef float vec3 __attribute__((ext_vector_type(3)));
+
fanju110 wrote:
> @fanju110, Thanks for seeing this through!
Hi @tarunprabhu , If everything looks good, could you please merge this PR at
your convenience (I don't have merge rights)? Thank you!
https://github.com/llvm/llvm-project/pull/136098
__
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-x86_64-debian`
running on `lldb-x86_64-debian` while building `clang` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/162/builds/23407
Here is the relevant piece of the build lo
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/141890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/141890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov closed
https://github.com/llvm/llvm-project/pull/141890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/QiYueFeiXue updated
https://github.com/llvm/llvm-project/pull/141485
>From 8a579b19136dc2754f530197f3b45d4c993556fc Mon Sep 17 00:00:00 2001
From: QiYue
Date: Mon, 26 May 2025 19:51:24 +0800
Subject: [PATCH 1/3] [Sema] built-in args type checking using
hasSameUnqualifiedType
zyn0217 wrote:
@QiYueFeiXue I don't think @shafik means to directly copy his words to the PR
body/title. Please at least flesh out these dotted parts and make the sentences
complete, thanks.
https://github.com/llvm/llvm-project/pull/141485
___
cfe-co
https://github.com/wangleiat edited
https://github.com/llvm/llvm-project/pull/140700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zyn0217 wrote:
/cherry-pick 61314076f
https://github.com/llvm/llvm-project/pull/141890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
Failed to cherry-pick: 61314076f
https://github.com/llvm/llvm-project/actions/runs/15317560723
Please manually backport the fix and push it to your github fork. Once this is
done, please create a [pull
request](https://github.com/llvm/llvm-project/compare)
https://github.com
@@ -0,0 +1,4 @@
+// RUN: not clang-doc %S/Inputs/basic-project/src/Circle.cpp
-output=/root/docs 2>&1 | FileCheck %s
ilovepi wrote:
Well, I had forgotten that `create_directories()` works differently than
`create_directory()`, in that its basically `mkdir -p`.
https://github.com/QiYueFeiXue edited
https://github.com/llvm/llvm-project/pull/141485
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,112 @@
+//===--- InvalidEnumDefaultInitializationCheck.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
Author: Orlando Cazalet-Hyams
Date: 2025-05-28T11:25:41+01:00
New Revision: c4d0d95a4fb92d65594f3575814a027815b5182f
URL:
https://github.com/llvm/llvm-project/commit/c4d0d95a4fb92d65594f3575814a027815b5182f
DIFF:
https://github.com/llvm/llvm-project/commit/c4d0d95a4fb92d65594f3575814a027815b518
https://github.com/OCHyams closed
https://github.com/llvm/llvm-project/pull/141607
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/140890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Aaron Ballman
Date: 2025-05-28T06:41:01-04:00
New Revision: 5ab944a8c6a213beb96f3747a441b02e497732e4
URL:
https://github.com/llvm/llvm-project/commit/5ab944a8c6a213beb96f3747a441b02e497732e4
DIFF:
https://github.com/llvm/llvm-project/commit/5ab944a8c6a213beb96f3747a441b02e497732e4.diff
https://github.com/OCHyams updated
https://github.com/llvm/llvm-project/pull/134651
>From bf3a37df3a82797227e476485f6cf4c0c9a7c912 Mon Sep 17 00:00:00 2001
From: Orlando Cazalet-Hyams
Date: Fri, 4 Apr 2025 14:36:43 +0100
Subject: [PATCH 1/5] [KeyInstr][Clang] Store-like builtin atoms
---
clan
@@ -4466,7 +4473,8 @@ RValue CodeGenFunction::EmitBuiltinExpr(const GlobalDecl
GD, unsigned BuiltinID,
Value *SizeVal = EmitScalarExpr(E->getArg(2));
EmitNonNullArgCheck(Dest, E->getArg(0)->getType(),
E->getArg(0)->getExprLoc(), FD, 0);
-Bui
https://github.com/SLTozer approved this pull request.
https://github.com/llvm/llvm-project/pull/141607
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kiranchandramohan wrote:
Does `CCC_OVERRIDE_OPTIONS` expands to Clang Compiler Commandline Override
Options? If so `FCC_OVERRIDE_OPTIONS` expanding to Fortran Compiler Commandline
Override Options seems the right replacement.
If `CCC_OVERRIDE_OPTIONS` expands to Clang C Compiler Override Optio
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/141741
032ad59 taught the instantiator to expand template argument packs for rewrite.
However we might already be in a pack expansion when we synthesizing the CTAD
guide, so we reset the ArgPackSubstIndex to ensure it
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Younan Zhang (zyn0217)
Changes
032ad59 taught the instantiator to expand template argument packs for rewrite.
However we might already be in a pack expansion when we synthesizing the CTAD
guide, so we reset the ArgPackSubstIndex to ensure
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/141741
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
antoniofrighetto wrote:
> Sorry, can you back up and explain the root problem you're trying to solve?
> You have a platform where we're emitting accesses to `errno` as if it weren't
> an `int`? What does it look like in source?
Oh, sorry for not providing context earlier. The reasoning behind
https://github.com/jthackray created
https://github.com/llvm/llvm-project/pull/141743
None
>From a76663bd47510c46b7c4415d4b5d97642ce69967 Mon Sep 17 00:00:00 2001
From: Jonathan Thackray
Date: Wed, 28 May 2025 11:47:00 +0100
Subject: [PATCH] [NFC][AArch64] Add relnote saying modal FP8 intrinsi
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
The constructor starts the lifetime of all the subobjects.
---
Full diff: https://github.com/llvm/llvm-project/pull/141742.diff
3 Files Affected:
- (modified) clang/lib/AST/ByteCode/DynamicAllocator.cpp (+
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/141742
The constructor starts the lifetime of all the subobjects.
>From 11f8d2e4c3282a9b49f9cf08b7d16f6c32e08134 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 28 May 2025 11:27:39 +0200
Subj
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jonathan Thackray (jthackray)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/141743.diff
1 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+4)
``diff
diff --git a/clang/docs/ReleaseNotes.rst b/cla
https://github.com/xlauko edited
https://github.com/llvm/llvm-project/pull/141700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/OCHyams updated
https://github.com/llvm/llvm-project/pull/134646
>From 0c7a83007e218dd7338b92f1d50319b0f35cf55f Mon Sep 17 00:00:00 2001
From: Orlando Cazalet-Hyams
Date: Thu, 3 Apr 2025 19:12:47 +0100
Subject: [PATCH 1/7] [KeyInstr][Clang] For stmt atom
This patch is part o
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/141633
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kiranchandramohan wrote:
> > Thank you for seeing this through and making all the little changes. I have
> > requested reviews from @MaskRay and @aeubanks for the clang side of things.
>
> hello,I noticed that this PR has been awaiting clang review for three weeks.
> I still haven't gotten a c
https://github.com/heiher approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141703
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -388,6 +388,76 @@ bool LoongArchTargetInfo::handleTargetFeatures(
return true;
}
+enum class AttrFeatureKind { Arch, Tune, NoFeature, Feature, Invalid };
+
+static std::pair
+getAttrFeatureTypeAndValue(llvm::StringRef AttrFeature) {
+ if (auto Split = AttrFeature.split("
https://github.com/Ami-zhang edited
https://github.com/llvm/llvm-project/pull/140700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Ami-zhang edited
https://github.com/llvm/llvm-project/pull/140700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Ami-zhang updated
https://github.com/llvm/llvm-project/pull/140700
>From a9f8fa53dd260f8161a2762c5ec118722cbf22a1 Mon Sep 17 00:00:00 2001
From: Ami-zhang
Date: Wed, 14 May 2025 15:21:54 +0800
Subject: [PATCH 1/2] [Clang][LoongArch] Support target attribute for function
This
eZWALT wrote:
> > AnalyzeLoopSequence
>
> Could you point, where this is located?
Of course, [See line 14315 in
`SemaOpenMP.cpp`](https://github.com/eZWALT/llvm-project/blob/main/clang/lib/Sema/SemaOpenMP.cpp#L14315)
https://github.com/llvm/llvm-project/pull/140532
__
eZWALT wrote:
> > AnalyzeLoopSequence
>
> Could you point, where this is located?
Although now that i think about it, this function is not inside this PR but
rather on the PR #139293, but the dependency is clear.
https://github.com/llvm/llvm-project/pull/140532
__
https://github.com/frasercrmck approved this pull request.
LGTM, thanks
https://github.com/llvm/llvm-project/pull/141721
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
frasercrmck wrote:
> > That said, I don't believe it "works" in the way it's supposed to. It still
> > grabs the host tools using `get_host_tool_path` in CMake, and custom
> > commands to build with that. I take it we're supposed to use
> > `CMAKE_C_COMPILER` as if we were a regular CMake proj
https://github.com/frasercrmck updated
https://github.com/llvm/llvm-project/pull/141574
>From f5278d261d203cea8889174c8b0a16c03d1cbad9 Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Tue, 27 May 2025 11:11:14 +0100
Subject: [PATCH 1/2] [libclc] Support LLVM_ENABLE_RUNTIMES when building
Th
@@ -3195,6 +3195,17 @@ bool Sema::checkTargetAttr(SourceLocation LiteralLoc,
StringRef AttrStr) {
}
}
+ if (Context.getTargetInfo().getTriple().isLoongArch()) {
+for (const auto &Feature : ParsedAttrs.Features) {
+ StringRef CurFeature = Feature;
+ if (Cu
https://github.com/Rajveer100 updated
https://github.com/llvm/llvm-project/pull/139859
>From 2fc6e7ccab49888b60bbb43e0e6350555c5eb9f1 Mon Sep 17 00:00:00 2001
From: Rajveer
Date: Wed, 14 May 2025 13:44:31 +0530
Subject: [PATCH] [clang][Sema] Diagnose exceptions only in non-dependent
context in
https://github.com/Keenuts approved this pull request.
https://github.com/llvm/llvm-project/pull/137805
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zyn0217 wrote:
@cor3ntin can we merge it? building clang with ToT clang results in a lot of
warnings now.
https://github.com/llvm/llvm-project/pull/141091
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
Author: David Green
Date: 2025-05-28T12:26:54+01:00
New Revision: 3a42cbd47d3e92b8794378d2a0e8ec7ae81950d7
URL:
https://github.com/llvm/llvm-project/commit/3a42cbd47d3e92b8794378d2a0e8ec7ae81950d7
DIFF:
https://github.com/llvm/llvm-project/commit/3a42cbd47d3e92b8794378d2a0e8ec7ae81950d7.diff
L
Rajveer100 wrote:
Done.
https://github.com/llvm/llvm-project/pull/139859
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/CarolineConcatto approved this pull request.
LGTM!
Thank you Jonathan.
https://github.com/llvm/llvm-project/pull/141743
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
AaronBallman wrote:
> If you feel that there's a technical reason that the compiler output is no
> longer reasonable, you are welcome to bring it up as a technical argument. My
> point is simply that "my transform stopped working because clang's compiler
> output changed" is not, in and of its
https://github.com/davemgreen updated
https://github.com/llvm/llvm-project/pull/126945
>From 5a17166859760dcbc258892be46f7f909c2b00a9 Mon Sep 17 00:00:00 2001
From: Tomas Matheson
Date: Wed, 12 Feb 2025 14:31:47 +
Subject: [PATCH 1/3] Add missing Neon Types
The AAPCS64 adds a number of vec
davemgreen wrote:
> LGTM with one nitpick. I've run this through a fuzzer which tests ABI
> compatibility with GCC and didn't find any problems.
Thanks for the review and the extra testing!
https://github.com/llvm/llvm-project/pull/126945
___
cfe-com
Author: Carlos Alberto Enciso
Date: 2025-05-28T09:15:48+01:00
New Revision: d1b0cbff806b50d399826e79b9a53e4726c21302
URL:
https://github.com/llvm/llvm-project/commit/d1b0cbff806b50d399826e79b9a53e4726c21302
DIFF:
https://github.com/llvm/llvm-project/commit/d1b0cbff806b50d399826e79b9a53e4726c213
https://github.com/CarlosAlbertoEnciso closed
https://github.com/llvm/llvm-project/pull/130255
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
CarlosAlbertoEnciso wrote:
@dwblaikie @labath @pogo59 @jmorse @Michael137 @tromey Many thanks for all your
valuable feedback.
https://github.com/llvm/llvm-project/pull/130255
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
https://github.com/labath approved this pull request.
Looks good. Thanks for the fix.
https://github.com/llvm/llvm-project/pull/141478
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1044,6 +1057,24 @@ StringRef CIRGenModule::getMangledName(GlobalDecl gd) {
return mangledDeclNames[canonicalGd] = result.first->first();
}
+void CIRGenModule::emitTentativeDefinition(const VarDecl *d) {
+ assert(!d->getInit() && "Cannot emit definite definitions here!")
@@ -135,6 +135,10 @@ class CIRGenConsumer : public clang::ASTConsumer {
}
}
}
+
+ void CompleteTentativeDefinition(VarDecl *D) override {
+Gen->CompleteTentativeDefinition(D);
+ }
xlauko wrote:
```suggestion
void CompleteTentativeDefinition(V
https://github.com/xlauko approved this pull request.
lgtm, with minor nits
https://github.com/llvm/llvm-project/pull/141700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -388,6 +388,76 @@ bool LoongArchTargetInfo::handleTargetFeatures(
return true;
}
+enum class AttrFeatureKind { Arch, Tune, NoFeature, Feature, Invalid };
+
+static std::pair
+getAttrFeatureTypeAndValue(llvm::StringRef AttrFeature) {
+ if (auto Split = AttrFeature.split("
@@ -3195,6 +3195,17 @@ bool Sema::checkTargetAttr(SourceLocation LiteralLoc,
StringRef AttrStr) {
}
}
+ if (Context.getTargetInfo().getTriple().isLoongArch()) {
+for (const auto &Feature : ParsedAttrs.Features) {
+ StringRef CurFeature = Feature;
+ if (Cu
@@ -85,6 +85,7 @@ struct ArchInfo {
};
bool isValidArchName(StringRef Arch);
+bool isValidFeatureName(StringRef Arch);
wangleiat wrote:
Arch -> Feature
https://github.com/llvm/llvm-project/pull/140700
___
cfe-commi
@@ -522,6 +522,12 @@ enum class TemplateSubstitutionKind : char {
llvm::PointerUnion *
findInstantiationOf(const Decl *D);
+/// Similar to \p findInstantiationOf(), but it wouldn't assert if the
+/// instantiation was not found within the current instantiation
googlewalt wrote:
Never mind. Fixed by
https://github.com/llvm/llvm-project/commit/f3b404be973507432cf86c177978d9de708d850c.
https://github.com/llvm/llvm-project/pull/134034
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
@@ -85,6 +85,7 @@ struct ArchInfo {
};
bool isValidArchName(StringRef Arch);
+bool isValidFeatureName(StringRef Arch);
Ami-zhang wrote:
Done.
https://github.com/llvm/llvm-project/pull/140700
___
cfe-commits mailing
@@ -13458,6 +13473,10 @@ class Sema final : public SemaBase {
// FIXME: Should we have a similar limit for other forms of synthesis?
unsigned NonInstantiationEntries;
+ /// The number of \p CodeSynthesisContexts that are not constraint
+ /// substitution.
+ unsigned Non
https://github.com/zyn0217 deleted
https://github.com/llvm/llvm-project/pull/122423
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Devon Loehr
Date: 2025-05-28T12:15:22+02:00
New Revision: 63de20c0de05ce7b8b3968a9d210aa0f3d01acd4
URL:
https://github.com/llvm/llvm-project/commit/63de20c0de05ce7b8b3968a9d210aa0f3d01acd4
DIFF:
https://github.com/llvm/llvm-project/commit/63de20c0de05ce7b8b3968a9d210aa0f3d01acd4.diff
L
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/141091
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
googlewalt wrote:
This doesn't build with -Werror, as seen in llvm-ci build
https://lab.llvm.org/buildbot/#/builders/145/builds/7332. Can you please take
a look?
```
FAILED:
tools/clang/utils/TableGen/CMakeFiles/clang-tblgen.dir/ClangBuiltinTemplatesEmitter.cpp.o
ccache /home/buildbots/llv
hanickadot wrote:
> @hanickadot Ping :)
Soon, busy, will be back.
https://github.com/llvm/llvm-project/pull/98756
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mgoudar edited
https://github.com/llvm/llvm-project/pull/141730
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mylai-mtk wrote:
> There are intentional tradeoffs here, for the ABI, which should be made
> carefully; in some cases, it might be the right tradeoff to reject
> standard-compliant code. If you've considered it, I guess I won't object. I
> can't find any discussion of it, though.
The reductio
https://github.com/owenca updated
https://github.com/llvm/llvm-project/pull/141714
>From 2d30adbd7bcce89c157e5b3eb517854cfbd35f48 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Tue, 27 May 2025 20:50:56 -0700
Subject: [PATCH] [clang-format] Handle .h files for LK_C and LK_ObjC
Fix #137792
---
Author: Steven Perron
Date: 2025-05-28T07:51:03-04:00
New Revision: 5584020d8abf46f2852a59ed5333a7f2145bfec5
URL:
https://github.com/llvm/llvm-project/commit/5584020d8abf46f2852a59ed5333a7f2145bfec5
DIFF:
https://github.com/llvm/llvm-project/commit/5584020d8abf46f2852a59ed5333a7f2145bfec5.diff
https://github.com/s-perron closed
https://github.com/llvm/llvm-project/pull/140061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4466,7 +4473,8 @@ RValue CodeGenFunction::EmitBuiltinExpr(const GlobalDecl
GD, unsigned BuiltinID,
Value *SizeVal = EmitScalarExpr(E->getArg(2));
EmitNonNullArgCheck(Dest, E->getArg(0)->getType(),
E->getArg(0)->getExprLoc(), FD, 0);
-Bui
1 - 100 of 401 matches
Mail list logo
