Simplify the version check

Address the review comments by simplifying the version check to
just "< 8".
This commit is contained in:
Tom Tromey 2019-01-22 11:44:23 -07:00
parent 1c95f5a34c
commit 9452a8dfa3
1 changed files with 3 additions and 4 deletions

View File

@ -1164,11 +1164,10 @@ fn use_enum_fallback(cx: &CodegenCx) -> bool {
// On MSVC we have to use the fallback mode, because LLVM doesn't
// lower variant parts to PDB.
return cx.sess().target.target.options.is_like_msvc
|| llvm_util::get_major_version() < 7
// LLVM version 7 did not release with an important bug fix;
// but the required patch is in the equivalent Rust LLVM.
// See https://github.com/rust-lang/rust/issues/57762.
|| (llvm_util::get_major_version() == 7 && unsafe { !llvm::LLVMRustIsRustLLVM() });
// but the required patch is in the LLVM 8. Rust LLVM reports
// 8 as well.
|| llvm_util::get_major_version() < 8;
}
// Describes the members of an enum value: An enum is described as a union of