[clang] [dataflow] CXXOperatorCallExpr equal operator might not be a glvalue (PR #80991)

2024-02-07 Thread via 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&

[clang] [dataflow] CXXOperatorCallExpr equal operator might not be a glvalue (PR #80991)

2024-02-07 Thread via cfe-commits
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

[clang] [dataflow] CXXOperatorCallExpr equal operator might not be a glvalue (PR #80991)

2024-02-07 Thread via cfe-commits
@@ -2313,6 +2313,42 @@ TEST(TransferTest, AssignmentOperatorWithInitAndInheritance) { ASTContext ) {}); } +TEST(TransferTest, CXXOperatorCallExprEqualReturnsVoid) { martinboehme wrote: ```suggestion TEST(TransferTest, AssignmentOperatorReturnsVoid)

[clang] [dataflow] CXXOperatorCallExpr equal operator might not be a glvalue (PR #80991)

2024-02-07 Thread via cfe-commits
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

[clang] [llvm] [AMDGPU] Introduce GFX9/10.1/10.3/11 Generic Targets (PR #76955)

2024-02-07 Thread Pierre van Houtryve via 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/6] [AMDGPU] Introduce GFX9/10.1/10.3/11 Generic Targets These

[clang] 5d8a731 - [clang][Interp] Support __builtin_eh_return_data_regno

2024-02-07 Thread Timm Bäder via cfe-commits
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

[clang] [llvm] [mlir] [OpenMP] Migrate GPU Reductions CodeGen from Clang to OMPIRBuilder (PR #80343)

2024-02-07 Thread Akash Banerjee via cfe-commits
@@ -2042,35 +2057,1378 @@ OpenMPIRBuilder::createSection(const LocationDescription , /*IsCancellable*/ true); } -/// Create a function with a unique name and a "void (i8*, i8*)" signature in -/// the given module and return it. -Function

[clang] [llvm] [mlir] [OpenMP] Migrate GPU Reductions CodeGen from Clang to OMPIRBuilder (PR #80343)

2024-02-07 Thread Akash Banerjee via cfe-commits
@@ -2719,10 +4051,24 @@ getKmpcForStaticLoopForType(Type *Ty, OpenMPIRBuilder *OMPBuilder, static void createTargetLoopWorkshareCall( OpenMPIRBuilder *OMPBuilder, WorksharingLoopType LoopType, BasicBlock *InsertBlock, Value *Ident, Value *LoopBodyArg, -Type

[clang] [Clang][TableGen] Add Features to TargetBuiltin (PR #80279)

2024-02-07 Thread Wang Pengcheng via cfe-commits
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

[clang] [llvm] [mlir] [OpenMP] Migrate GPU Reductions CodeGen from Clang to OMPIRBuilder (PR #80343)

2024-02-07 Thread Akash Banerjee via cfe-commits
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

[clang] [NVPTX][AMDGPU][CodeGen] Fix `local_space nullptr` handling for NVPTX and local/private `nullptr` value for AMDGPU. (PR #78759)

2024-02-07 Thread Matt Arsenault via 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 , +

[clang] [NVPTX][AMDGPU][CodeGen] Fix `local_space nullptr` handling for NVPTX and local/private `nullptr` value for AMDGPU. (PR #78759)

2024-02-07 Thread Matt Arsenault via 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 , +

[clang] [llvm] [LinkerWrapper] Support relocatable linking for offloading (PR #80066)

2024-02-07 Thread Joseph Huber via cfe-commits
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

[clang] 5c84054 - [LinkerWrapper] Support relocatable linking for offloading (#80066)

2024-02-07 Thread via cfe-commits
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

[clang] [Clang][Sema] Do not attempt to instantiate a deleted move constructor (PR #80959)

2024-02-07 Thread Erich Keane via cfe-commits
@@ -7894,13 +7895,18 @@ bool Sema::CheckExplicitlyDefaultedSpecialMember(CXXMethodDecl *MD, if (ShouldDeleteForTypeMismatch || ShouldDeleteSpecialMember(MD, CSM)) { if (First) { SetDeclDeleted(MD, MD->getLocation()); - if (!inTemplateInstantiation() &&

[clang] [Clang][Sema] Do not attempt to instantiate a deleted move constructor (PR #80959)

2024-02-07 Thread Aaron Ballman via 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

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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

[clang] 52bf531 - [Clang][Sema] Fix out-of-bounds access (#80978)

2024-02-07 Thread via 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

[clang] [clang-tools-extra] [llvm] [Clang] Fix : More Detailed "No expected directives found" (PR #78338)

2024-02-07 Thread Aaron Ballman via cfe-commits
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

[clang] 4e58f03 - [Clang] Fix : More Detailed "No expected directives found" (#78338)

2024-02-07 Thread via cfe-commits
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

[clang] Turn 'counted_by' into a type attribute and parse it into 'CountAttributedType' (PR #78000)

2024-02-07 Thread Aaron Ballman via cfe-commits
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 >

[clang] [dataflow] CXXOperatorCallExpr equal operator might not be a glvalue (PR #80991)

2024-02-07 Thread via cfe-commits
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

[clang] [dataflow] CXXOperatorCallExpr equal operator might not be a glvalue (PR #80991)

2024-02-07 Thread Paul Semel via cfe-commits
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

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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

[clang] Issue #63106: [сlang] Representation of ellipsis in AST (PR #80976)

2024-02-07 Thread via 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

[clang] [llvm] [LinkerWrapper] Support relocatable linking for offloading (PR #80066)

2024-02-07 Thread Yaxun Liu via 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

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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

[clang] [llvm] [PowerPC] Support local-dynamic TLS relocation on AIX (PR #66316)

2024-02-07 Thread Felix via cfe-commits
@@ -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

[clang] [Clang][TableGen] Add Features to TargetBuiltin (PR #80279)

2024-02-07 Thread Aaron Ballman via cfe-commits
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?

[clang] [dataflow] CXXForRangeStmt should extend flow condition (PR #80989)

2024-02-07 Thread via cfe-commits
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] [dataflow] CXXForRangeStmt should extend flow condition (PR #80989)

2024-02-07 Thread Paul Semel via cfe-commits
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

[clang] [llvm] [PowerPC] Support local-dynamic TLS relocation on AIX (PR #66316)

2024-02-07 Thread Felix via cfe-commits
@@ -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

[clang] [clang] Use CPlusPlus language option instead of Bool (PR #80975)

2024-02-07 Thread Aaron Ballman via cfe-commits
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

[clang] [llvm] [PowerPC] Support local-dynamic TLS relocation on AIX (PR #66316)

2024-02-07 Thread Felix via cfe-commits
@@ -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

[clang] b0c6fc8 - [clang][Interp] Ignore UsingEnumDecls

2024-02-07 Thread Timm Bäder via 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

[clang] [llvm] [LinkerWrapper] Support relocatable linking for offloading (PR #80066)

2024-02-07 Thread Joseph Huber via 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

[clang] [llvm] [PowerPC] Support local-dynamic TLS relocation on AIX (PR #66316)

2024-02-07 Thread Felix via cfe-commits
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

[clang] [clang] Fix OMPT ident flag in combined distribute parallel for pragma (PR #80987)

2024-02-07 Thread via cfe-commits
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 --

[clang] [clang] Fix OMPT ident flag in combined distribute parallel for pragma (PR #80987)

2024-02-07 Thread via cfe-commits
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) -

[clang] [clang] Fix OMPT ident flag in combined distribute parallel for pragma (PR #80987)

2024-02-07 Thread via cfe-commits
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)

[clang] a97ff2d - [clang][Interp] Add missing static_assert message

2024-02-07 Thread Timm Bäder via cfe-commits
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

[clang] 61c7a69 - [clang][Interp] Fix sizeof of reference types

2024-02-07 Thread Timm Bäder via cfe-commits
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

[clang] af6656c - [clang][Interp][NFC] Fix moveArrayTy byte offset

2024-02-07 Thread Timm Bäder via cfe-commits
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

[clang] [clang][analyzer] Add note tags to alpha.unix.BlockInCriticalSection (PR #80029)

2024-02-07 Thread Endre Fülöp via cfe-commits
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

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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] [flang] [llvm] [InstCombine] Canonicalize constant GEPs to i8 source element type (PR #68882)

2024-02-07 Thread Nikita Popov via cfe-commits
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

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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 --

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via 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] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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:

[clang] [Clang][Sema] Fix out-of-bounds access (PR #80978)

2024-02-07 Thread via cfe-commits
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

[clang] [Driver] Allow -fbasic-block-sections for AArch64 ELF (PR #80916)

2024-02-07 Thread Daniel Hoekwater via cfe-commits
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

[clang] Issue #63106: [сlang] Representation of ellipsis in AST (PR #80976)

2024-02-07 Thread via cfe-commits
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 --

[clang] [Driver] Allow -fbasic-block-sections for AArch64 ELF (PR #80916)

2024-02-07 Thread Daniel Hoekwater via cfe-commits
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

[clang] Issue #63106: [сlang] Representation of ellipsis in AST (PR #80976)

2024-02-07 Thread Shahid Iqbal via cfe-commits
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

[clang] [NVPTX][AMDGPU][CodeGen] Fix `local_space nullptr` handling for NVPTX and local/private `nullptr` value for AMDGPU. (PR #78759)

2024-02-07 Thread via 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 ) {} +// CHECK-DAG: define dso_local void @[[RAW_REF:[a-zA-Z0-9_]+]](ptr noundef nonnull align 4 dereferenceable(4) % +void

[clang] [Driver] Allow -fbasic-block-sections for AArch64 ELF (PR #80916)

2024-02-07 Thread Daniel Hoekwater via cfe-commits
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

[clang] [NVPTX][AMDGPU][CodeGen] Fix `local_space nullptr` handling for NVPTX and local/private `nullptr` value for AMDGPU. (PR #78759)

2024-02-07 Thread via cfe-commits
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`

[clang] [scan-build] Use python3 instead of python in scripts. (PR #80977)

2024-02-07 Thread via cfe-commits
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

[clang] [scan-build] Use python3 instead of python in scripts. (PR #80977)

2024-02-07 Thread via cfe-commits
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,

[clang] [scan-build] Use python3 instead of python in scripts. (PR #80977)

2024-02-07 Thread Collin Funk via cfe-commits
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

[clang] Issue #63106: [сlang] Representation of ellipsis in AST (PR #80976)

2024-02-07 Thread via cfe-commits
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

[clang] Issue #63106: [сlang] Representation of ellipsis in AST (PR #80976)

2024-02-07 Thread Shahid Iqbal via cfe-commits
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

[clang] [clang] Use CPlusPlus language option instead of Bool (PR #80975)

2024-02-07 Thread Mariya Podchishchaeva via cfe-commits
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

[clang] [clang] Use CPlusPlus language option instead of Bool (PR #80975)

2024-02-07 Thread via cfe-commits
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

[clang] [clang] Use CPlusPlus language option instead of Bool (PR #80975)

2024-02-07 Thread Mariya Podchishchaeva via cfe-commits
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

[clang] [clang] Match -isysroot behaviour with system compiler on Darwin (PR #80524)

2024-02-07 Thread Liviu Ionescu via cfe-commits
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

[clang] [clang] Add fbasic-block-sections support for AArch64 (PR #80916)

2024-02-07 Thread Daniel Hoekwater via cfe-commits
@@ -5972,6 +5972,15 @@ void Clang::ConstructJob(Compilation , const JobAction , << A->getAsString(Args) << A->getValue(); else A->render(Args, CmdArgs); +} else if (Triple.isAArch64()) { + // "all" is not supported on AArch64 since branch

[clang] [clang] Add fbasic-block-sections support for AArch64 (PR #80916)

2024-02-07 Thread Daniel Hoekwater via cfe-commits
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

[clang] [clang] Add fbasic-block-sections support for AArch64 (PR #80916)

2024-02-07 Thread Daniel Hoekwater via cfe-commits
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

[clang] [clang] Add fbasic-block-sections support for AArch64 (PR #80916)

2024-02-07 Thread Daniel Hoekwater via cfe-commits
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

[clang] [clang] Match -isysroot behaviour with system compiler on Darwin (PR #80524)

2024-02-07 Thread Dmitry Polukhin via cfe-commits
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

[clang] [dataflow] Fix crash when InitListExpr is not a prvalue (PR #80970)

2024-02-07 Thread via cfe-commits
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)

[clang] [dataflow] Fix crash when InitListExpr is not a prvalue (PR #80970)

2024-02-07 Thread Paul Semel via cfe-commits
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 ---

[clang] [Clang][Sema] Do not attempt to instantiate a deleted move constructor (PR #80959)

2024-02-07 Thread via 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

[clang] [Clang][Sema] Do not attempt to instantiate a deleted move constructor (PR #80959)

2024-02-07 Thread via cfe-commits
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

[clang] [Clang][Sema] Do not attempt to instantiate a deleted move constructor (PR #80959)

2024-02-07 Thread via cfe-commits
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

[clang] [Clang][Sema] Do not attempt to instantiate a deleted move constructor (PR #80959)

2024-02-07 Thread via cfe-commits
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

[clang] Allow default arguments to be evaluated like other arguments. (PR #80956)

2024-02-07 Thread Ryosuke Niwa via cfe-commits
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

[clang] Allow default arguments to be evaluated like other arguments. (PR #80956)

2024-02-07 Thread via cfe-commits
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. ---

[clang] Allow default arguments to be evaluated like other arguments. (PR #80956)

2024-02-07 Thread Ryosuke Niwa via 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

[clang] [clang] Add fbasic-block-sections support for AArch64 (PR #80916)

2024-02-07 Thread Fangrui Song via cfe-commits
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

[clang] [llvm] [AMDGPU] Introduce GFX9/10.1/10.3/11 Generic Targets (PR #76955)

2024-02-07 Thread Pierre van Houtryve via 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

[clang] [Clang][Sema] fix crash in codegen stage when an lambda expression declared in an unevaluated context (PR #80802)

2024-02-07 Thread Qizhi Hu via cfe-commits
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

[clang] [llvm] [AMDGPU] Introduce GFX9/10.1/10.3/11 Generic Targets (PR #76955)

2024-02-07 Thread Pierre van Houtryve via 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

[clang] [Clang][Sema] fix crash in codegen stage when an lambda expression declared in an unevaluated context (PR #80802)

2024-02-07 Thread Qizhi Hu via cfe-commits
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

[clang] Diagnose misuse of the cleanup attribute (PR #80040)

2024-02-07 Thread Bhuminjay Soni via cfe-commits
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

[clang] [llvm] [AMDGPU] Introduce GFX9/10.1/10.3/11 Generic Targets (PR #76955)

2024-02-07 Thread Pierre van Houtryve via cfe-commits
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

[clang] [llvm] [clang][MBD] set up module build daemon infrastructure (PR #67562)

2024-02-07 Thread Connor Sughrue via cfe-commits
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

<    1   2   3   4   5