https://github.com/cpsughrue updated
https://github.com/llvm/llvm-project/pull/67562
>From 03d3310ca300630a94517fa300858d1f2645e843 Mon Sep 17 00:00:00 2001
From: cpsughrue
Date: Sun, 9 Jul 2023 23:19:58 -0400
Subject: [PATCH 1/8] [clang][MBD] set up module build daemon infrastructure
The modu
Pierre-vh wrote:
For the MD changes, it's just to describe the version increment, nothing else.
I think describing is important as the V6 diff already updated the
amdhsa.version.
If amdhsa.version didn't need to change then i need to fix that first, and then
we can remove the V6 MD section
ht
11happy wrote:
I have fixed the merge conflict , will be adding more tests as I am able to
test them locally.
Thank you
https://github.com/llvm/llvm-project/pull/80040
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-
https://github.com/jcsxky edited https://github.com/llvm/llvm-project/pull/80802
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Pierre-vh updated
https://github.com/llvm/llvm-project/pull/76955
>From 616dda8bc9e000e4243ddb8f6b7f4b04f956a620 Mon Sep 17 00:00:00 2001
From: pvanhout
Date: Thu, 4 Jan 2024 14:48:05 +0100
Subject: [PATCH 1/5] [AMDGPU] Introduce GFX9/10.1/10.3/11 Generic Targets
These gener
https://github.com/jcsxky edited https://github.com/llvm/llvm-project/pull/80802
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -520,6 +520,102 @@ Every processor supports every OS ABI (see
:ref:`amdgpu-os`) with the following
=== === = =
=== === ==
+Generic processors allow execution of a single
MaskRay wrote:
Perhaps `[Driver] Allow -fbasic-block-sections for ELF AArch64'
https://github.com/llvm/llvm-project/pull/80916
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rniwa created https://github.com/llvm/llvm-project/pull/80956
This PR aligns the evaluation of default arguments with other kinds of
arguments by extracting the expressions within them as argument values to be
evaluated.
>From 9d3f7377901539abeef949c1b33a99b1278900d8 Mon Sep
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Ryosuke Niwa (rniwa)
Changes
This PR aligns the evaluation of default arguments with other kinds of
arguments by extracting the expressions within them as argument values to be
evaluated.
---
https://github.com/rniwa updated https://github.com/llvm/llvm-project/pull/80956
>From bc390afd3a2e9de6e2a883205ce3862e45d26e06 Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Wed, 7 Feb 2024 00:57:00 -0800
Subject: [PATCH] [alpha.webkit.UncountedCallArgsChecker] Allow default
arguments to be
https://github.com/Sirraide created
https://github.com/llvm/llvm-project/pull/80959
Sema would incorrectly skip diagnosing a malformed use of `= default` on an
implcitly deleted move constructor while performing template instantiation,
even if we were about to then generate a definition of sai
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (Sirraide)
Changes
Sema would incorrectly skip diagnosing a malformed use of `= default` on an
implcitly deleted move constructor while performing template instantiation,
even if we were about to then generate a definition of said mo
Sirraide wrote:
CC @shafik, @erichkeane, @AaronBallman
https://github.com/llvm/llvm-project/pull/80959
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Sirraide wrote:
> which also begs the question as to what change actually caused this to break
Since I’m not sure how that is usually done here, is bisecting a reasonable
approach? I’m asking because rebuilding a project the size of Clang
unfortunately ends up taking a rather substantial amoun
https://github.com/paulsemel created
https://github.com/llvm/llvm-project/pull/80970
None
>From e56ced76e19d57a020030ab41a125012460d08d7 Mon Sep 17 00:00:00 2001
From: Paul Semel
Date: Wed, 7 Feb 2024 10:26:23 +
Subject: [PATCH] [dataflow] Fix crash when InitListExpr is not a prvalue
---
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-analysis
Author: Paul Semel (paulsemel)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/80970.diff
2 Files Affected:
- (modified) clang/lib/Analysis/FlowSensitive/Transfer.cpp (+6)
- (modified) clan
dmpolukhin wrote:
@ldionne if downstream is catching upstream, I would love to discuss rationale
behind ignoring command line option. I asked this question several times and
haven't got answer. Original [diff](https://reviews.llvm.org/D89001) that
introduced this behavior also didn't explain w
https://github.com/dhoekwater updated
https://github.com/llvm/llvm-project/pull/80916
>From ccd5fd925a3629d99c0e12ba4757758f740c1407 Mon Sep 17 00:00:00 2001
From: Daniel Hoekwater
Date: Wed, 7 Feb 2024 01:03:17 +
Subject: [PATCH] [Driver] Allow -fbasic-block-sections for AArch64 ELF
Basic
https://github.com/dhoekwater updated
https://github.com/llvm/llvm-project/pull/80916
>From 36f9de4ab82f4f27131f561417749481eada0f3e Mon Sep 17 00:00:00 2001
From: Daniel Hoekwater
Date: Wed, 7 Feb 2024 01:03:17 +
Subject: [PATCH] [Driver] Allow -fbasic-block-sections for AArch64 ELF
Basic
https://github.com/dhoekwater updated
https://github.com/llvm/llvm-project/pull/80916
>From 18f602b2cc0c603564c17737241bb9f188b965e7 Mon Sep 17 00:00:00 2001
From: Daniel Hoekwater
Date: Wed, 7 Feb 2024 01:03:17 +
Subject: [PATCH] [Driver] Allow -fbasic-block-sections for AArch64 ELF
Basic
@@ -5972,6 +5972,15 @@ void Clang::ConstructJob(Compilation &C, const JobAction
&JA,
<< A->getAsString(Args) << A->getValue();
else
A->render(Args, CmdArgs);
+} else if (Triple.isAArch64()) {
+ // "all" is not supported on AArch64 since branc
ilg-ul wrote:
Sorry, the discussion went too specific for me :-(
But after taking a quick look, my impression is that this change reverses the
bug fixed in https://github.com/llvm/llvm-project/pull/70817.
To recap, that PR preferred the C++ headers available in the toolchain
distribution over
https://github.com/Fznamznon created
https://github.com/llvm/llvm-project/pull/80975
As it was pointed out in https://github.com/llvm/llvm-project/pull/80724, we
should not be checking `getLangOpts().Bool` when determining something related
to logical operators, since it only indicates that bo
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Mariya Podchishchaeva (Fznamznon)
Changes
As it was pointed out in https://github.com/llvm/llvm-project/pull/80724, we
should not be checking `getLangOpts().Bool` when determining something related
to logical operators, since it only indi
Fznamznon wrote:
Not sure about what to put in a release note, and whether it is necessary at
all.
https://github.com/llvm/llvm-project/pull/80975
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/shahidiqbal13 created
https://github.com/llvm/llvm-project/pull/80976
- Raising for preliminary review (forget about formating)
- Fixed the '...' catch all ellipsis in catch stmt
- Will fix the failed testcases once we get agree on this
- I don't think we need to support for t
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Shahid Iqbal (shahidiqbal13)
Changes
- Raising for preliminary review (forget about formating)
- Fixed the '...' catch all ellipsis in catch stmt
- Will fix the failed testcases once we get agree on this
- I don't think we need to support f
https://github.com/collinfunk created
https://github.com/llvm/llvm-project/pull/80977
This was done in some other python files in the following commit:
4c1e487c413810655757ea19d0cf3d4f5c40898e
Without this change these scripts do not run on Debian or Ubuntu systems. See
the referenced patch fo
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be
notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Collin Funk (collinfunk)
Changes
This was done in some other python files in the following commit:
4c1e487c413810655757ea19d0cf3d4f5c40898e
Without this change these scripts do not run on Debian or Ubuntu systems. See
the referenced patch
https://github.com/mmoadeli updated
https://github.com/llvm/llvm-project/pull/78759
>From 286ac8f3ea6aec711827ccab9608b010e78b18cf Mon Sep 17 00:00:00 2001
From: m moadeli
Date: Fri, 19 Jan 2024 18:42:24 +
Subject: [PATCH 1/2] - Address space cast of a `local_space` specialized
`nullptr` i
https://github.com/dhoekwater edited
https://github.com/llvm/llvm-project/pull/80916
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,122 @@
+// RUN: %clang_cc1 -triple nvptx64-nvidia-cuda -fsycl-is-device
-disable-llvm-passes -emit-llvm %s -o - | FileCheck %s
+void bar(int &Data) {}
+// CHECK-DAG: define dso_local void @[[RAW_REF:[a-zA-Z0-9_]+]](ptr noundef
nonnull align 4 dereferenceable(4) %
+voi
https://github.com/shahidiqbal13 updated
https://github.com/llvm/llvm-project/pull/80976
>From b6bfb18e25c111baf6c95a0a4a1c3d667bb25b6d Mon Sep 17 00:00:00 2001
From: Shahid Iqbal
Date: Thu, 16 Nov 2023 11:26:43 -0500
Subject: [PATCH 1/3] TESTING infra
---
clang/NOTES.txt | 2 ++
1 file chang
https://github.com/dhoekwater updated
https://github.com/llvm/llvm-project/pull/80916
>From 3eb66efd6d5a0f027191e2cdd65921f5058220d2 Mon Sep 17 00:00:00 2001
From: Daniel Hoekwater
Date: Wed, 7 Feb 2024 01:03:17 +
Subject: [PATCH] [Driver] Allow -fbasic-block-sections for AArch64 ELF
Basic
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff ec42d547eba5c0ad0bddbecc8902d35383968e78
79c592853c77fb0a96746d860be63d4605ea87e7 --
https://github.com/dhoekwater updated
https://github.com/llvm/llvm-project/pull/80916
>From 7eaf94fefa1250fc8a46982cea8ce99abacae11f Mon Sep 17 00:00:00 2001
From: Daniel Hoekwater
Date: Wed, 7 Feb 2024 01:03:17 +
Subject: [PATCH] [Driver] Allow -fbasic-block-sections for AArch64 ELF
Basic
https://github.com/Sirraide created
https://github.com/llvm/llvm-project/pull/80978
Trying to compile a C-style variadic member function with an explicit object
parameter was crashing in Sema because of an out-of-bounds access.
This fixes #80971.
>From 9c32a0ad9b85328b34545c8ec3a47611ecb4cd72
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (Sirraide)
Changes
Trying to compile a C-style variadic member function with an explicit object
parameter was crashing in Sema because of an out-of-bounds access.
This fixes #80971.
---
Full diff: https://github.com/llvm/llvm-projec
Sirraide wrote:
CC @AaronBallman
https://github.com/llvm/llvm-project/pull/80978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Sirraide updated
https://github.com/llvm/llvm-project/pull/80978
>From 673ecd846857075fd9c9f2a7cf5cb326e23abb06 Mon Sep 17 00:00:00 2001
From: Sirraide
Date: Wed, 7 Feb 2024 12:49:18 +0100
Subject: [PATCH] [Clang][Sema] Fix out-of-bounds access
---
clang/lib/Sema/SemaOverlo
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 390b99743bdd60649414fe470d7a9bacc9992231
9c32a0ad9b85328b34545c8ec3a47611ecb4cd72 --
https://github.com/Sirraide updated
https://github.com/llvm/llvm-project/pull/80978
>From 366651f687eb58ab1775e88da55b1489c118e874 Mon Sep 17 00:00:00 2001
From: Sirraide
Date: Wed, 7 Feb 2024 12:49:18 +0100
Subject: [PATCH] [Clang][Sema] Fix out-of-bounds access
---
clang/lib/Sema/SemaOverlo
Sirraide wrote:
If you’re wondering what the force-pushes are about, first I forgot to run `git
clang-format`, then I was also missing a newline at the end of the test file...
https://github.com/llvm/llvm-project/pull/80978
___
cfe-commits mailing lis
nikic wrote:
@Artem-B I've put up https://github.com/llvm/llvm-project/pull/80983 to address
this issue.
https://github.com/llvm/llvm-project/pull/68882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
https://github.com/Sirraide updated
https://github.com/llvm/llvm-project/pull/80978
>From 366651f687eb58ab1775e88da55b1489c118e874 Mon Sep 17 00:00:00 2001
From: Sirraide
Date: Wed, 7 Feb 2024 12:49:18 +0100
Subject: [PATCH 1/2] [Clang][Sema] Fix out-of-bounds access
---
clang/lib/Sema/SemaOv
https://github.com/gamesh411 updated
https://github.com/llvm/llvm-project/pull/80029
From f7875a7f1ff20f3cf850ce1c23bec6d6c3d88d57 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Endre=20F=C3=BCl=C3=B6p?=
Date: Tue, 30 Jan 2024 11:33:30 +0100
Subject: [PATCH 1/2] [clang][analyzer] Add note tags to
al
Author: Timm Bäder
Date: 2024-02-07T13:07:42+01:00
New Revision: af6656c375b8aa9c9156575f7c0ac678a57070d5
URL:
https://github.com/llvm/llvm-project/commit/af6656c375b8aa9c9156575f7c0ac678a57070d5
DIFF:
https://github.com/llvm/llvm-project/commit/af6656c375b8aa9c9156575f7c0ac678a57070d5.diff
LO
Author: Timm Bäder
Date: 2024-02-07T13:32:07+01:00
New Revision: 61c7a69fa0c020e92b1b10882d5d2957f3b8da21
URL:
https://github.com/llvm/llvm-project/commit/61c7a69fa0c020e92b1b10882d5d2957f3b8da21
DIFF:
https://github.com/llvm/llvm-project/commit/61c7a69fa0c020e92b1b10882d5d2957f3b8da21.diff
LO
Author: Timm Bäder
Date: 2024-02-07T13:41:27+01:00
New Revision: a97ff2d35af0a5c640c20abf4215c08e0e69aa41
URL:
https://github.com/llvm/llvm-project/commit/a97ff2d35af0a5c640c20abf4215c08e0e69aa41
DIFF:
https://github.com/llvm/llvm-project/commit/a97ff2d35af0a5c640c20abf4215c08e0e69aa41.diff
LO
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (mikaoP)
Changes
---
Patch is 1.89 MiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/80987.diff
88 Files Affected:
- (modified) clang/lib/CodeGen/CGOpenMPRuntime.cpp (+1)
- (modified) cl
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: None (mikaoP)
Changes
---
Patch is 1.89 MiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/80987.diff
88 Files Affected:
- (modified) clang/lib/CodeGen/CGOpenMPRuntime.cpp (+1)
- (modi
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 95403b42da0de500f4f86add7a60b0daf8ec98d0
62a2704a725d8fcf81392c42d292d3bb1b27e56c --
https://github.com/orcguru deleted
https://github.com/llvm/llvm-project/pull/66316
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
ping
https://github.com/llvm/llvm-project/pull/80066
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Bäder
Date: 2024-02-07T14:24:37+01:00
New Revision: b0c6fc81fe132f20eed38bd836620dfcb5ac17e9
URL:
https://github.com/llvm/llvm-project/commit/b0c6fc81fe132f20eed38bd836620dfcb5ac17e9
DIFF:
https://github.com/llvm/llvm-project/commit/b0c6fc81fe132f20eed38bd836620dfcb5ac17e9.diff
LO
@@ -145,9 +164,89 @@ namespace {
.addImm(0);
if (IsAIX) {
- // The variable offset and region handle are copied in r4 and r3. The
- // copies are followed by GETtlsADDR32AIX/GETtlsADDR64AIX
https://github.com/AaronBallman commented:
The change in OpenCL behavior looks reasonable to me, but let's wait for
@AnastasiaStulova to have a chance to weigh in.
I think we can skip a release note as this is *almost* an NFC change except for
the OpenCL bits. We could put a release note under
@@ -145,9 +164,89 @@ namespace {
.addImm(0);
if (IsAIX) {
- // The variable offset and region handle are copied in r4 and r3. The
- // copies are followed by GETtlsADDR32AIX/GETtlsADDR64AIX
https://github.com/paulsemel created
https://github.com/llvm/llvm-project/pull/80989
This is needed in order to correctly assume implicit iterator validity in the
iterator checker.
>From b227aa82732cdc087ead5df62077055042d22696 Mon Sep 17 00:00:00 2001
From: Paul Semel
Date: Wed, 7 Feb 2024 1
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Paul Semel (paulsemel)
Changes
This is needed in order to correctly assume implicit iterator validity in the
iterator checker.
---
Full diff: https://github.com/llvm/llvm-project/pull/80989.diff
1 Files Affected:
- (modified) clang/lib
https://github.com/AaronBallman commented:
The changes seem reasonable to me but I'd feel more comfortable if the
functionality was also being used (so that we'd get test coverage verifying its
correctness). Do you think it would be reasonable to include the RISCV changes
as well?
https://git
@@ -145,9 +164,89 @@ namespace {
.addImm(0);
if (IsAIX) {
- // The variable offset and region handle are copied in r4 and r3. The
- // copies are followed by GETtlsADDR32AIX/GETtlsADDR64AIX
https://github.com/cor3ntin approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/80978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yxsamliu approved this pull request.
LGTM. Thanks
https://github.com/llvm/llvm-project/pull/80066
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
Using an extra bit for that in `Decl` (that is the base class of all
declaration), seems a bit heavy handed as bits in that part of the astnode are
precious.
Did you consider storing the EllipsisLoc in CXXCatchStmt ?
https://github.com/llvm/llvm-project/pull/80976
Sirraide wrote:
> LGTM, thanks!
I don’t have commit access, so you or someone else would have to merge it
https://github.com/llvm/llvm-project/pull/80978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
https://github.com/paulsemel created
https://github.com/llvm/llvm-project/pull/80991
Although in a normal implementation the assumption is reasonable, it seems that
some esoteric implementation are not returning a T&. This should be handled
correctly and the values be propagated.
>From 4aad70
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Paul Semel (paulsemel)
Changes
Although in a normal implementation the assumption is reasonable, it seems that
some esoteric implementation are not returning a T&. This should be handled
correctly and the values be propagated.
---
Full d
AaronBallman wrote:
> > One possibility would be to use a type qualifier? It has basically the kind
> > of properties you want... the underlying type is
> > still there, and places that strip qualifiers will automatically do the
> > right thing in a lot of cases. It might require a bit more
> >
Author: Shourya Goel
Date: 2024-02-07T09:10:17-05:00
New Revision: 4e58f03f298eb5c74877942c1c68911341c678bd
URL:
https://github.com/llvm/llvm-project/commit/4e58f03f298eb5c74877942c1c68911341c678bd
DIFF:
https://github.com/llvm/llvm-project/commit/4e58f03f298eb5c74877942c1c68911341c678bd.diff
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/78338
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Sirraide
Date: 2024-02-07T15:12:15+01:00
New Revision: 52bf531630d19e115d30b4ca46f1ef03b9a724c6
URL:
https://github.com/llvm/llvm-project/commit/52bf531630d19e115d30b4ca46f1ef03b9a724c6
DIFF:
https://github.com/llvm/llvm-project/commit/52bf531630d19e115d30b4ca46f1ef03b9a724c6.diff
LOG:
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/80978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
> > which also begs the question as to what change actually caused this to break
>
> Since I’m not sure how that is usually done here, is bisecting a reasonable
> approach? I’m asking because rebuilding a project the size of Clang
> unfortunately ends up taking a rather sub
@@ -7894,13 +7895,18 @@ bool
Sema::CheckExplicitlyDefaultedSpecialMember(CXXMethodDecl *MD,
if (ShouldDeleteForTypeMismatch || ShouldDeleteSpecialMember(MD, CSM)) {
if (First) {
SetDeclDeleted(MD, MD->getLocation());
- if (!inTemplateInstantiation() && !HadErr
Author: Joseph Huber
Date: 2024-02-07T08:20:07-06:00
New Revision: 5c84054223102b00cc0dd343a4023e3c6cba42b2
URL:
https://github.com/llvm/llvm-project/commit/5c84054223102b00cc0dd343a4023e3c6cba42b2
DIFF:
https://github.com/llvm/llvm-project/commit/5c84054223102b00cc0dd343a4023e3c6cba42b2.diff
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/80066
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -285,6 +289,20 @@ void
NVPTXTargetCodeGenInfo::addNVVMMetadata(llvm::GlobalValue *GV,
bool NVPTXTargetCodeGenInfo::shouldEmitStaticExternCAliases() const {
return false;
}
+
+llvm::Constant *
+NVPTXTargetCodeGenInfo::getNullPointer(const CodeGen::CodeGenModule &CGM,
+
@@ -285,6 +289,20 @@ void
NVPTXTargetCodeGenInfo::addNVVMMetadata(llvm::GlobalValue *GV,
bool NVPTXTargetCodeGenInfo::shouldEmitStaticExternCAliases() const {
return false;
}
+
+llvm::Constant *
+NVPTXTargetCodeGenInfo::getNullPointer(const CodeGen::CodeGenModule &CGM,
+
TIFitis wrote:
> There is some code that is related to Flang/MLIR that shouldn't be in the
> patch. e.g. ReductionInfoManager is not used anywhere. I am currently working
> on a patch that refactors this code a bit and how it is used in the MLIR
> lowering.
Thanks for pointing this out, I've
wangpc-pp wrote:
> The changes seem reasonable to me but I'd feel more comfortable if the
> functionality was also being used (so that we'd get test coverage verifying
> its correctness). Do you think it would be reasonable to include the RISCV
> changes as well?
Yeah, I separated RISCV chang
@@ -2719,10 +4051,24 @@ getKmpcForStaticLoopForType(Type *Ty, OpenMPIRBuilder
*OMPBuilder,
static void createTargetLoopWorkshareCall(
OpenMPIRBuilder *OMPBuilder, WorksharingLoopType LoopType,
BasicBlock *InsertBlock, Value *Ident, Value *LoopBodyArg,
-Type *Parall
@@ -2042,35 +2057,1378 @@ OpenMPIRBuilder::createSection(const
LocationDescription &Loc,
/*IsCancellable*/ true);
}
-/// Create a function with a unique name and a "void (i8*, i8*)" signature in
-/// the given module and return it.
-Function *get
Author: Timm Bäder
Date: 2024-02-07T15:29:43+01:00
New Revision: 5d8a7318b2192ce8e396ec420f3f4d8f1a07cc10
URL:
https://github.com/llvm/llvm-project/commit/5d8a7318b2192ce8e396ec420f3f4d8f1a07cc10
DIFF:
https://github.com/llvm/llvm-project/commit/5d8a7318b2192ce8e396ec420f3f4d8f1a07cc10.diff
LO
https://github.com/Pierre-vh updated
https://github.com/llvm/llvm-project/pull/76955
>From 616dda8bc9e000e4243ddb8f6b7f4b04f956a620 Mon Sep 17 00:00:00 2001
From: pvanhout
Date: Thu, 4 Jan 2024 14:48:05 +0100
Subject: [PATCH 1/6] [AMDGPU] Introduce GFX9/10.1/10.3/11 Generic Targets
These gener
https://github.com/martinboehme edited
https://github.com/llvm/llvm-project/pull/80991
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2313,6 +2313,42 @@ TEST(TransferTest,
AssignmentOperatorWithInitAndInheritance) {
ASTContext &ASTCtx) {});
}
+TEST(TransferTest, CXXOperatorCallExprEqualReturnsVoid) {
martinboehme wrote:
```suggestion
TEST(TransferTest, AssignmentOperatorReturn
https://github.com/martinboehme requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/80991
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -535,7 +535,19 @@ class TransferVisitor : public
ConstStmtVisitor {
return;
copyRecord(*LocSrc, *LocDst, Env);
- Env.setStorageLocation(*S, *LocDst);
+
+ // If the expr is a glvalue, we can reasonably assume the operator is
+ // returning T& an
@@ -2313,6 +2313,42 @@ TEST(TransferTest,
AssignmentOperatorWithInitAndInheritance) {
ASTContext &ASTCtx) {});
}
+TEST(TransferTest, CXXOperatorCallExprEqualReturnsVoid) {
+ // This is a crash repro.
+ std::string Code = R"(
+struct B {
+ void operator=(B&
@@ -2313,6 +2313,42 @@ TEST(TransferTest,
AssignmentOperatorWithInitAndInheritance) {
ASTContext &ASTCtx) {});
}
+TEST(TransferTest, CXXOperatorCallExprEqualReturnsVoid) {
+ // This is a crash repro.
+ std::string Code = R"(
+struct B {
martinb
@@ -535,7 +535,19 @@ class TransferVisitor : public
ConstStmtVisitor {
return;
copyRecord(*LocSrc, *LocDst, Env);
- Env.setStorageLocation(*S, *LocDst);
+
+ // If the expr is a glvalue, we can reasonably assume the operator is
+ // returning T& an
@@ -2313,6 +2313,42 @@ TEST(TransferTest,
AssignmentOperatorWithInitAndInheritance) {
ASTContext &ASTCtx) {});
}
+TEST(TransferTest, CXXOperatorCallExprEqualReturnsVoid) {
+ // This is a crash repro.
+ std::string Code = R"(
+struct B {
+ void operator=(B&
@@ -3780,6 +3780,30 @@ static void handleCleanupAttr(Sema &S, Decl *D, const
ParsedAttr &AL) {
<< NI.getName() << ParamTy << Ty;
return;
}
+ VarDecl *VD = cast(D);
+ // Create a reference to the variable declaration. This is a fake/dummy
+ // reference.
+ DeclR
@@ -3780,6 +3780,30 @@ static void handleCleanupAttr(Sema &S, Decl *D, const
ParsedAttr &AL) {
<< NI.getName() << ParamTy << Ty;
return;
}
+ VarDecl *VD = cast(D);
+ // Create a reference to the variable declaration. This is a fake/dummy
+ // reference.
+ DeclR
@@ -3780,6 +3780,30 @@ static void handleCleanupAttr(Sema &S, Decl *D, const
ParsedAttr &AL) {
<< NI.getName() << ParamTy << Ty;
return;
}
+ VarDecl *VD = cast(D);
+ // Create a reference to the variable declaration. This is a fake/dummy
+ // reference.
+ DeclR
@@ -3780,6 +3780,30 @@ static void handleCleanupAttr(Sema &S, Decl *D, const
ParsedAttr &AL) {
<< NI.getName() << ParamTy << Ty;
return;
}
+ VarDecl *VD = cast(D);
+ // Create a reference to the variable declaration. This is a fake/dummy
+ // reference.
+ DeclR
https://github.com/Zonotora updated
https://github.com/llvm/llvm-project/pull/75481
>From 3b0af99a0e13ed9d6b8c0365a9fa7ea68ec5 Mon Sep 17 00:00:00 2001
From: Zonotora
Date: Sat, 16 Dec 2023 19:33:21 +0100
Subject: [PATCH 1/2] [clang] Extract negativity check lambda to function
---
clang/l
1 - 100 of 491 matches
Mail list logo
