• cbruni's avatar
    Revert of [runtime] Do not use the enum-cache for non-prototype objects.... · 1c523a44
    cbruni authored
    Revert of [runtime] Do not use the enum-cache for non-prototype objects. (patchset #1 id:1 of https://codereview.chromium.org/1612413003/ )
    
    Reason for revert:
    let me quickly revert the revert, wut?
    Goal: my CL should not be in the tree!
    
    Original issue's description:
    > Reland of [runtime] Do not use the enum-cache for non-prototype objects. (patchset #1 id:1 of https://codereview.chromium.org/1619803003/ )
    >
    > Reason for revert:
    > the deopt issues have been taken care of by benedikt
    >
    > Original issue's description:
    > > Revert of [runtime] Do not use the enum-cache for non-prototype objects. (patchset #10 id:180001 of https://codereview.chromium.org/1608523002/ )
    > >
    > > Reason for revert:
    > > tanks for-in significantly
    > >
    > > Original issue's description:
    > > > [runtime] Do not use the enum-cache for keys retrieval.
    > > >
    > > > Currently we fail to properly handle shadowed properties. If the
    > > > receiver defines a non-enumerable property that reappears on the
    > > > prototype as enumerable it incorrectly shows up in [[Enumerate]].
    > > > By extending the KeyAccumulator to track non-enumerable properties
    > > > we can now properly filter them out when seeing them further up in
    > > > the prototype-chain.
    > > >
    > > > BUG=v8:705
    > > > LOG=y
    > > >
    > > > Committed: https://crrev.com/ed24dfe80d1da0827b8571839ee52c03ad09c9c7
    > > > Cr-Commit-Position: refs/heads/master@{#33405}
    > >
    > > TBR=jkummerow@chromium.org,bmeurer@chromium.org
    > > # Not skipping CQ checks because original CL landed more than 1 days ago.
    > > BUG=v8:705
    > > LOG=n
    > >
    > > Committed: https://crrev.com/6e0573c6fff1c3041bab106d1197ab1b64aa9a6a
    > > Cr-Commit-Position: refs/heads/master@{#33443}
    >
    > TBR=jkummerow@chromium.org,bmeurer@chromium.org
    > # Skipping CQ checks because original CL landed less than 1 days ago.
    > NOPRESUBMIT=true
    > NOTREECHECKS=true
    > NOTRY=true
    > BUG=v8:705
    >
    > Committed: https://crrev.com/5569e270eda517b5ea74e3a7676b3230cbe2f7a9
    > Cr-Commit-Position: refs/heads/master@{#33458}
    
    TBR=jkummerow@chromium.org,bmeurer@chromium.org
    # Skipping CQ checks because original CL landed less than 1 days ago.
    NOPRESUBMIT=true
    NOTREECHECKS=true
    NOTRY=true
    BUG=v8:705
    
    Review URL: https://codereview.chromium.org/1614313003
    
    Cr-Commit-Position: refs/heads/master@{#33459}
    1c523a44
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest 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...