[clang] Add REQUIRES: x86-registered-target to test added in #173311. (PR #178271)

via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 27 10:24:52 PST 2026


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: None (dyung)

<details>
<summary>Changes</summary>

PR #<!-- -->173311 added a test `clang/test/CodeGen/stack-protector-vars.cpp` that fails if the x86 backend is not built. This adds the proper `REQUIRES` line to fix that.

Should fix the test failure on buildbots that do not build the x86 backend such as https://lab.llvm.org/buildbot/#/builders/190/builds/35171 and https://lab.llvm.org/buildbot/#/builders/154/builds/26976.

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


1 Files Affected:

- (modified) clang/test/CodeGen/stack-protector-vars.cpp (+2) 


``````````diff
diff --git a/clang/test/CodeGen/stack-protector-vars.cpp b/clang/test/CodeGen/stack-protector-vars.cpp
index d11cd822cd871..a8869ef9a9712 100644
--- a/clang/test/CodeGen/stack-protector-vars.cpp
+++ b/clang/test/CodeGen/stack-protector-vars.cpp
@@ -3,6 +3,8 @@
 // RUN: %clang -target x86_64-apple-darwin -emit-llvm -S -o - %s -fstack-protector-all | FileCheck %s
 // RUN: %clang -target x86_64-apple-darwin -Xclang -verify -fstack-protector-all %s -o %t -c
 
+// REQUIRES: x86-reigstered-target
+
 typedef __SIZE_TYPE__ size_t;
 
 int printf(const char * _Format, ...);

``````````

</details>


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


More information about the cfe-commits mailing list