[PATCH] D29169: Do not apply redundant LastCallToStaticBonus

[Easwaran Raman via Phabricator via llvm-commits]

eraman added a comment.

Sorry about the delay. The patch itself is straightforward and fixes an obvious error. Before I LGTM, do you see any size/performance improvement as the result of this?



================
Comment at: test/Transforms/Inline/last-call-bonus.ll:1
+; The goal of this test is checking LastCallToStaticBonus is applied correctly.
+; For the test code below, when inliner evaluates the callsite of bar->baz, it
----------------
nit: Make it clear that this checks that LastCallToStaticBonus is applied correctly while deciding inline deferral (since it has nothing to do with application of LastCallBonus in InlineCost.cpp)


https://reviews.llvm.org/D29169