[PATCH] D55048: [ThinLTO] Allow importing of multiple symbols with same GUID

[Teresa Johnson via Phabricator via cfe-commits]

tejohnson created this revision.
tejohnson added a reviewer: evgeny777.
Herald added subscribers: dexonsmith, steven_wu, eraman, inglorion, mehdi_amini.

The is the clang side of the fix in D55047 <https://reviews.llvm.org/D55047>, to handle the case where
two different modules have local variables with the same GUID because
they had the same source file name at compilation time. Allow multiple
symbols with the same GUID to be imported, and test that this case works
with the distributed backend path.

Depends on D55047 <https://reviews.llvm.org/D55047>.


Repository:
  rC Clang

https://reviews.llvm.org/D55048

Files:
  lib/CodeGen/BackendUtil.cpp
  test/CodeGen/Inputs/thinlto_backend_local_name_conflict1.ll
  test/CodeGen/Inputs/thinlto_backend_local_name_conflict2.ll
  test/CodeGen/thinlto_backend_local_name_conflict.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55048.175814.patch
Type: text/x-patch
Size: 4190 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181129/51ed9a95/attachment.bin>