• gsathya's avatar
    Promises: Lazily create arrays to store resolve, reject callbacks · 1d4fe002
    gsathya authored
    For the common use case of having a single resolve or reject callback,
    the callbacks are stored directly. Only when an additional callback is
    registered, we create an array to store these callbacks.
    
    There are 3 possible states for the resolve, reject symbols when we add
    a new callback --
    1) UNDEFINED -- This is the zero state where there is no callback
    registered. When we see this state, we directly attach the callbacks to
    the symbol.
    2) !IS_ARRAY -- There is a single callback directly attached to the
    symbols. We need to create a new array to store additional callbacks.
    3) IS_ARRAY -- There are multiple callbacks already registered,
    therefore we can just push the new callback to the existing array.
    
    Also, this change creates a new symbol for storing the deferred objects.
    Previously the deferred objects were stored in the callback arrays, but
    since we no longer create arrays for the initial case, we need this new
    symbol. The cctest has been updated to account for this new symbol.
    
    This patch results in a 19% improvement(over 5 runs) in the bluebird benchmark.
    
    BUG=v8:5046
    
    Review-Url: https://codereview.chromium.org/2007803002
    Cr-Commit-Position: refs/heads/master@{#36536}
    1d4fe002
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
fuzzer Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
simdjs Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
webkit Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
ignition.gyp Loading commit data...
ignition.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.gyp Loading commit data...
perf.isolate Loading commit data...