[PATCH] D18298: ThinLTO: do not promote GlobalVariable that have a specific section.

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 25 16:19:53 PDT 2016


joker.eph added a comment.

Hitting a wall because I was trying to plumb `const StringMap<std::map<GlobalValue::GUID, GlobalValueSummary *>> &

  ModuleToDefinedGVSummaries()` to get the definedGV for the module we import from, but could not because of `void llvm::ComputeCrossModuleImportForModule`.

The latter was introduced in http://reviews.llvm.org/D18945, for which we had some discussions. 
I think we're moving in a direction where we can remove this anyway right?


http://reviews.llvm.org/D18298





More information about the llvm-commits mailing list