[PATCH] D35081: [ThinLTO] Allow multiple summary entries.

[Teresa Johnson via Phabricator via cfe-commits]

tejohnson added a comment.

In https://reviews.llvm.org/D35081#805506, @yunlian wrote:

> This error happens when I try to triage a thinLTO failure on ARM.
>
> Initially I got some error like 
>  Instruction does not dominate all uses!
>
>   %205 = bitcast i1 (%"class.blink::LayoutObject"*)** %194 to i8*, !dbg !51180
>   %200 = getelementptr i8, i8* %205, i32 ptrtoint (i8* @__typeid__ZTSN5blink12LayoutObjectE_100_byte to i32), !dbg !51170
>
> LLVM ERROR: Broken function found, compilation aborted!
>  clang-5.0: error: linker command failed with exit code 1 (use -v to see invocation)
>
> I then used distributed thinLTO to try to find a reduced test case.
>  I use -Wl,-plugin-opt,thinlto-index-only=file to get the index file for each bite code and I
> run
>  $cmd -fthinlto-index=${obj}.thinlto.bc -x ir ${obj} -o a.o -c
>
> to get the error shown in this change.


That's really odd - I'm not sure why you would get more than one summary for a GUID in the individual index files. Can you package up your thin link inputs and command for me to take a look?


https://reviews.llvm.org/D35081