forked from OSchip/llvm-project
Re-commit r231545: PECOFF: Do not add extraneous symbols to the dead strip root
That commit was reverted in r231582 as it was a culprit for buildbot breakage. Turned out it's not. llvm-svn: 231610
This commit is contained in:
parent
6d7cb4668f
commit
04597ac258
|
@ -375,8 +375,10 @@ void FileCOFF::beforeLink() {
|
|||
|
||||
// Add /INCLUDE'ed symbols to the file as if they existed in the
|
||||
// file as undefined symbols.
|
||||
for (StringRef sym : undefSyms)
|
||||
for (StringRef sym : undefSyms) {
|
||||
addUndefinedSymbol(sym);
|
||||
_ctx.addDeadStripRoot(sym);
|
||||
}
|
||||
|
||||
// One can define alias symbols using /alternatename:<sym>=<sym> option.
|
||||
// The mapping for /alternatename is in the context object. This helper
|
||||
|
@ -387,10 +389,6 @@ void FileCOFF::beforeLink() {
|
|||
// SEH. Disable SEH if the file being read is not compatible.
|
||||
if (!isCompatibleWithSEH())
|
||||
_ctx.setSafeSEH(false);
|
||||
|
||||
if (_ctx.deadStrip())
|
||||
for (const UndefinedAtom *undef : undefined())
|
||||
_ctx.addDeadStripRoot(undef->name());
|
||||
}
|
||||
|
||||
/// Iterate over the symbol table to retrieve all symbols.
|
||||
|
|
Loading…
Reference in New Issue