• bmeurer's avatar
    Revert of [runtime] Introduce dedicated JSBoundFunction to represent bound... · 1cf8b105
    bmeurer authored
    Revert of [runtime] Introduce dedicated JSBoundFunction to represent bound functions. (patchset #14 id:260001 of https://codereview.chromium.org/1542963002/ )
    
    Reason for revert:
    Breaks arm64 sim nosnap: https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20arm64%20-%20sim%20-%20nosnap%20-%20debug/builds/805/steps/Check/logs/function-bind
    
    Original issue's description:
    > [runtime] Introduce dedicated JSBoundFunction to represent bound functions.
    >
    > According to the ES2015 specification, bound functions are exotic
    > objects, and thus don't need to be implemented as JSFunctions. So
    > we introduce a new JSBoundFunction type to represent bound functions
    > and make them optimizable. This already improves the performance of
    > calling or constructing bound functions by 10-100x depending on the
    > use case because we avoid the crazy dance between JavaScript and C++
    > that was implemented in v8natives.js previously.
    >
    > There's still room for improvement in the performance of actually
    > creating bound functions, which is also relevant in practice, but
    > we already have a plan how to accomplish that later.
    >
    > The mips/mips64 ports were contributed by akos.palfi@imgtec.com.
    >
    > CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
    > BUG=chromium:535408, chromium:571299, v8:4629
    > LOG=n
    >
    > Committed: https://crrev.com/ca8623eaa468cba65a5adafcdfb4615966f43ce2
    > Cr-Commit-Position: refs/heads/master@{#33042}
    
    TBR=cbruni@chromium.org,hpayer@chromium.org,yangguo@chromium.org,akos.palfi@imgtec.com
    NOPRESUBMIT=true
    NOTREECHECKS=true
    NOTRY=true
    BUG=chromium:535408, chromium:571299, v8:4629
    
    Review URL: https://codereview.chromium.org/1552473002
    
    Cr-Commit-Position: refs/heads/master@{#33043}
    1cf8b105
builtins-arm.cc 75.9 KB