diff options
| author | bors <bors@rust-lang.org> | 2018-12-17 04:18:14 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2018-12-17 04:18:14 +0000 |
| commit | 63f8e6e12b3f9655fb58282e7a75f411b7e8b4ee (patch) | |
| tree | 75450095f0d6357635b763ac09087285226ee575 /src/rustllvm/ArchiveWrapper.cpp | |
| parent | c6fb01d62916afa0c489670b665a39d1fe903a4a (diff) | |
| parent | 6c2d704950a5f05b8ee5468df53d8db9b94e2028 (diff) | |
| download | rust-63f8e6e12b3f9655fb58282e7a75f411b7e8b4ee.tar.gz rust-63f8e6e12b3f9655fb58282e7a75f411b7e8b4ee.zip | |
Auto merge of #56642 - nikic:llvm-6, r=alexcrichton
Bump minimum required LLVM version to 6.0 Based on the discussion in #55842, while the overall position of Rust wrt LLVM continues to be contentious, there does seem to be a consensus that there is no need for continued support of LLVM 5. This PR bumps our version requirement to LLVM 6.0 and makes Travis run against that. I hope that this is going to unblock #52694. If I understand correctly, while this issue still exists in LLVM 6, Ubuntu has backported the relevant patch. r? @alexcrichton
Diffstat (limited to 'src/rustllvm/ArchiveWrapper.cpp')
| -rw-r--r-- | src/rustllvm/ArchiveWrapper.cpp | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/rustllvm/ArchiveWrapper.cpp b/src/rustllvm/ArchiveWrapper.cpp index 1355c0377ce..2f8dc0d466f 100644 --- a/src/rustllvm/ArchiveWrapper.cpp +++ b/src/rustllvm/ArchiveWrapper.cpp @@ -216,16 +216,11 @@ LLVMRustWriteArchive(char *Dst, size_t NumMembers, Members.push_back(std::move(*MOrErr)); } } + auto Result = writeArchive(Dst, Members, WriteSymbtab, Kind, true, false); -#if LLVM_VERSION_GE(6, 0) if (!Result) return LLVMRustResult::Success; LLVMRustSetLastError(toString(std::move(Result)).c_str()); -#else - if (!Result.second) - return LLVMRustResult::Success; - LLVMRustSetLastError(Result.second.message().c_str()); -#endif return LLVMRustResult::Failure; } |
