[PATCH] D156525: [ThinLTO] Use module hash instead of module ID for cache key

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

> A caveat here is that this doesn't quite work for linkonce_odr functions, because it seems like the function from the first encountered module gets picked (rather than say, the prevailing function). This doesn't really matter for Rust's purposes (because it does not use linkonce_odr linkage), but may still be worth addressing. For now I'm using a variant of the test using internal instead of linkonce_odr functions.

Can you clarify what doesn't work with linkonce_odr? This patch shouldn't affect any importing decisions, which have already been made, so the import list sorted here should only include a single copy of a linkonce_odr function.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156525/new/

https://reviews.llvm.org/D156525