• Michael Achenbach's avatar
    Revert "[ic] Properly integrate the CallIC into Ignition." · 018128a4
    Michael Achenbach authored
    This reverts commit 6c541561.
    
    Reason for revert:
    https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20nosnap/builds/17240
    
    Original change's description:
    > [ic] Properly integrate the CallIC into Ignition.
    > 
    > Drop the deprecated CallConstructStub and remove the use of CallICStub
    > from fullcodegen, since that feedback is unused completely every since
    > Crankshaft got removed, thus we can safely unlink all the CallIC stuff
    > from fullcodegen nowadays, and completely nuke the CallICStub and the
    > CallICTrampolineStub now (we can also transitively nuke the unused
    > CreateAllocationSiteStub and CreateWeakCellStub).
    > 
    > Instead the CallIC logic is integrated into Ignition now, and part of
    > the bytecode handlers for [[Call]] and [[Construct]]. There's still some
    > follow-up cleanup with the way the Array constructor feedback is
    > integrated, but that's way easier now.
    > 
    > Bug: v8:5517, v8:6399, v8:6409, v8:6679
    > Change-Id: Ia0efc6145ee64633757a6c3fd1879d4906ea2835
    > Reviewed-on: https://chromium-review.googlesource.com/602134
    > Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
    > Reviewed-by: Yang Guo <yangguo@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#47192}
    
    TBR=rmcilroy@chromium.org,yangguo@chromium.org,bmeurer@chromium.org
    
    Change-Id: I416ce6646f62ceb4127b3acee43912ee0d701c23
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:5517, v8:6399, v8:6409, v8:6679
    Reviewed-on: https://chromium-review.googlesource.com/603647Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
    Commit-Queue: Michael Achenbach <machenbach@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#47193}
    018128a4
code-factory.cc 15.4 KB