• bmeurer's avatar
    [test] Increase test coverage for Array constructor inlining. · 23bb8fa9
    bmeurer authored
    This still doesn't cover all the paths yet, since some paths are
    impossible to trigger at this point due to the way the CanInlineCall
    predicate works on the AllocationSite, which says multiple things:
    
     - In case of Array(len), the len was always a Smi so far.
     - In case of Array(...args), storing the args didn't change the
       elements kind.
     - In case of Array(len), the len was always less than the initial
       maximum fast element array size.
    
    These conditions are tailored towards Crankshaft and don't really
    make a lot of sense in the TurboFan world. We'd need more fine
    grained protections, which we will achieve by refactoring the Array
    constructor.
    
    BUG=chromium:715404,v8:6262
    TBR=machenbach@chromium.org
    
    Review-Url: https://codereview.chromium.org/2843033002
    Cr-Commit-Position: refs/heads/master@{#44901}
    23bb8fa9
array-constructor.js 2.3 KB