[clang] [Clang] Only check for error in C++20 mode (PR #84624)

via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 9 03:40:19 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: None (Sirraide)

<details>
<summary>Changes</summary>

Fix a test that was added in #<!-- -->81014 and which caused buildbots to fail. Only check for the ‘never produces a constant expression error’ in C++20 mode.

This fixes #<!-- -->84623.

---
Full diff: https://github.com/llvm/llvm-project/pull/84624.diff


1 Files Affected:

- (modified) clang/test/SemaCXX/cxx23-assume.cpp (+2-2) 


``````````diff
diff --git a/clang/test/SemaCXX/cxx23-assume.cpp b/clang/test/SemaCXX/cxx23-assume.cpp
index 2b99cbd3e788a1..2d7c9b174d9019 100644
--- a/clang/test/SemaCXX/cxx23-assume.cpp
+++ b/clang/test/SemaCXX/cxx23-assume.cpp
@@ -65,8 +65,8 @@ static_assert(h(4) == sizeof(int));
 static_assert(__has_cpp_attribute(assume) == 202207L);
 static_assert(__has_attribute(assume));
 
-constexpr bool i() { // expected-error {{never produces a constant expression}}
-  [[assume(false)]]; // expected-note {{assumption evaluated to false}} expected-note {{assumption evaluated to false}} ext-warning {{C++23 extension}}
+constexpr bool i() { // ext-error {{never produces a constant expression}}
+  [[assume(false)]]; // ext-note {{assumption evaluated to false}} expected-note {{assumption evaluated to false}} ext-warning {{C++23 extension}}
   return true;
 }
 

``````````

</details>


https://github.com/llvm/llvm-project/pull/84624


More information about the cfe-commits mailing list