Restore "[ThinLTO] Prevent exporting of locals used/defined in module level asm"
authorTeresa Johnson <tejohnson@google.com>
Mon, 14 Nov 2016 17:12:32 +0000 (17:12 +0000)
committerTeresa Johnson <tejohnson@google.com>
Mon, 14 Nov 2016 17:12:32 +0000 (17:12 +0000)
commit3624bdf60aa49dd3943ac4a2cdb552143df3964a
treef9fa0d34048fa99a546a335d4c97d15971f1d04d
parentd428cf8b5f5c1fce2034320e059894f136ea2c43
Restore "[ThinLTO] Prevent exporting of locals used/defined in module level asm"

This restores the rest of r286297 (part was restored in r286475).
Specifically, it restores the part requiring adding a dependency from
the Analysis to Object library (downstream use changed to correctly
model split BitReader vs BitWriter libraries).

Original description of this part of patch follows:

Module level asm may also contain defs of values. We need to prevent
export of any refs to local values defined in module level asm (e.g. a
ref in normal IR), since that also requires renaming/promotion of the
local. To do that, the summary index builder looks at all values in the
module level asm string that are not marked Weak or Global, which is
exactly the set of locals that are defined. A summary is created for
each of these local defs and flagged as NoRename.

This required adding handling to the BitcodeWriter to look at GV
declarations to see if they have a summary (rather than skipping them
all).

Finally, added an assert to IRObjectFile::CollectAsmUndefinedRefs to
ensure that an MCAsmParser is available, otherwise the module asm parse
would silently fail. Initialized the asm parser in the opt tool for use
in testing this fix.

Fixes PR30610.

llvm-svn: 286844
llvm/include/llvm/Support/TargetRegistry.h
llvm/lib/Analysis/LLVMBuild.txt
llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
llvm/lib/Object/IRObjectFile.cpp
llvm/test/LTO/X86/current-section.ll
llvm/test/ThinLTO/X86/module_asm2.ll
llvm/tools/opt/opt.cpp