• Leszek Swirski's avatar
    [destructuring] Get non-coercible message contents in runtime · 5e2c23e2
    Leszek Swirski authored
    For desrtucturing assignments from null/undefined, we throw an error
    that references the destructuring object literal's property name, e.g.
    for
      var { x } = null;
    we report that we cannot destructure 'x' from null.
    
    Rather than calculating this property during bytecode generation (and
    including it in the bytecode as an argument to the type error
    constructor), we can calculate it at exception throwing time, by
    re-parsing the source in a similar way to the existing call site
    rendering.
    
    This slightly decreases bytecode size and slightly decreases the amount
    of work the bytecode compiler needs to do. In the future, it could also
    allow us to give more detailed error messages, as we now have access to
    the entire AST and are on the slow path anyway.
    
    Bug: v8:6499
    Change-Id: Icdbd4667db548b4e5e62ef97797a3771b5c1bf72
    Reviewed-on: https://chromium-review.googlesource.com/c/1396080Reviewed-by: 's avatarToon Verwaest <verwaest@chromium.org>
    Reviewed-by: 's avatarSathya Gunasekaran <gsathya@chromium.org>
    Commit-Queue: Leszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#58706}
    5e2c23e2
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-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...