[PATCH] D113870: Fix missing REQURIES for Tests in D108261
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Nov 14 21:25:12 PST 2021
dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.
Please move the tests into the relevant subdirectories that implicitly require the relevant target, rather than adding individual REQUIRES clauses.
================
Comment at: llvm/test/DebugInfo/debugline-endsequence.s:3
+# REQUIRES: x86-registered-target
+
----------------
This test could be moved into the `llvm/test/DebugInfo/X86` directory instead, which implies that requires clause for all tests there.
The other test can go in `llvm/test/DebugInfo/AArch64`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113870/new/
https://reviews.llvm.org/D113870
More information about the llvm-commits
mailing list