diff options
| author | Jonas Devlieghere <jonas@devlieghere.com> | 2018-07-26 14:16:19 +0000 |
|---|---|---|
| committer | Jonas Devlieghere <jonas@devlieghere.com> | 2018-07-26 14:16:19 +0000 |
| commit | 640e790af2921f17d24b639ce9f0afa23d79ac78 (patch) | |
| tree | ac1eceb2652fb4c27986ea054404a9cbc41b8231 | |
| parent | 66b5e7da4cc91e5e1ebc07158370aa0ab108ceba (diff) | |
| download | bcm5719-llvm-640e790af2921f17d24b639ce9f0afa23d79ac78.tar.gz bcm5719-llvm-640e790af2921f17d24b639ce9f0afa23d79ac78.zip | |
[test] Disable dsymutil update test on windows
Apparently, the issue with dsymutil update functionality on Windows was
that Windows doesn't like dsymutil renaming files that have open handles
to them. This disables the new accelerator test and updates the comment
in the other two test.
We should be able to enable the tests again once we updated the
implementation to use TempFile::keep() to keep the temporary files in
MachOUtils.
A big thank you to Jeremy Morse from Sony for figuring this out and
bringing it to my attention.
llvm-svn: 338030
| -rw-r--r-- | llvm/test/tools/dsymutil/X86/accelerator.test | 4 | ||||
| -rw-r--r-- | llvm/test/tools/dsymutil/X86/update-one-CU.test | 3 | ||||
| -rw-r--r-- | llvm/test/tools/dsymutil/X86/update.test | 3 |
3 files changed, 8 insertions, 2 deletions
diff --git a/llvm/test/tools/dsymutil/X86/accelerator.test b/llvm/test/tools/dsymutil/X86/accelerator.test index 96fc58ee568..906b0e645cf 100644 --- a/llvm/test/tools/dsymutil/X86/accelerator.test +++ b/llvm/test/tools/dsymutil/X86/accelerator.test @@ -1,3 +1,7 @@ +UNSUPPORTED: system-windows +Windows does not like renaming files that have open handles to them. We +need to use TempFile::keep to move them in a portable way. + RUN: dsymutil -accelerator=Dwarf -oso-prepend-path=%p/.. %p/../Inputs/basic.macho.x86_64 -o %t.dwarf.dSYM RUN: dsymutil -accelerator=Apple -oso-prepend-path=%p/.. %p/../Inputs/basic.macho.x86_64 -o %t.apple.dSYM diff --git a/llvm/test/tools/dsymutil/X86/update-one-CU.test b/llvm/test/tools/dsymutil/X86/update-one-CU.test index 0759334a62e..5d36ce7135f 100644 --- a/llvm/test/tools/dsymutil/X86/update-one-CU.test +++ b/llvm/test/tools/dsymutil/X86/update-one-CU.test @@ -1,5 +1,6 @@ UNSUPPORTED: system-windows -Remove UNSUPPORTED once we figure out why this fails on Windows. +Windows does not like renaming files that have open handles to them. We +need to use TempFile::keep to move them in a portable way. RUN: dsymutil -oso-prepend-path=%p/.. %p/../Inputs/objc.macho.x86_64 -o %t.dSYM RUN: dsymutil -update %t.dSYM diff --git a/llvm/test/tools/dsymutil/X86/update.test b/llvm/test/tools/dsymutil/X86/update.test index 93e9b3617a6..cbfff63d6a8 100644 --- a/llvm/test/tools/dsymutil/X86/update.test +++ b/llvm/test/tools/dsymutil/X86/update.test @@ -1,5 +1,6 @@ UNSUPPORTED: system-windows -Remove UNSUPPORTED once we figure out why this fails on Windows. +Windows does not like renaming files that have open handles to them. We +need to use TempFile::keep to move them in a portable way. RUN: rm -rf %t.dir RUN: mkdir -p %t.dir |
