[PATCH] D23949: NFC: add early exit in ModuleSummaryAnalysis

[Teresa Johnson via llvm-commits]

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

LGTM thanks for the cleanup.

We should count intrinsic calls towards the instruction count of the function, because they often do translate to real instructions. Could in fact be more than 1. CallAnalyzer::visitCallSite does some analysis based on the type of intrinsic for the inliner, but I hadn't looked into cloning that here - if we unify the inliner and importer cost analysis we would presumably pick that up.


https://reviews.llvm.org/D23949