diff options
| author | NAKAMURA Takumi <geek4civic@gmail.com> | 2013-11-08 12:35:56 +0000 |
|---|---|---|
| committer | NAKAMURA Takumi <geek4civic@gmail.com> | 2013-11-08 12:35:56 +0000 |
| commit | 6af8781854a15eadb9df6b45f27fa5cceda5eb16 (patch) | |
| tree | 234c7a0cf19a4e30c2d35d149cdca38f6fff99be /tools/llvm-ar | |
| parent | de712386cdde314ee18ea44b733d48a30d63de10 (diff) | |
| download | external_llvm-6af8781854a15eadb9df6b45f27fa5cceda5eb16.zip external_llvm-6af8781854a15eadb9df6b45f27fa5cceda5eb16.tar.gz external_llvm-6af8781854a15eadb9df6b45f27fa5cceda5eb16.tar.bz2 | |
llvm-ar: Let opening a directory failed in llvm-ar.
Linux cannot open directories with open(2), although cygwin and *bsd can.
Motivation: The test, Object/directory.ll, had been failing with --target=cygwin on Linux. XFAIL was improper for host issues.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@194257 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'tools/llvm-ar')
| -rw-r--r-- | tools/llvm-ar/llvm-ar.cpp | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/tools/llvm-ar/llvm-ar.cpp b/tools/llvm-ar/llvm-ar.cpp index 64ef3fa..d70db72 100644 --- a/tools/llvm-ar/llvm-ar.cpp +++ b/tools/llvm-ar/llvm-ar.cpp @@ -782,6 +782,13 @@ static void performWriteOperation(ArchiveOperation Operation, sys::fs::file_status Status; failIfError(sys::fs::status(FD, Status), FileName); + // Opening a directory doesn't make sense. Let it failed. + // Linux cannot open directories with open(2), although + // cygwin and *bsd can. + if (Status.type() == sys::fs::file_type::directory_file) + failIfError(error_code(errc::is_a_directory, posix_category()), + FileName); + OwningPtr<MemoryBuffer> File; failIfError(MemoryBuffer::getOpenFile(FD, FileName, File, Status.getSize(), false), |