• Maya Lekova's avatar
    Revert "[ic] Add a new MegaDOM IC" · 3f8aa89e
    Maya Lekova authored
    This reverts commit c83c9590.
    
    Reason for revert: Speculatively reverting for a failure on Arm GC stress bot - https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Arm%20GC%20Stress/b8851256837192083520/overview
    
    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: Ib6a55796f2a3c345d4923f9eaa215a6ff55ed15b
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2794437
    Auto-Submit: Maya Lekova <mslekova@chromium.org>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/master@{#73734}
    3f8aa89e
ic.cc 116 KB