[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