[PATCH] D16440: [ThinLTO] Link in only necessary DICompileUnit operands

[Mehdi AMINI via llvm-commits]

joker.eph added a comment.

So I debugged the assertion I see and the issue seems that we don't import anymore a retained type.

`DwarfDebug::beginModule()` calls `TypeIdentifierMap = generateDITypeIdentifierMap(CU_Nodes);` to build (from the retained types list) a map from type name to matching DICompositeType.

When emitting a DISubprogram, `getOrCreateSubprogramDIE` will call `resolve(SP->getScope())` which query the map. 
It seems that any scope for a DISubprogram needs to be in the retained types list, is it correct?


http://reviews.llvm.org/D16440