davidxl added a comment. This version looks very good -- the only thing I dislike is that we still need to pass default threshold around via getInlineCost interface -- I think we can deal with that in the future. http://reviews.llvm.org/D15401