[PATCH] D15696: [ThinLTO] Enable in-place symbol changes for exporting module

[Teresa Johnson via llvm-commits]

tejohnson added a comment.

Once this goes in, I can change the signature of renameModuleForThinLTO so that it takes a Module reference and returns a bool instead of a unique_ptr, and then move the call out of clang (and gold-plugin in http://reviews.llvm.org/D15390) and into FunctionImportPass.

Note doing the ThinLTO promotion/renaming in a single pass over the module after function importing is done seems very difficult, for a couple reasons. One is that we need the source module information to determine how to rename during promotion. It is much easier to invoke on each source module being imported from. Also, to support ThinLTO testing via llvm-link, which does not run the pass manager.


http://reviews.llvm.org/D15696