llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Hans Wennborg (zmodem)

<details>
<summary>Changes</summary>

On targets which don't support `thread_local`, such as 
`arm64-apple-darwin-unknown`, the test would fail with a bunch of `thread-local 
storage is not supported for the current target` messages. Follow-up to #<!-- 
-->197732

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


1 Files Affected:

- (modified) clang/test/CXX/drs/cwg7xx.cpp (+1-1) 


``````````diff
diff --git a/clang/test/CXX/drs/cwg7xx.cpp b/clang/test/CXX/drs/cwg7xx.cpp
index 83ea88157ef27..36f176f9a4ed5 100644
--- a/clang/test/CXX/drs/cwg7xx.cpp
+++ b/clang/test/CXX/drs/cwg7xx.cpp
@@ -100,7 +100,7 @@ static_assert(!is_volatile<void()volatile&>::value, "");
 } // namespace cwg713
 
 namespace cwg717 { // cwg717: 3.3
-#if __cplusplus >= 201103L
+#if __cplusplus >= 201103L && __has_feature(cxx_thread_local)
 void f() {
   thread_local extern int i;
   thread_local extern int& j;

``````````

</details>


https://github.com/llvm/llvm-project/pull/197930
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to