https://github.com/dyung created 
https://github.com/llvm/llvm-project/pull/156166

The test added in #155573 assumes the compiler defaults to the current default 
of C++17. If the compiler is changed to default to C++20, the test fails 
because the expected warnings about a construct being a C++20 extension are no 
longer emitted. This change fixes up the test to work in either C++17 or C++20 
mode by disabling the warning and removing the check for it as this is not what 
is being tested here.

>From 6ebb26468bfda2c80fab6af66737c6705194c088 Mon Sep 17 00:00:00 2001
From: Douglas Yung <douglas.y...@sony.com>
Date: Sat, 30 Aug 2025 08:54:59 +0000
Subject: [PATCH] Fixup test added in #155573 to work when the compiler
 defaults to C++20.

---
 clang/test/AST/ByteCode/vectors.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/clang/test/AST/ByteCode/vectors.cpp 
b/clang/test/AST/ByteCode/vectors.cpp
index 6b41c8d79d5b4..91fec8f86f613 100644
--- a/clang/test/AST/ByteCode/vectors.cpp
+++ b/clang/test/AST/ByteCode/vectors.cpp
@@ -1,5 +1,5 @@
-// RUN: %clang_cc1 -fexperimental-new-constant-interpreter 
-verify=expected,both -flax-vector-conversions=none %s
-// RUN: %clang_cc1 -verify=ref,both -flax-vector-conversions=none %s
+// RUN: %clang_cc1 -Wno-c++20-extensions 
-fexperimental-new-constant-interpreter -verify=expected,both 
-flax-vector-conversions=none %s
+// RUN: %clang_cc1 -Wno-c++20-extensions -verify=ref,both 
-flax-vector-conversions=none %s
 
 typedef int __attribute__((vector_size(16))) VI4;
 constexpr VI4 A = {1,2,3,4};
@@ -147,7 +147,7 @@ namespace {
 namespace Assign {
   constexpr int a2() {
       VI a = {0, 0, 0, 0};
-      VI b; // both-warning {{C++20 extension}}
+      VI b;
 
       b = {1,1,1,1};
       return b[0] + b[1] + b[2] + b[3];
@@ -161,7 +161,7 @@ namespace Assign {
 
   constexpr bool invalid() {
     v2int16_t a = {0, 0};
-    v2int_t b; // both-warning {{C++20 extension}}
+    v2int_t b;
     b = a; // both-error {{incompatible type}}
 
     return true;

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to