• Dan Elphick's avatar
    [parser] Fix bytecode mismatch for this · dd547367
    Dan Elphick authored
    Fixes bytecode mismatch between lazy and non-lazy where "this" was
    marked as maybe assigned in constructors that called the super
    constructor. Since this will return the hole in cases where it was not
    yet initialized by super (and the hole is explicitly handled by
    JSContextSpecialization::ReduceJSLoadContext), it's safe to treat it as
    a constant in all cases. In the case of lazy compilation case, "this"
    is never added to the ScopeInfo so is never seen as mutable.
    
    Bug: chromium:994719
    Change-Id: I43478fbc626b19eb1533aa9dec61b7f276ae140b
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1762025
    Commit-Queue: Dan Elphick <delphick@chromium.org>
    Reviewed-by: 's avatarRoss McIlroy <rmcilroy@chromium.org>
    Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#63283}
    dd547367
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector 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...
mkgrokdump Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-api-tests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...