[PATCH] D15024: [ThinLTO] Support for specifying function index from pass manager

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 4 12:35:38 PST 2015


joker.eph requested changes to this revision.
joker.eph added a comment.
This revision now requires changes to proceed.

The function index is owned by the PassManagerBuilder in this patch, which is not good: there is not guarantee it will outlive the PassManager. The PassManagerBuild should take a pointer and not own the FunctionIndex I think.


http://reviews.llvm.org/D15024





More information about the llvm-commits mailing list