Object: Downgrade invalid weak externals from an assert fail to an llvm::Error when creating an irsymtab.

This fixes bitcode emission for modules containing invalid weak externals.

llvm-svn: 312686
This commit is contained in:
Peter Collingbourne 2017-09-07 01:33:52 +00:00
parent 61f47b81ab
commit 9dbee112d7
2 changed files with 17 additions and 3 deletions

View File

@ -231,11 +231,14 @@ Error Builder::addSymbol(const ModuleSymbolTable &Msymtab,
if ((Flags & object::BasicSymbolRef::SF_Weak) &&
(Flags & object::BasicSymbolRef::SF_Indirect)) {
auto *Fallback = dyn_cast<GlobalValue>(
cast<GlobalAlias>(GV)->getAliasee()->stripPointerCasts());
if (!Fallback)
return make_error<StringError>("Invalid weak external",
inconvertibleErrorCode());
std::string FallbackName;
raw_string_ostream OS(FallbackName);
Msymtab.printSymbolName(
OS, cast<GlobalValue>(
cast<GlobalAlias>(GV)->getAliasee()->stripPointerCasts()));
Msymtab.printSymbolName(OS, Fallback);
OS.flush();
setStr(Uncommon().COFFWeakExternFallbackName, Saver.save(FallbackName));
}

View File

@ -0,0 +1,11 @@
; REQUIRES: x86-registered-target
; RUN: opt -o %t %s
; RUN: not llvm-lto2 dump-symtab %t 2>&1 | FileCheck %s
target datalayout = "e-m:x-p:32:32-i64:64-f80:32-n8:16:32-a:0:32-S32"
target triple = "x86_64-pc-windows-msvc"
@bar = global i32 0
; CHECK: Invalid weak external
@foo = weak alias i32, i32* getelementptr (i32, i32* @bar, i32 1)