[PATCH] D141647: [clang][test] Remove unnecessary 'REQUIRES'

Ben Shi via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 12 17:33:15 PST 2023


benshi001 created this revision.
benshi001 added reviewers: aaron.ballman, MaskRay.
Herald added subscribers: kosarev, tpr.
Herald added a project: All.
benshi001 requested review of this revision.
Herald added subscribers: cfe-commits, jacquesguan.
Herald added a project: clang.

The test 'Preprocessor/predefined-arch-macros.c' contains many
target tests than 'amdgpu'. If clang is built without 'amdgpu',
then failures in other targets will not be reported.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D141647

Files:
  clang/test/Preprocessor/predefined-arch-macros.c


Index: clang/test/Preprocessor/predefined-arch-macros.c
===================================================================
--- clang/test/Preprocessor/predefined-arch-macros.c
+++ clang/test/Preprocessor/predefined-arch-macros.c
@@ -1,4 +1,3 @@
-// REQUIRES: amdgpu-registered-target
 // Begin X86/GCC/Linux tests ----------------
 
 // RUN: %clang -march=i386 -m32 -E -dM %s -o - 2>&1 \


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D141647.488818.patch
Type: text/x-patch
Size: 389 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230113/257fb635/attachment.bin>


More information about the cfe-commits mailing list