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

2018-11-29 Thread Teresa Johnson via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC347887: [ThinLTO] Allow importing of multiple symbols with same GUID (authored by tejohnson, committed by ). Changed prior to commit: https://reviews.llvm.org/D55048?vs=175814=175881#toc Repository:

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

2018-11-29 Thread Eugene Leviant via Phabricator via cfe-commits
evgeny777 accepted this revision. evgeny777 added a comment. This revision is now accepted and ready to land. LGTM Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55048/new/ https://reviews.llvm.org/D55048 ___

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

2018-11-28 Thread 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 , to handle the case where two different modules have local