[clang] Fixup test added in #155573 to work when the compiler defaults to C++20. (PR #156166)
via cfe-commits
cfe-commits at lists.llvm.org
Sat Aug 30 01:59:12 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: None (dyung)
<details>
<summary>Changes</summary>
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.
---
Full diff: https://github.com/llvm/llvm-project/pull/156166.diff
1 Files Affected:
- (modified) clang/test/AST/ByteCode/vectors.cpp (+4-4)
``````````diff
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;
``````````
</details>
https://github.com/llvm/llvm-project/pull/156166
More information about the cfe-commits
mailing list