https://github.com/erichkeane approved this pull request.
Perfect, thanks! Whoever notices that CI is happy can merge this. Remind us
if you notice/cannot.
https://github.com/llvm/llvm-project/pull/133806
___
cfe-commits mailing list
cfe-commits@lis
@@ -9920,7 +9920,9 @@ QualType
Sema::DeduceTemplateSpecializationFromInitializer(
if (auto *AliasTemplate = dyn_cast_or_null(
TemplateName.getAsTemplateDecl())) {
Diag(Kind.getLocation(),
- diag::warn_cxx17_compat_ctad_for_alias_templates);
+
@@ -9920,7 +9920,9 @@ QualType
Sema::DeduceTemplateSpecializationFromInitializer(
if (auto *AliasTemplate = dyn_cast_or_null(
TemplateName.getAsTemplateDecl())) {
Diag(Kind.getLocation(),
- diag::warn_cxx17_compat_ctad_for_alias_templates);
+
https://github.com/GeorgeKA updated
https://github.com/llvm/llvm-project/pull/133806
>From dd978982b2ab41d3d2a55abb448e653a80158ecd Mon Sep 17 00:00:00 2001
From: George Asante
Date: Mon, 31 Mar 2025 17:41:20 -0400
Subject: [PATCH 1/4] Add warning message for C++17 alias template CTAD
---
cla
Sirraide wrote:
(grep for `DiagCompat()` if you’re unsure as to what to do)
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/cor3ntin closed
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
GeorgeKA wrote:
> thanks. will you need us to merge that for you?
Yes, if you wouldn't mind @cor3ntin . The checks have passed
https://github.com/llvm/llvm-project/pull/133806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.
erichkeane wrote:
Please dont' merge this until it is rebased on top of @Sirrade's patch and the
SemaInit.cpp uses the new interface instead.
https://github.com/llvm/llvm-project/pull/133806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
https://github.com/GeorgeKA 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/GeorgeKA updated
https://github.com/llvm/llvm-project/pull/133806
>From dd978982b2ab41d3d2a55abb448e653a80158ecd Mon Sep 17 00:00:00 2001
From: George Asante
Date: Mon, 31 Mar 2025 17:41:20 -0400
Subject: [PATCH 1/3] Add warning message for C++17 alias template CTAD
---
cla
@@ -9920,7 +9920,9 @@ QualType
Sema::DeduceTemplateSpecializationFromInitializer(
if (auto *AliasTemplate = dyn_cast_or_null(
TemplateName.getAsTemplateDecl())) {
Diag(Kind.getLocation(),
- diag::warn_cxx17_compat_ctad_for_alias_templates);
+
@@ -9920,7 +9920,9 @@ QualType
Sema::DeduceTemplateSpecializationFromInitializer(
if (auto *AliasTemplate = dyn_cast_or_null(
TemplateName.getAsTemplateDecl())) {
Diag(Kind.getLocation(),
- diag::warn_cxx17_compat_ctad_for_alias_templates);
+
https://github.com/zyn0217 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
13 matches
Mail list logo