• 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
StandardForLoop.golden 15 KB