]> granicus.if.org Git - llvm/commit
[ThinLTO] Fix crash when importing an opaque type
authorMehdi Amini <mehdi.amini@apple.com>
Sat, 19 Nov 2016 18:44:16 +0000 (18:44 +0000)
committerMehdi Amini <mehdi.amini@apple.com>
Sat, 19 Nov 2016 18:44:16 +0000 (18:44 +0000)
commitc695a0be78c8ad700d8f62909369fdfe555452e7
treeb9f3a0c6aabbd5c50bb8e9bf77d8c8d94027db06
parent4bd62ab48924f095dc74b710158a232539060f79
[ThinLTO] Fix crash when importing an opaque type

It seems that because ThinLTO does not import the full module,
some invariant of the type mapper are broken.

In Monolithic LTO, we import every globals: when calling
IRLinker::copyFunctionProto() on @foo(), we end-up calling
TypeMapTy::get(FTy) on the type of @foo(), which will map
%0 and record the destination as opaque.

ThinLTO skips this because @foo is not imported and goes directly
to the next stage.

Next we call computeTypeMapping() that map the types for each
globals, and ends up checking for type isomorphism, and may add
type mapping. However it doesn't record if there was an opaque
destination type that was resolved.

Instead of lazily "discovering" opaque type in the destination
module on the go, we change the TypeFinder to eagerly record all
types and not only the named ones.

Differential Revision: https://reviews.llvm.org/D26840

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@287453 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Linker/IRMover.cpp
test/ThinLTO/X86/Inputs/import_opaque_type.ll [new file with mode: 0644]
test/ThinLTO/X86/import_opaque_type.ll [new file with mode: 0644]