@@ -7117,6 +7117,13 @@ void Clang::ConstructJob(Compilation &C, const JobAction
&JA,
}
}
+ // The offloading devices do not support RTTI.
asudarsa wrote:
I see that IsHIPDevice has been added to the check here (compared to what was
deleted inside Cod
https://github.com/jansvoboda11 created
https://github.com/llvm/llvm-project/pull/133827
This essentially reverts commit 227f71995804
(https://reviews.llvm.org/D150320). The original change to disable checking for
relocated modules was built on the assumption that the file system is immutable
asudarsa wrote:
Hi @jhuber6
Just a quick ping to check if this PR is still alive. I can take a look if it
is.
Thanks
https://github.com/llvm/llvm-project/pull/125556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: None (metkarpoonam)
Changes
Implement dst algorithm in the hlsl_intrinsics.h and added test cases for HLSL
codegen and sema
fixes: https://github.com/llvm/llvm-project/issues/99108
---
Full diff: https://github.com/llvm/llvm-project
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Jan Svoboda (jansvoboda11)
Changes
This essentially reverts commit 227f71995804
(https://reviews.llvm.org/D150320). The original change to disable checking for
relocated modules was built on the assumption that the file system is immutabl
https://github.com/andykaylor created
https://github.com/llvm/llvm-project/pull/133829
The initial upstreaming of unary operations left promoted types unhandled for
the unary plus and minus operators. This change implements support for promoted
types and performs a bit of related code cleanup.
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Andy Kaylor (andykaylor)
Changes
The initial upstreaming of unary operations left promoted types unhandled for
the unary plus and minus operators. This change implements support for promoted
types and performs a bit of related code cleanu
andykaylor wrote:
@mmha
https://github.com/llvm/llvm-project/pull/133829
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
andykaylor wrote:
@mmha
https://github.com/llvm/llvm-project/pull/133815
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?q?Iñaki?= Amatria Barral
Message-ID:
In-Reply-To:
@@ -863,6 +863,12 @@ static void
parsePreprocessorArgs(Fortran::frontend::PreprocessorOptions &opts,
(currentArg->getOption().matches(clang::driver::options::OPT_cpp))
? PPMacrosFlag::Include
MagentaTreehouse wrote:
> Happy to adjust this @MagentaTreehouse for LLVM 20 - what would that require?
@lyledean1 I think the process is that the release notes are left unchanged in
this PR; after it is merged we add this to [LLVM 20.X
Release](https://github.com/llvm/llvm-project/milestone/2
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/133755
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -327,4 +337,4 @@ class CIRBaseBuilderTy : public mlir::OpBuilder {
} // namespace cir
-#endif
+#endif
andykaylor wrote:
This indicates that you (or some tool you're using) removed the newline
character at the end of the file. It's a good practice to conf
@@ -0,0 +1,274 @@
+//===--- CIRGenExprAgg.cpp - Emit CIR Code from Aggregate Expressions
-===//
+//
+// 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,274 @@
+//===--- CIRGenExprAgg.cpp - Emit CIR Code from Aggregate Expressions
-===//
+//
+// 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,274 @@
+//===--- CIRGenExprAgg.cpp - Emit CIR Code from Aggregate Expressions
-===//
+//
+// 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,274 @@
+//===--- CIRGenExprAgg.cpp - Emit CIR Code from Aggregate Expressions
-===//
+//
+// 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,274 @@
+//===--- CIRGenExprAgg.cpp - Emit CIR Code from Aggregate Expressions
-===//
+//
+// 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,274 @@
+//===--- CIRGenExprAgg.cpp - Emit CIR Code from Aggregate Expressions
-===//
+//
+// 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,274 @@
+//===--- CIRGenExprAgg.cpp - Emit CIR Code from Aggregate Expressions
-===//
+//
+// 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-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-aarch64-ubuntu`
running on `linaro-lldb-aarch64-ubuntu` while building `clang` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/59/builds/15227
Here is the relevant piece of the
@@ -1098,12 +1098,39 @@ static TypeSourceInfo
*getTypeSourceInfoForStdAlignValT(Sema &S,
return S.Context.getTrivialTypeSourceInfo(StdAlignValDecl);
}
+// When searching for custom allocators on the PromiseType we want to
+// warn that we will ignore type aware allocators.
@@ -16327,79 +16531,181 @@ static CanQualType RemoveAddressSpaceFromPtr(Sema
&SemaRef,
PtrTy->getPointeeType().getUnqualifiedType(), PtrQuals)));
}
-static inline bool
-CheckOperatorNewDeleteTypes(Sema &SemaRef, const FunctionDecl *FnDecl,
-C
https://github.com/lyledean1 edited
https://github.com/llvm/llvm-project/pull/132116
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/asudarsa edited
https://github.com/llvm/llvm-project/pull/133776
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/GeorgeKA deleted
https://github.com/llvm/llvm-project/pull/133806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/GeorgeKA closed
https://github.com/llvm/llvm-project/pull/133597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jyknight wrote:
Just to say again, I think the discussion may be crisper if we consider this as
two entirely-distinct features:
1. An expansion of "compatible type" for purposes of pointer casts/non-UB/etc.
2. The ability to provide two definitions of the same struct type in the same
scope, to
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/133426
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,86 @@
+// RUN: %clang_cc1 -triple arm64-apple-ios -fsyntax-only -verify
-fptrauth-intrinsics %s
+// RUN: %clang_cc1 -triple aarch64-linux-gnu -fsyntax-only -verify
-fptrauth-intrinsics %s
+
+#if __has_feature(ptrauth_qualifier)
+#warning __ptrauth qualifier enabled!
+
pawosm-arm wrote:
> I guessing we pass `-lm` twice here to work around `ld.bfd` not handling
> static libraries in a pleasant way?
This is all due to the way libamath incremetnally emerged into its present
existence. And yes, static linking added another layer of problems here.
https://githu
https://github.com/nicovank updated
https://github.com/llvm/llvm-project/pull/133756
>From 4df359ed712486c9fc94f1296474cdb06715 Mon Sep 17 00:00:00 2001
From: Nicolas van Kempen
Date: Mon, 31 Mar 2025 15:16:32 -0400
Subject: [PATCH] [clang-tidy] Use --match-full-lines instead of
--strict-w
https://github.com/nicovank updated
https://github.com/llvm/llvm-project/pull/133756
>From 4df359ed712486c9fc94f1296474cdb06715 Mon Sep 17 00:00:00 2001
From: Nicolas van Kempen
Date: Mon, 31 Mar 2025 15:16:32 -0400
Subject: [PATCH] [clang-tidy] Use --match-full-lines instead of
--strict-w
@@ -818,6 +818,12 @@ void CodeGenModule::EmitCXXModuleInitFunc(Module *Primary)
{
Fn->addFnAttr("device-init");
}
+ if (getTarget().isBranchProtectionSupportedArch(
+ getTarget().getTargetOpts().CPU)) {
+TargetInfo::BranchProtectionInfo BPI(getLangOpts());
jyknight wrote:
LLVM's intentional usage is either to reduce redundant vtable emission via
ensuring there is a key method, OR, to intentionally create a useless vtable,
in order to reduce redundant debuginfo emission (it can be emitted only in the
vtable's TU when there's a vtable.) I expect b
https://github.com/andykaylor approved this pull request.
lgtm
https://github.com/llvm/llvm-project/pull/133152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/andykaylor updated
https://github.com/llvm/llvm-project/pull/133157
>From ecaf51cd128e9045b9926ecafc918ec3e35a9908 Mon Sep 17 00:00:00 2001
From: Andy Kaylor
Date: Mon, 24 Mar 2025 14:07:34 -0700
Subject: [PATCH 1/2] [CIR] Upstream support for while and do..while loops
This
@@ -88,8 +88,8 @@ TYPE("assembler-with-cpp", Asm, PP_Asm,
"S", phases
// modules when Flang needs to emit pre-processed files. Therefore, the
// `PP_TYPE` is set to `PP_Fortran` so that the driver is fine with
// "pre-processing a pre-processed fil
@@ -10918,7 +11083,44 @@ bool
OverloadCandidate::NotValidBecauseConstraintExprHasError() const {
OverloadingResult
OverloadCandidateSet::BestViableFunction(Sema &S, SourceLocation Loc,
iterator &Best) {
+
+ bool TwoPhaseResolution =
+
https://github.com/kiranchandramohan approved this pull request.
LGTM. Please wait for other reviewers.
https://github.com/llvm/llvm-project/pull/133745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
dtcxzyw wrote:
> > The ubsan handler cannot be eliminated because inaccessiblemem: readwrite
> > indicates that it has externally observable side effects.
>
> That is not how things work. For example:
>
> ```
> $ echo "declare void @g() define void @f() nounwind { call void @g()
> memory(argm
@@ -1390,7 +1390,19 @@ class ASTReader
/// predefines buffer may contain additional definitions.
std::string SuggestedPredefines;
- llvm::DenseMap DefinitionSource;
+ struct DefinitionSourceFlags {
+ExtKind HasExternalDefinitions : 2;
+
+/// Indicates if given fu
https://github.com/ChuanqiXu9 commented:
LGTM. I'll leave the formal approval to @efriedma-quic
https://github.com/llvm/llvm-project/pull/133716
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
https://github.com/ChuanqiXu9 approved this pull request.
LGTM. Thanks.
https://github.com/llvm/llvm-project/pull/133500
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Lancern approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/133815
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4853,9 +4853,16 @@ void UnwrappedLineParser::readToken(int LevelDifference)
{
PreviousWasComment = FormatTok->is(tok::comment);
while (!Line->InPPDirective && FormatTok->is(tok::hash) &&
- (!Style.isVerilog() ||
-Keywords.isVerilogPPDirective
Author: Owen Pan
Date: 2025-03-31T23:16:41-07:00
New Revision: d3be29642fa65e5ade434d860cfcc193f8278d4e
URL:
https://github.com/llvm/llvm-project/commit/d3be29642fa65e5ade434d860cfcc193f8278d4e
DIFF:
https://github.com/llvm/llvm-project/commit/d3be29642fa65e5ade434d860cfcc193f8278d4e.diff
LOG:
https://github.com/owenca closed
https://github.com/llvm/llvm-project/pull/133673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2832,6 +2832,23 @@ TEST_F(TokenAnnotatorTest, UnderstandTableGenTokens) {
Tokens = Annotate("!cond");
EXPECT_TOKEN(Tokens[0], tok::identifier, TT_TableGenCondOperator);
+ // The paste operator should not be treated as a preprocessor directive even
+ // if it is on a
https://github.com/junlarsen created
https://github.com/llvm/llvm-project/pull/133870
This function was previously used to get a type to the protocol that
was used to bitcast the initializer of GenerateProtocol. This bitcast
has later been removed (thanks to opaque pointers), but the member was
junlarsen wrote:
* **#133870** https://app.graphite.dev/github/pr/llvm/llvm-project/133870?utm_source=stack-comment-icon";
target="_blank">https://static.graphite.dev/graphite-32x32-black.png"; alt="Graphite"
width="10px" height="10px"/> 👈 https://app.graphite.dev/github/pr/llvm/llvm-project/13
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Mats Jun Larsen (junlarsen)
Changes
This function was previously used to get a type to the protocol that
was used to bitcast the initializer of GenerateProtocol. This bitcast
has later been removed (thanks to opaque pointers), but t
https://github.com/mariusdr edited
https://github.com/llvm/llvm-project/pull/133574
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1578,6 +1578,11 @@ class MovDPP8Pattern : GCNPat <
let OtherPredicates = [Pred];
}
+def : GCNPat <
jayfoad wrote:
Can you do this in the definition of V_CVT_OFF_F32_I4, instead of using a
separate Pat?
https://github.com/llvm/llvm-project/pull/133741
asudarsa wrote:
> > > @farzonl can you please revert the original commit? It seems to break
> > > just any LLVM build, so I guess the CI testing of all new PRs should be
> > > affected.
> >
> >
> > I don't see it breaking any build in LLVM, just the amd offload ones. Al
> > the Premerge test
@@ -3115,11 +3115,24 @@ void CastOperation::CheckCStyleCast() {
Self.CurFPFeatureOverrides());
}
}
- if (DestType->isNullPtrType() && !SrcType->isNullPtrType()) {
-Self.Diag(SrcExpr.get()->getExprLoc(), diag::err_nullptr_cast)
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`amdgpu-offload-rhel-8-cmake-build-only` running on `rocm-docker-rhel-8` while
building `clang` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/204/builds/5082
Here is the releva
llvmbot wrote:
@llvm/pr-subscribers-hlsl
Author: Finn Plummer (inbelic)
Changes
Reverts llvm/llvm-project#133302
Reverting to inspect build failures that were introduced from use of the
`clang::Preprocessor`, and the warning about an unused declaration. See linked
issue for failures.
-
https://github.com/inbelic created
https://github.com/llvm/llvm-project/pull/133790
Reverts llvm/llvm-project#133302
Reverting to inspect build failures that were introduced from use of the
`clang::Preprocessor`, and the warning about an unused declaration. See linked
issue for failures.
>Fr
https://github.com/asudarsa approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/133776
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
GeorgeKA wrote:
@mizvekov requesting review please and thanks.
The goal now is simply to let the user know the officially supported version
when the feature is used with C++17.
https://github.com/llvm/llvm-project/pull/133806
___
cfe-commits mailing l
https://github.com/rniwa created
https://github.com/llvm/llvm-project/pull/133804
There are some system libraries such as sqlite3 which forward declare a struct
then use a pointer to that forward declared type in various APIs. Ignore these
types ForwardDeclChecker like other pointer types.
>F
@@ -8444,9 +8444,16 @@ let CategoryName = "Lambda Issue" in {
"C++ standards before C++20">, InGroup, DefaultIgnore;
// C++20 class template argument deduction for alias templates.
- def warn_cxx17_compat_ctad_for_alias_templates : Warning<
- "class template argument d
Sirraide wrote:
ping
https://github.com/llvm/llvm-project/pull/132348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/133806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/133790
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -7353,6 +7354,69 @@ void Sema::CheckCompletedCXXClass(Scope *S,
CXXRecordDecl *Record) {
else if (Record->hasAttr())
checkCUDADeviceBuiltinTextureClassTemplate(*this, Record);
}
+
+ llvm::SmallVector TypeAwareNewDecls;
+ llvm::SmallVector TypeAwareDeleteDecls;
https://github.com/owenca updated
https://github.com/llvm/llvm-project/pull/133673
>From 1ab37d1726be943206c9e1b576468a9d02594783 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Sun, 30 Mar 2025 19:29:39 -0700
Subject: [PATCH 1/2] [clang-format] Correctly annotate pointer/reference in
_Generic
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/133844
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2025-03-31T22:11:06-07:00
New Revision: ee3c892b3570281698170130a435f9c6b32c3ef5
URL:
https://github.com/llvm/llvm-project/commit/ee3c892b3570281698170130a435f9c6b32c3ef5
DIFF:
https://github.com/llvm/llvm-project/commit/ee3c892b3570281698170130a435f9c6b32c3ef5.diff
L
@@ -12071,6 +12071,37 @@ NamespaceDecl *Sema::getOrCreateStdNamespace() {
return getStdNamespace();
}
+/// Check that the template-head of this class template is acceptable for
+/// a declaration of 'std::initializer_list', and optionally diagnose if
+/// it is not.
+/// \re
vgvassilev wrote:
We really need to figure out how to add in-tree tests.
https://github.com/llvm/llvm-project/pull/133037
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
anutosh491 wrote:
> We really need to figure out how to add in-tree tests.
Yupp sadly I was only able to test this through cppinterop and xeus-cpp as I've
commented above
(https://github.com/llvm/llvm-project/pull/133037#discussion_r2015819533) where
it works as expected :\
I shall take out
https://github.com/asudarsa updated
https://github.com/llvm/llvm-project/pull/133797
>From 675595e453b2452e49847ba3b949909367c7942a Mon Sep 17 00:00:00 2001
From: Arvind Sudarsanam
Date: Mon, 31 Mar 2025 13:04:47 -0700
Subject: [PATCH 1/2] [clang-sycl-linker] Replace llvm-link with API calls
T
https://github.com/s-perron approved this pull request.
https://github.com/llvm/llvm-project/pull/133469
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/t-rasmud approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/133755
___
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
`polly-x86_64-linux-shared-plugin` running on `polly-x86_64-gce2` while
building `clang` at step 4 "cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/118/builds/5516
Here is the releva
https://github.com/offsetof created
https://github.com/llvm/llvm-project/pull/133822
Require `std::initializer_list` to be a class template with a template-head
equivalent to `template` and no default arguments.
>From b5798e04281fb6d9475a1ae6af8b94bc0ed85a43 Mon Sep 17 00:00:00 2001
From: offs
@@ -12182,10 +12182,14 @@ QualType Sema::BuildStdInitializerList(QualType
Element, SourceLocation Loc) {
Args.addArgument(TemplateArgumentLoc(TemplateArgument(Element),
Context.getTrivialTypeSourceInfo(Element,
https://github.com/frasercrmck updated
https://github.com/llvm/llvm-project/pull/129679
>From aabdf579b3d88e59f8604943573c9beeafb4cf66 Mon Sep 17 00:00:00 2001
From: Fraser Cormack
Date: Tue, 4 Mar 2025 10:03:00 +
Subject: [PATCH 1/3] [libclc] Move several 'native' builtins to CLC library
https://github.com/pawosm-arm closed
https://github.com/llvm/llvm-project/pull/133578
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Paul Osmialowski
Date: 2025-03-31T21:55:58+01:00
New Revision: cb7c223625f950c5ad14fed39aad8b358874fcf0
URL:
https://github.com/llvm/llvm-project/commit/cb7c223625f950c5ad14fed39aad8b358874fcf0
DIFF:
https://github.com/llvm/llvm-project/commit/cb7c223625f950c5ad14fed39aad8b358874fcf0.di
farzonl wrote:
> @farzonl can you please revert the original commit? It seems to break just
> any LLVM build, so I guess the CI testing of all new PRs should be affected.
I don't see it breaking any build in LLVM, just the amd offload ones. Al the
Premerge tests also passed. I suspect the `AM
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Arvind Sudarsanam (asudarsa)
Changes
This PR has the following changes:
Replace llvm-link with calls to linkInModule to link device files Add
-print-linked-module option to dump linked module for testing Added a test to
verify that linki
dschuff wrote:
Thanks for working on this!
This will be the first time I'm not going to oppose an effort to remove Native
Client support 🎉
Although I am going to ask you to wait a couple of months to land it, until we
finally turn it off for good and start deleting the support code from Chromi
Author: Ryosuke Niwa
Date: 2025-03-31T14:59:41-07:00
New Revision: 6ff33edcdc29be049829934399b2fb2585252439
URL:
https://github.com/llvm/llvm-project/commit/6ff33edcdc29be049829934399b2fb2585252439
DIFF:
https://github.com/llvm/llvm-project/commit/6ff33edcdc29be049829934399b2fb2585252439.diff
@@ -8444,9 +8444,16 @@ let CategoryName = "Lambda Issue" in {
"C++ standards before C++20">, InGroup, DefaultIgnore;
// C++20 class template argument deduction for alias templates.
- def warn_cxx17_compat_ctad_for_alias_templates : Warning<
- "class template argument d
Author: Yaxun (Sam) Liu
Date: 2025-03-31T20:28:29-04:00
New Revision: 0248d277cabab370b48114cc62aff393b273971b
URL:
https://github.com/llvm/llvm-project/commit/0248d277cabab370b48114cc62aff393b273971b
DIFF:
https://github.com/llvm/llvm-project/commit/0248d277cabab370b48114cc62aff393b273971b.dif
https://github.com/yxsamliu closed
https://github.com/llvm/llvm-project/pull/133590
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vgvassilev approved this pull request.
Lgtm!
https://github.com/llvm/llvm-project/pull/127467
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vgvassilev milestoned
https://github.com/llvm/llvm-project/pull/131578
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/petrhosek approved this pull request.
This is definitely an improvement over status quo! Ideally, developers wouldn't
have to list Clang libraries separately from other libraries but that would
require some non-trivial build system refactoring that would be better done in
fo
@@ -10933,27 +11096,100 @@ OverloadCandidateSet::BestViableFunction(Sema &S,
SourceLocation Loc,
// -fgpu-exclude-wrong-side-overloads is off. When
// -fgpu-exclude-wrong-side-overloads is on, all candidates are compared
// uniformly in isBetterOverloadCandidate.
- if (
@@ -1126,7 +1208,8 @@ class Sema;
OperatorRewriteInfo RewriteInfo;
constexpr static unsigned NumInlineBytes =
-24 * sizeof(ImplicitConversionSequence);
+32 * sizeof(ImplicitConversionSequence);
zyn0217 wrote:
Can we add some comments?
@@ -14354,6 +14584,17 @@ ExprResult Sema::BuildOverloadedCallExpr(Scope *S,
Expr *Fn,
OverloadingResult OverloadResult =
CandidateSet.BestViableFunction(*this, Fn->getBeginLoc(), Best);
+ // [C++23][over.call.func]
+ // if overload resolution selects a non-static me
anutosh491 wrote:
> I am wondering if we can add such CI for llvm, too... maybe you can discuss
> that at the llvm discord's infrastructure channel...
Absolutely, I shall do that and get back to you on this (I think there are
people at llvm looking into llvm+emscripten use cases and should sup
@@ -4853,9 +4853,16 @@ void UnwrappedLineParser::readToken(int LevelDifference)
{
PreviousWasComment = FormatTok->is(tok::comment);
while (!Line->InPPDirective && FormatTok->is(tok::hash) &&
- (!Style.isVerilog() ||
-Keywords.isVerilogPPDirective
efriedma-quic wrote:
> CodeGen/Thumb2/pacbti-m-outliner-1.ll
```
diff --git a/llvm/test/CodeGen/Thumb2/pacbti-m-outliner-1.ll
b/llvm/test/CodeGen/Thumb2/pacbti-m-outliner-1.ll
index 23924b2..dc06c26 100644
--- a/llvm/test/CodeGen/Thumb2/pacbti-m-outliner-1.ll
+++ b/llvm/test/CodeGen/Thumb2/pacb
vzakhari wrote:
> > @farzonl can you please revert the original commit? It seems to break just
> > any LLVM build, so I guess the CI testing of all new PRs should be affected.
>
> I don't see it breaking any build in LLVM, just the amd offload ones. Al the
> Premerge tests also passed. I suspe
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Tom Eccles (tblah)
Changes
Precise OpenMP standards support information is being documented in #132707
Flang now has good support for OpenMP Version 3.1 and earlier.
---
Full diff: https://github.com/llvm/llvm-project/pull/133745.diff
8
1 - 100 of 390 matches
Mail list logo
