https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/130012
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -588,6 +590,19 @@ let Predicates = [HasVendorXqcilo, IsRV32] in {
def QC_E_SW: QCIRVInstESStore<0b110, 0b11, "qc.e.sw">;
} // Predicates = [HasVendorXqcilo, IsRV32]
+let Predicates = [HasVendorXqcili, IsRV32] in {
+let hasSideEffects = 0, mayLoad = 0, mayStore = 0 in
@@ -588,6 +590,19 @@ let Predicates = [HasVendorXqcilo, IsRV32] in {
def QC_E_SW: QCIRVInstESStore<0b110, 0b11, "qc.e.sw">;
} // Predicates = [HasVendorXqcilo, IsRV32]
+let Predicates = [HasVendorXqcili, IsRV32] in {
+let hasSideEffects = 0, mayLoad = 0, mayStore = 0 in
arsenm wrote:
reverse ping
https://github.com/llvm/llvm-project/pull/115821
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
> Is this PR intended to match the signature of `__enqueue_kernel_basic`
> defined in device library? e.g.
Yes
> https://github.com/ROCm/llvm-project/blob/amd-staging/amd/device-libs/opencl/src/devenq/enqueue.cl#L172
>
> Since ndrange_t is passed by value, it becomes pointer to
@@ -257,6 +266,17 @@ class SME2_Tile_Movaz_Pat(name # _PSEUDO) $tile, $base, $offset)>;
+class SME2_ZA_Tile_TwoVec_Pat
+: Pat<(intrinsic imm_ty:$tile, vt:$Zn, vt:$Zm),
+ (!cast(name # _PSEUDO) $tile, $Zn, $Zm)>;
+
+class SME2_ZA_Tile_Vec_Multi_Pat
L
https://github.com/yxsamliu approved this pull request.
https://github.com/llvm/llvm-project/pull/130011
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9180,6 +9180,12 @@ bool Sema::hasAcceptableDefinition(NamedDecl *D,
NamedDecl **Suggested,
if (!getLangOpts().Modules && !getLangOpts().ModulesLocalVisibility)
return true;
+ // The external source may have additional definitions of this entity that
are
+ // visi
https://github.com/frasercrmck milestoned
https://github.com/llvm/llvm-project/pull/126908
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9180,6 +9180,12 @@ bool Sema::hasAcceptableDefinition(NamedDecl *D,
NamedDecl **Suggested,
if (!getLangOpts().Modules && !getLangOpts().ModulesLocalVisibility)
return true;
+ // The external source may have additional definitions of this entity that
are
+ // visi
https://github.com/frasercrmck closed
https://github.com/llvm/llvm-project/pull/126908
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9180,6 +9180,12 @@ bool Sema::hasAcceptableDefinition(NamedDecl *D,
NamedDecl **Suggested,
if (!getLangOpts().Modules && !getLangOpts().ModulesLocalVisibility)
return true;
+ // The external source may have additional definitions of this entity that
are
+ // visi
frasercrmck wrote:
/cherry-pick a2b05761724e5243056988d9d6bf1a5a94715b74
https://github.com/llvm/llvm-project/pull/126908
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/momo5502 updated
https://github.com/llvm/llvm-project/pull/128866
>From bb4091d2f9b7062aa83e5bee2ba525478a7dbd0a Mon Sep 17 00:00:00 2001
From: Maurice Heumann
Date: Wed, 26 Feb 2025 14:31:47 +0100
Subject: [PATCH 1/7] Instantiate destructors from initialized anonymous union
@@ -5283,6 +5283,102 @@ Sema::SetDelegatingInitializer(CXXConstructorDecl
*Constructor,
return false;
}
+static void MarkFieldDestructorReferenced(Sema &S, SourceLocation Location,
+ FieldDecl *Field) {
+ if (Field->isInvalidDecl())
https://github.com/momo5502 updated
https://github.com/llvm/llvm-project/pull/128866
>From bb4091d2f9b7062aa83e5bee2ba525478a7dbd0a Mon Sep 17 00:00:00 2001
From: Maurice Heumann
Date: Wed, 26 Feb 2025 14:31:47 +0100
Subject: [PATCH 1/7] Instantiate destructors from initialized anonymous union
@@ -5283,6 +5283,102 @@ Sema::SetDelegatingInitializer(CXXConstructorDecl
*Constructor,
return false;
}
+static void MarkFieldDestructorReferenced(Sema &S, SourceLocation Location,
+ FieldDecl *Field) {
+ if (Field->isInvalidDecl())
@@ -46,21 +52,124 @@ void CIRGenFunction::emitAutoVarAlloca(const VarDecl &d) {
address = createTempAlloca(allocaTy, alignment, loc, d.getName());
declare(address.getPointer(), &d, ty, getLoc(d.getSourceRange()), alignment);
+ emission.Addr = address;
setAddrOfLocalVar
@@ -0,0 +1,66 @@
+//===--===//
+//
+// 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: Apac
@@ -95,6 +96,10 @@ class DependencyScanningService {
return SharedCache;
}
+ ModuleCacheMutexes &getSharedModuleCacheMutexes() {
jansvoboda11 wrote:
We don't _need_ it, but we currently call
`std::make_shared` for each TU, which is unnecessary. We
co
erichkeane wrote:
> Ah sorry I forgot to reply to your review.
>
> > We might take a look at `SetVector` and see if there are any learnings we
> > can take from that. It has a set and a vector together, but might have some
> > interesting things.
>
> I did take a look at this actually, but it
https://github.com/shafik commented:
Just throwing in my $0.02 and saying I agree w/ Aaron and Vlad here, this is
not the right direction. Having a `const` method return a non-const that we
will then mutate is just not clean code.
Having to use `const_cast` documents we are violating expectati
https://github.com/ymand approved this pull request.
https://github.com/llvm/llvm-project/pull/129930
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -551,91 +551,92 @@ void transferCallReturningOptional(const CallExpr *E,
setHasValue(*Loc, State.Env.makeAtomicBoolValue(), State.Env);
}
+// Returns true if the const accessor is handled by caching.
+// Returns false if we could not cache. We should perform default handl
https://github.com/jvoung updated
https://github.com/llvm/llvm-project/pull/129930
>From 81f5cfde1029b3c9ddd62eb0587ed370d67cccab Mon Sep 17 00:00:00 2001
From: Jan Voung
Date: Wed, 5 Mar 2025 20:15:48 +
Subject: [PATCH 1/7] [clang][dataflow] Add test for crash repro and clean up
const acc
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Chris B (llvm-beanz)
Changes
This adds a flag to the DXC driver to enable calling the metal shader converter
if it is available to convert the final shader output for metal.
---
Full diff: https://github.com/llvm/llvm-project/pull/130173.
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-hlsl
Author: Chris B (llvm-beanz)
Changes
This adds a flag to the DXC driver to enable calling the metal shader converter
if it is available to convert the final shader output for metal.
---
Full diff: https://github.c
https://github.com/llvm-beanz created
https://github.com/llvm/llvm-project/pull/130173
This adds a flag to the DXC driver to enable calling the metal shader converter
if it is available to convert the final shader output for metal.
>From 0e5a24665c3a9522263b2ddfbce5f46e6a0e33c9 Mon Sep 17 00:0
@@ -94,10 +203,59 @@ void CIRGenFunction::emitVarDecl(const VarDecl &d) {
assert(d.hasLocalStorage());
- assert(!cir::MissingFeatures::opAllocaVarDeclContext());
+ CIRGenFunction::VarDeclContext varDeclCtx{*this, &d};
return emitAutoVarDecl(d);
}
+void CIRGenFunctio
@@ -46,21 +52,124 @@ void CIRGenFunction::emitAutoVarAlloca(const VarDecl &d) {
address = createTempAlloca(allocaTy, alignment, loc, d.getName());
declare(address.getPointer(), &d, ty, getLoc(d.getSourceRange()), alignment);
+ emission.Addr = address;
setAddrOfLocalVar
https://github.com/mpark edited https://github.com/llvm/llvm-project/pull/129982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mpark edited https://github.com/llvm/llvm-project/pull/129982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/andykaylor closed
https://github.com/llvm/llvm-project/pull/129072
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nikic wrote:
This causes a large compile-time regression for C++ code:
https://llvm-compile-time-tracker.com/compare_clang.php?from=37aad2c1196ea3de242d855cfe38bc25a65d6f5e&to=a24523ac8dc07f3478311a5969184b922b520395&stat=instructions%3Au&sortBy=interestingness
It adds 1-3% overhead per file du
https://github.com/mpark edited https://github.com/llvm/llvm-project/pull/129982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -70,16 +94,38 @@ class ParentMapContext::ParentMap {
push_back(Value);
}
bool contains(const DynTypedNode &Value) {
- return Seen.contains(Value);
+ assert(Value.getMemoizationData());
+ bool found = FragileLazySeenCache.contains(&Value);
+
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/130173
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -551,91 +551,92 @@ void transferCallReturningOptional(const CallExpr *E,
setHasValue(*Loc, State.Env.makeAtomicBoolValue(), State.Env);
}
+// Returns true if the const accessor is handled by caching.
+// Returns false if we could not cache. We should perform default handl
@@ -141,6 +141,8 @@ void Parser::ParseHLSLAnnotations(ParsedAttributes &Attrs,
return;
}
+ II = PP.getIdentifierInfo(II->getName().lower());
spall wrote:
Okay that might requirement a significant change. I'll look into it
https://github.com/llvm/llvm
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Aaron Ballman (AaronBallman)
Changes
This paper (https://www.open-std.org/jtc1/sc22/wg14/www/docs/n3411.pdf) allows
a source file to end without a newline. Clang has supported this as a
conforming extension for a long time, so this suppre
@@ -0,0 +1,36 @@
+[build-system]
+requires = ["setuptools>=42", "setuptools_scm==8.1.0"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "clang"
+description = "libclang python bindings"
+readme = {file = "README.txt", content-type = "text/plain"}
+
+license = {text
https://github.com/therealcoochieman created
https://github.com/llvm/llvm-project/pull/130182
Aims to fix #61702.
I encountered an interesting behavior during my testing. It seems the Itanium
Record Layout Builder can give out Wpadded-bitfield warnings with the wrong
amount of padding when mi
@@ -0,0 +1,36 @@
+[build-system]
+requires = ["setuptools>=42", "setuptools_scm==8.1.0"]
henryiii wrote:
Is there a reason you've pinned to setuptools_scm exactly, but not setuptools?
I would generally avoid this, unless there's a really good reason to pin it.
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Theo de Magalhaes (therealcoochieman)
Changes
Aims to fix #61702.
I encountered an interesting behavior during my testing. It seems the Itanium
Record Layout Builder can give out Wpadded-bitfield warnings with the wrong
amount of padding
https://github.com/jhuber6 created
https://github.com/llvm/llvm-project/pull/130177
Summary:
This attribute is mostly borrowed from OpenCL, but is useful in general
for accessing the LLVM vector types. Previously the only way to use it
was through typedefs. This patch changes that to allow use a
https://github.com/mizvekov closed
https://github.com/llvm/llvm-project/pull/126088
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/129504
___
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/115052
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Carlos =?utf-8?q?Gálvez?= ,
Carlos =?utf-8?q?Gálvez?=
Message-ID:
In-Reply-To:
https://github.com/carlosgalvezp updated
https://github.com/llvm/llvm-project/pull/128150
>From a05da40d6103c664e5057d8a5010bbb57424b427 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20G=C3=A1lvez?=
Date: Thu, 2
https://github.com/mpark updated
https://github.com/llvm/llvm-project/pull/129982
>From 39b845fd64c2ce173b95ddcafbbab2d5116aa68e Mon Sep 17 00:00:00 2001
From: Michael Park
Date: Wed, 5 Mar 2025 18:46:10 -0800
Subject: [PATCH 1/3] [clang][modules] Add a unit test for the assertion
failure in b
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/130107
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/efriedma-quic updated
https://github.com/llvm/llvm-project/pull/129952
>From 9ac636ee137248cafe38f4d2e016cfb885142dff Mon Sep 17 00:00:00 2001
From: Eli Friedman
Date: Wed, 5 Mar 2025 14:20:21 -0800
Subject: [PATCH 1/2] [clang] Reject constexpr-unknown values as constant
exp
@@ -78,7 +78,7 @@ typedef int __attribute__((ext_vector_type(0))) e4;
// expected-e
// no support for vector enum type
enum { e_2 } x3 __attribute__((vector_size(64))); // expected-error {{invalid
vector element type}}
-int x4 __attribute__((ext_vector_type(64
@@ -70,16 +94,38 @@ class ParentMapContext::ParentMap {
push_back(Value);
}
bool contains(const DynTypedNode &Value) {
- return Seen.contains(Value);
+ assert(Value.getMemoizationData());
+ bool found = FragileLazySeenCache.contains(&Value);
+
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/130177
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -78,7 +78,7 @@ typedef int __attribute__((ext_vector_type(0))) e4;
// expected-e
// no support for vector enum type
enum { e_2 } x3 __attribute__((vector_size(64))); // expected-error {{invalid
vector element type}}
-int x4 __attribute__((ext_vector_type(64
https://github.com/joaosaffran approved this pull request.
https://github.com/llvm/llvm-project/pull/130173
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Aaron Ballman
Date: 2025-03-06T16:03:11-05:00
New Revision: 73b2ad51f46e67d00cbc7394a2c1cf9b3316cf92
URL:
https://github.com/llvm/llvm-project/commit/73b2ad51f46e67d00cbc7394a2c1cf9b3316cf92
DIFF:
https://github.com/llvm/llvm-project/commit/73b2ad51f46e67d00cbc7394a2c1cf9b3316cf92.diff
@@ -78,7 +78,7 @@ typedef int __attribute__((ext_vector_type(0))) e4;
// expected-e
// no support for vector enum type
enum { e_2 } x3 __attribute__((vector_size(64))); // expected-error {{invalid
vector element type}}
-int x4 __attribute__((ext_vector_type(64
https://github.com/Zhenhang1213 updated
https://github.com/llvm/llvm-project/pull/130027
>From 5af4748d72c4953f4c38594c32fddf1fb276cdc7 Mon Sep 17 00:00:00 2001
From: Austin
Date: Thu, 6 Mar 2025 17:25:55 +0800
Subject: [PATCH] [ARM] Using cp15 while mtp =auto and arch is arm_arch6k and
suppo
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Tom Honermann (tahonermann)
Changes
Static analysis identified two uses of `getAs()` for which the result pointer
was unconditionally dereferenced. Source code inspection confirmed that the
target type is assured by prior checks.
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/130188
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/130164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Meinersbur wrote:
@tblah I don't think it is blocking cancellation, we just don't know what is
going on. The `omp.par.outlined.exit` apparently was once need for cancellation
points. Looking at the `cancel_codegen.cpp` test case, it seems to evantually
jump to `omp.par.exit.exitStub`. That one
Keenuts wrote:
Hi all!
This PR was dormant for a while because proposals had to be made & merged.
The HLSL proposal which covers `hlsl_private` is now merged:
https://github.com/llvm/wg-hlsl/commit/4c9e11aa50727c53b10d232195fb88fd5376f19c
https://github.com/llvm/llvm-project/pull/122103
__
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/130177
>From 728e1bd9cccb56a0acaf5abb35fe64cacc5b4ae9 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Thu, 6 Mar 2025 15:08:25 -0600
Subject: [PATCH 1/2] [Clang] Treat `ext_vector_type` as a regular type
attribute
@@ -78,7 +78,7 @@ typedef int __attribute__((ext_vector_type(0))) e4;
// expected-e
// no support for vector enum type
enum { e_2 } x3 __attribute__((vector_size(64))); // expected-error {{invalid
vector element type}}
-int x4 __attribute__((ext_vector_type(64
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/130177
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
VladiKrapp-Arm wrote:
@Zhenhang1213 Might I suggest changes along the lines of
https://github.com/VladiKrapp-Arm/llvm-project/commit/63e4f10eaefc024e2afc41e5151411a383226a8b
This separates the check for whether the platform allows hardware tls from
whether it is possible to encode using thumb
https://github.com/sarnex ready_for_review
https://github.com/llvm/llvm-project/pull/129545
___
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/130177
>From 728e1bd9cccb56a0acaf5abb35fe64cacc5b4ae9 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Thu, 6 Mar 2025 15:08:25 -0600
Subject: [PATCH 1/3] [Clang] Treat `ext_vector_type` as a regular type
attribute
Author: Oleksandr T.
Date: 2025-03-07T01:05:56+02:00
New Revision: 93f0f3d33be596321730e9194cf24a73a75ce702
URL:
https://github.com/llvm/llvm-project/commit/93f0f3d33be596321730e9194cf24a73a75ce702
DIFF:
https://github.com/llvm/llvm-project/commit/93f0f3d33be596321730e9194cf24a73a75ce702.diff
Author: Tom Honermann
Date: 2025-03-06T18:12:20-05:00
New Revision: 813325e5d44c01f50aaca035629fcba8157fd5c0
URL:
https://github.com/llvm/llvm-project/commit/813325e5d44c01f50aaca035629fcba8157fd5c0
DIFF:
https://github.com/llvm/llvm-project/commit/813325e5d44c01f50aaca035629fcba8157fd5c0.diff
@@ -588,6 +590,19 @@ let Predicates = [HasVendorXqcilo, IsRV32] in {
def QC_E_SW: QCIRVInstESStore<0b110, 0b11, "qc.e.sw">;
} // Predicates = [HasVendorXqcilo, IsRV32]
+let Predicates = [HasVendorXqcili, IsRV32] in {
+let hasSideEffects = 0, mayLoad = 0, mayStore = 0 in
kiranchandramohan wrote:
> @kiranchandramohan #130135
>
> After a hard time getting even #114419 approved, I don't think anybody is
> really interested in reviewing/fixing non-SESE regions. With cancellation
> points basically not supported, I guess it doesn't matter whether this PR
> breaks
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/129917
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
Thanks! You should also add a release note to `clang/docs/ReleaseNotes.rst` so
users know about the change.
https://github.com/llvm/llvm-project/pull/129917
___
cfe-commits mailing list
cfe-commits@lists.llvm
@@ -7596,6 +7596,17 @@ void Sema::ProcessDeclAttributeList(
D->setInvalidDecl();
}
+ // Warn on global constructors and destructors created by attributes.
+ if (D->hasAttr() &&
+ !getDiagnostics().isIgnored(diag::warn_global_constructor,
+
@@ -1358,6 +1358,12 @@ def HasVendorXqciint
AssemblerPredicate<(all_of FeatureVendorXqciint),
"'Xqciint' (Qualcomm uC Interrupts Extension)">;
+def FeatureVendorXqcili : RISCVExperimentalExtension<0, 2, "Qualcomm uC Load
Large Immediate Extensio
https://github.com/arsenm closed
https://github.com/llvm/llvm-project/pull/129837
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/130143
I think this is the one. Patch needs a little cleanup and documentation, which
I'll do later.
I'll also later add the test results. For now, check the CI once.
>From 61da70b5a1df38d66ba93ca49093d0802e7ef1b3 M
https://github.com/balazske updated
https://github.com/llvm/llvm-project/pull/129557
From 8006fde8ad615dffcaf38bc017b58d6bbed59ae2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?=
Date: Mon, 3 Mar 2025 17:57:19 +0100
Subject: [PATCH 1/3] [clang][analyzer] Add BugReporterVisito
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
I think this is the one. Patch needs a little cleanup and documentation, which
I'll do later.
I'll also later add the test results. For now, check the CI once.
---
Full diff: https://github.com/llvm/llvm-pr
@@ -551,15 +551,18 @@ void transferCallReturningOptional(const CallExpr *E,
setHasValue(*Loc, State.Env.makeAtomicBoolValue(), State.Env);
}
-void handleConstMemberCall(const CallExpr *CE,
+bool handleConstMemberCall(const CallExpr *CE,
jvoung wrote:
Done!
https://github.com/jvoung updated
https://github.com/llvm/llvm-project/pull/129930
>From 81f5cfde1029b3c9ddd62eb0587ed370d67cccab Mon Sep 17 00:00:00 2001
From: Jan Voung
Date: Wed, 5 Mar 2025 20:15:48 +
Subject: [PATCH 1/5] [clang][dataflow] Add test for crash repro and clean up
const acc
https://github.com/rjmccall approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/126382
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2138,6 +2138,8 @@ static void handleConstructorAttr(Sema &S, Decl *D, const
ParsedAttr &AL) {
if (AL.getNumArgs() &&
!S.checkUInt32Argument(AL, AL.getArgAsExpr(0), priority))
return;
+ S.Diag(D->getLocation(), diag::warn_global_constructor)
Aa
@@ -715,8 +715,16 @@ MCSymbol *AsmPrinter::getSymbolPreferLocal(const
GlobalValue &GV) const {
return TM.getSymbol(&GV);
}
-/// EmitGlobalVariable - Emit the specified global variable to the .s file.
void AsmPrinter::emitGlobalVariable(const GlobalVariable *GV) {
+ MaybeA
https://github.com/svs-quic commented:
Please add entries in RISCVUsage.rst and ReleaseNotes.md.
https://github.com/llvm/llvm-project/pull/130012
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
Author: erichkeane
Date: 2025-03-06T08:55:32-08:00
New Revision: 93b022944c04634eb30689cb72f0fe0ac2cb5f0d
URL:
https://github.com/llvm/llvm-project/commit/93b022944c04634eb30689cb72f0fe0ac2cb5f0d
DIFF:
https://github.com/llvm/llvm-project/commit/93b022944c04634eb30689cb72f0fe0ac2cb5f0d.diff
LO
https://github.com/topperc requested changes to this pull request.
Dropping my approval based on other comments.
https://github.com/llvm/llvm-project/pull/130012
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
AaronBallman wrote:
> Thanks for the feedback!
>
> Is there a way to call `ASTContext::createMangleContext()` when I have a
> `const ASTUnit`? I've tried to make `ASTContext::createMangleContext()` const
> but this failed as it seems to actually call methods that potentially modify
> the cont
@@ -107,7 +107,7 @@ RawAddress CodeGenFunction::CreateTempAlloca(llvm::Type
*Ty, CharUnits Align,
// be different from the type defined by the language. For example,
// in C++ the auto variables are in the default address space. Therefore
// cast alloca to the default ad
@@ -614,6 +614,20 @@ void
AMDGPUTargetCodeGenInfo::setCUDAKernelCallingConvention(
FT, FT->getExtInfo().withCallingConv(CC_OpenCLKernel));
}
+/// Return IR struct type for rtinfo struct in rocm-device-libs used for device
+/// enqueue.
+///
+/// ptr addrspace(1) kernel_
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/129933
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/teresajohnson approved this pull request.
lgtm with the change suggested by @ellishg to use getProfileGenName. I believe
we always specify a profile filename using the `=` form of the option, which is
why we have not encountered this issue.
https://github.com/llvm/llvm-proje
https://github.com/teresajohnson edited
https://github.com/llvm/llvm-project/pull/129736
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -733,16 +733,17 @@ ENVIRONMENT
.. envvar:: CPATH
- If this environment variable is present, it is treated as a delimited list of
- paths to be added to the default system include path list. The delimiter is
- the platform dependent delimiter, as used in the PATH environ
@@ -0,0 +1,66 @@
+//===--===//
+//
+// 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: Apac
@@ -0,0 +1,66 @@
+//===--===//
+//
+// 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: Apac
201 - 300 of 336 matches
Mail list logo
