[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