[PATCH] D29169: Do not apply redundant LastCallToStaticBonus
Easwaran Raman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 3 12:00:30 PST 2017
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)
More information about the llvm-commits