• Sathya Gunasekaran's avatar
    Reland "[ic] Add a new MegaDOM IC" · fbd114bd
    Sathya Gunasekaran authored
    This is a reland of c83c9590
    
    Changes since revert: nothing, issue was crbug.com/v8/11666
    
    Original change's description:
    > [ic] Add a new MegaDOM IC
    >
    > This patch implements the MegaDOM IC setup and access. A new MegaDOM
    > IC state indicates that we've seen only DOM accessors at this access
    > site.
    >
    > This CL only adds support for DOM getters in LoadIC, other kinds of
    > access will be added in follow on CLs.
    >
    > Still remaining TODO before shipping:
    > 1. Have a mechanism to invalidate the protector
    > 2. Have a mechanism to find the accessors that aren't overloaded
    > 3. Use a new builtin to miss to runtime on access check failure
    >
    > Change-Id: Ie12efe5e9fa284f023043b996d61e7d74e710ee2
    > Bug: v8:11321
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2618239
    > Reviewed-by: Omer Katz <omerkatz@chromium.org>
    > Reviewed-by: Camillo Bruni <cbruni@chromium.org>
    > Reviewed-by: Dan Elphick <delphick@chromium.org>
    > Reviewed-by: Mythri Alle <mythria@chromium.org>
    > Commit-Queue: Sathya Gunasekaran  <gsathya@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#73733}
    
    Bug: v8:11321
    Change-Id: I2bec54465542b5b40c42adb6eb12b6ce72cce5bd
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794439Reviewed-by: 's avatarDan Elphick <delphick@chromium.org>
    Reviewed-by: 's avatarMythri Alle <mythria@chromium.org>
    Reviewed-by: 's avatarCamillo Bruni <cbruni@chromium.org>
    Reviewed-by: 's avatarOmer Katz <omerkatz@chromium.org>
    Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
    Commit-Queue: Sathya Gunasekaran  <gsathya@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#74056}
    fbd114bd
ic.cc 118 KB