[PATCH] D15137: Remove "ExportingModule" from ThinLTO Index

[Teresa Johnson via llvm-commits]

tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.

Yes your new check should be sufficient. Eventually when we prune functions from the index that are unlikely to be profitable to import (e.g. they are cold), we should ensure the module string isn't added to the combined index. Looks like that will happen naturally if we do the pruning before we add the module string in FunctionInfoIndex::mergeFrom().

LGTM


http://reviews.llvm.org/D15137