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 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
