diff options
| author | Zachary Turner <zturner@google.com> | 2017-09-16 19:01:04 +0000 |
|---|---|---|
| committer | Zachary Turner <zturner@google.com> | 2017-09-16 19:01:04 +0000 |
| commit | 6326d56195756a9b4ceb7ac97364ecbc09b27c93 (patch) | |
| tree | 96e6ee355e2ee67ab6cbca6320c7d95b8e56ef46 /llvm/test | |
| parent | c3023d1bd19d02fa00b9a09dbcfbea112990b694 (diff) | |
| download | bcm5719-llvm-6326d56195756a9b4ceb7ac97364ecbc09b27c93.tar.gz bcm5719-llvm-6326d56195756a9b4ceb7ac97364ecbc09b27c93.zip | |
[llvm-symbolizer] Fix coff-dwarf.test
This was a bug in the test that was only exposed as a result of
refactoring some code in lit configuration files. Previously,
llvm's lit configuration would only set the target-windows feature
if the system was also windows. Since cross-compilation is
a thing, this isn't correct. target-windows should be set
independently of system-windows.
Adding to that bug, this particular test then checked for
target-windows when it really meant "can I call a certain API on
the host machine", which is what system-windows is for.
Ultimately, this test only works if *both* the target and host
are Windows, so I've updated the test to reflect that.
llvm-svn: 313468
Diffstat (limited to 'llvm/test')
| -rw-r--r-- | llvm/test/tools/llvm-symbolizer/coff-dwarf.test | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/test/tools/llvm-symbolizer/coff-dwarf.test b/llvm/test/tools/llvm-symbolizer/coff-dwarf.test index 9611c891197..50763a6bde6 100644 --- a/llvm/test/tools/llvm-symbolizer/coff-dwarf.test +++ b/llvm/test/tools/llvm-symbolizer/coff-dwarf.test @@ -3,7 +3,7 @@ RUN: | llvm-symbolizer --inlining --relative-address -obj="%p/Inputs/coff-dw RUN: | FileCheck %s This test relies on UnDecorateSymbolName, which is Windows-only. -REQUIRES: target-windows +REQUIRES: target-windows, system-windows ADDR: 0x5009 ADDR: 0x5038 |
