[clang] [Clang][ASTMatcher] Add a matcher for the name of a DependentScopeDeclRefExpr (PR #121656)

2025-01-05 Thread Amr Hesham via cfe-commits
https://github.com/AmrDeveloper updated https://github.com/llvm/llvm-project/pull/121656 >From 4cd48886384570e0e5bb0b712f4fd45f8decd27e Mon Sep 17 00:00:00 2001 From: AmrDeveloper Date: Sat, 4 Jan 2025 17:36:05 +0100 Subject: [PATCH 1/3] [Clang][ASTMatcher] Add a matcher for the name of a Depe

[clang] [clang-format] Add LT_RequiresExpression and LT_SimpleRequirement (PR #121681)

2025-01-05 Thread Owen Pan via cfe-commits
https://github.com/owenca updated https://github.com/llvm/llvm-project/pull/121681 >From 14e26d9bb27724705e51fc134a4f3df2a09807e6 Mon Sep 17 00:00:00 2001 From: Owen Pan Date: Sat, 4 Jan 2025 23:42:38 -0800 Subject: [PATCH] [clang-format] Add LT_RequiresExpression and LT_SimpleRequirement The

[clang] [clang] Add test for CWG192 "Name lookup in parameters" (PR #121679)

2025-01-05 Thread via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/121679 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Add test for CWG156 "Name lookup for conversion functions" (PR #121654)

2025-01-05 Thread via cfe-commits
cor3ntin wrote: > To do that, we need to downgrade CWG from being available in Clang 3.2 to > "partial", and merge in at least the test that doesn't pass. Maybe we should > do that anyway. I'd say so, yes https://github.com/llvm/llvm-project/pull/121654 ___

[clang] [Clang] disallow attributes after namespace identifier (PR #121614)

2025-01-05 Thread via cfe-commits
@@ -100,7 +103,7 @@ Parser::DeclGroupPtrTy Parser::ParseNamespace(DeclaratorContext Context, } while (MoreToParse); }; - ReadAttributes(); + ReadAttributes(/*TrailingAttrs*/ false); cor3ntin wrote: Sorry for the misunderstanding. `CheckProhibitedCXX1

[clang] [clang] Fix implicit integer conversion for opaque enum declarations in class templates (PR #121039)

2025-01-05 Thread via cfe-commits
=?utf-8?q?André?= Brand Message-ID: In-Reply-To: https://github.com/thebrandre updated https://github.com/llvm/llvm-project/pull/121039 >From 974fc45ff52e15d42566f5a36c07c8c3c17a920d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A9=20Brand?= Date: Sun, 5 Jan 2025 14:48:40 +0100 Subject: [

[clang] [clang] Fix implicit integer conversion for opaque enums declared in class templates (PR #121039)

2025-01-05 Thread via cfe-commits
=?utf-8?q?André?= Brand Message-ID: In-Reply-To: https://github.com/thebrandre edited https://github.com/llvm/llvm-project/pull/121039 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add GCC's __builtin_stack_address() to Clang. (PR #121332)

2025-01-05 Thread via cfe-commits
https://github.com/aalhwc updated https://github.com/llvm/llvm-project/pull/121332 >From f247c1ab9fa89ca09476ed0a398a4c4385e75193 Mon Sep 17 00:00:00 2001 From: aalhwc Date: Mon, 30 Dec 2024 05:03:59 -0500 Subject: [PATCH 1/3] [Clang] Add GCC's __builtin_stack_address() to Clang (#82632). Thi

[clang] [clang] Add test for CWG156 "Name lookup for conversion functions" (PR #121654)

2025-01-05 Thread via cfe-commits
@@ -922,6 +922,49 @@ namespace cwg155 { // cwg155: dup 632 // expected-warning@-1 {{braces around scalar initializer}} } +namespace cwg156 { // cwg156: partial +namespace ex1 { +struct A { + operator int(); +} a; +void foo() { + typedef int T; + a.operator T(); // T is fo

[clang] [clang] Add test for CWG156 "Name lookup for conversion functions" (PR #121654)

2025-01-05 Thread via cfe-commits
https://github.com/cor3ntin edited https://github.com/llvm/llvm-project/pull/121654 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Add test for CWG156 "Name lookup for conversion functions" (PR #121654)

2025-01-05 Thread via cfe-commits
https://github.com/cor3ntin commented: I wonder if we just mark that as superseded and/or N/A. The original requirements that lookup must find "the same thing" everywhere is gone so in effect we will never support the behavior that this issue is describing https://github.com/llvm/llvm-project/

[clang] [clang] Add test for CWG156 "Name lookup for conversion functions" (PR #121654)

2025-01-05 Thread Vlad Serebrennikov via cfe-commits
@@ -922,6 +922,49 @@ namespace cwg155 { // cwg155: dup 632 // expected-warning@-1 {{braces around scalar initializer}} } +namespace cwg156 { // cwg156: partial +namespace ex1 { +struct A { + operator int(); +} a; +void foo() { + typedef int T; + a.operator T(); // T is fo

[clang] [Clang] disallow attributes after namespace identifier (PR #121614)

2025-01-05 Thread via cfe-commits
@@ -100,7 +103,7 @@ Parser::DeclGroupPtrTy Parser::ParseNamespace(DeclaratorContext Context, } while (MoreToParse); }; - ReadAttributes(); + ReadAttributes(/*TrailingAttrs*/ false); cor3ntin wrote: I would just call `CheckProhibitedCXX11Attribute` he

[clang] [Clang] disallow attributes after namespace identifier (PR #121614)

2025-01-05 Thread via cfe-commits
@@ -81,7 +81,7 @@ Parser::DeclGroupPtrTy Parser::ParseNamespace(DeclaratorContext Context, ParsedAttributes attrs(AttrFactory); - auto ReadAttributes = [&] { + auto ReadAttributes = [&](bool TrailingAttrs) { cor3ntin wrote: Pre-existing, but why is this

[clang] [clang] Add test for CWG156 "Name lookup for conversion functions" (PR #121654)

2025-01-05 Thread Vlad Serebrennikov via cfe-commits
Endilll wrote: > I wonder if we just mark that as superseded and/or N/A. To do that, we need to downgrade CWG from being available in Clang 3.2 to "partial", and merge in at least the test that doesn't pass. Maybe we should do that anyway. > The original requirements that lookup must find

[clang] [Clang][ASTMatcher] Add a matcher for the name of a DependentScopeDeclRefExpr (PR #121656)

2025-01-05 Thread Amr Hesham via cfe-commits
@@ -2235,6 +2235,21 @@ TEST_P(ASTMatchersTest, ArgumentCountIs_CXXConstructExpr) { Constructor1Arg)); } +TEST_P(ASTMatchersTest, hasDependentName_DependentScopeDeclRefExpr) { + if (!GetParam().isCXX() || GetParam().hasDelayedTemplateParsing()) { +// FIXM

[clang] [Clang][ASTMatcher] Add a matcher for the name of a DependentScopeDeclRefExpr (PR #121656)

2025-01-05 Thread Amr Hesham via cfe-commits
@@ -2235,6 +2235,21 @@ TEST_P(ASTMatchersTest, ArgumentCountIs_CXXConstructExpr) { Constructor1Arg)); } +TEST_P(ASTMatchersTest, hasDependentName_DependentScopeDeclRefExpr) { AmrDeveloper wrote: Done https://github.com/llvm/llvm-project/pul

[clang] Patch series to reapply #118734 and substantially improve it (PR #120534)

2025-01-05 Thread Chandler Carruth via cfe-commits
chandlerc wrote: @dyung -- Ok, my latest attempt to work around these MSVC issues is now pushed to this PR. It also contains a commit of specifically debugging hacks to try and extract more information from any failure here. If you could try doing another build with the latest commit ([2ec750

<    1   2