[clang] [Sema] Force x86_64-pc-win32 triple in throw-address-space.cpp (fix #224680) (PR #229996)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 7 23:08:10 PDT 2026
llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang
Author: ʟᴜɴᴇx (im-lunex)
<details>
<summary>Changes</summary>
### Context & Root Cause
* **The Problem:** The test added in `#<!-- -->224680` unconditionally expected diagnostics for `__ptr32` pointers.
* **The Architecture Conflict:** On 32-bit targets (such as the `clang-armv7-2stage` buildbot), `__ptr32` maps to the native pointer size and **does not** receive a non-default address space.
* **The Failure:** Because the non-default address space isn't assigned on 32-bit targets, the expected diagnostic errors did not appear, causing the test suite to fail on non-64-bit architectures.
PR title and description assisted by Google **Gemini**.
---
Full diff: https://github.com/llvm/llvm-project/pull/229996.diff
1 Files Affected:
- (modified) clang/test/Sema/throw-address-space.cpp (+1-1)
``````````diff
diff --git a/clang/test/Sema/throw-address-space.cpp b/clang/test/Sema/throw-address-space.cpp
index 0f085d13ae0cb..623a0b029455b 100644
--- a/clang/test/Sema/throw-address-space.cpp
+++ b/clang/test/Sema/throw-address-space.cpp
@@ -1,4 +1,4 @@
-// RUN: %clang_cc1 -fsyntax-only -fcxx-exceptions -fexceptions -fms-extensions -verify %s
+// RUN: %clang_cc1 -triple x86_64-pc-win32 -fsyntax-only -fcxx-exceptions -fexceptions -fms-extensions -verify %s
// Test for issue (https://github.com/llvm/llvm-project/issues/222931).
// Reject throwing/catching pointers and references whose pointee/referred type
``````````
</details>
https://github.com/llvm/llvm-project/pull/229996
More information about the cfe-commits
mailing list