[PATCH] Cleanup early-exit from analyzeCall
Easwaran Raman
eraman at google.com
Wed Mar 11 14:19:29 PDT 2015
This makes the following changes to the early bailout code:
1. Do not exit if the cost has exceeded the threshold taking into
account only the current vector bonus. It is possible that we might end up
with a larger vector bonus after we see the entire function. Added a test
case for this.
2. It currently calculates the 10% and 50% vector bonuses after applying
a single basic-block bonus on the original threshold. I think expressing
the bonuses in terms of the original threshold makes it less confusing.
3. Set the threshold to 0 instead of 1 when an unreachable instruction
is seen after the call. Has no practical effect, but keeps it in line
with the comment before that code.
4. Add a MaxThreshold field to track the maximum possible value
Threshold can reach. This is useful if more adjustments are made to the
threshold.
REPOSITORY
rL LLVM
http://reviews.llvm.org/D8267
Files:
lib/Analysis/IPA/InlineCost.cpp
test/Transforms/Inline/vector-bonus.ll
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8267.21758.patch
Type: text/x-patch
Size: 6204 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150311/c5c26e02/attachment.bin>
More information about the llvm-commits
mailing list