[PATCH] D68449: [test] Remove another two unnecessary uses of REQUIRES: target-windows. NFC.

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 4 00:38:36 PDT 2019


mstorsjo created this revision.
mstorsjo added a reviewer: rnk.
Herald added a subscriber: rupprecht.
Herald added a project: LLVM.
mstorsjo added a child revision: D68450: [lit] Remove setting of the target-windows feature.

Posting this one separately, as I had to revert D68133 <https://reviews.llvm.org/D68133>.


Repository:
  rL LLVM

https://reviews.llvm.org/D68449

Files:
  llvm/test/tools/llvm-symbolizer/coff-dwarf.test
  llvm/test/tools/llvm-symbolizer/coff-exports.test


Index: llvm/test/tools/llvm-symbolizer/coff-exports.test
===================================================================
--- llvm/test/tools/llvm-symbolizer/coff-exports.test
+++ llvm/test/tools/llvm-symbolizer/coff-exports.test
@@ -7,8 +7,6 @@
 
 This test relies on UnDecorateSymbolName, which is Win32-only.
 REQUIRES: system-windows
-REQUIRES: target-windows
-FIXME: This test depends on host, not target.
 
 We get the expected stack trace, except 'foo' appears for the 'bar' frame
 because 'bar' isn't in the export table.
Index: llvm/test/tools/llvm-symbolizer/coff-dwarf.test
===================================================================
--- llvm/test/tools/llvm-symbolizer/coff-dwarf.test
+++ llvm/test/tools/llvm-symbolizer/coff-dwarf.test
@@ -6,7 +6,7 @@
 RUN:     | FileCheck %s
 
 This test relies on UnDecorateSymbolName, which is Windows-only.
-REQUIRES: target-windows, system-windows
+REQUIRES: system-windows
 
 CHECK: foo(void)
 CHECK: coff-dwarf.cpp:7


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68449.223158.patch
Type: text/x-patch
Size: 982 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191004/77a3a185/attachment.bin>


More information about the llvm-commits mailing list