• Creddy's avatar
    Reland "[interpreter] Separate bytecodes for one-shot property loads and stores" · 0714bd9f
    Creddy authored
    This is a reland of eccf1867
    
    Original change's description:
    > [interpreter] Separate bytecodes for one-shot property loads and stores
    > 
    > Create LdaNamedPropertyNoFeedback and StaNamedPropertyNoFeedback
    > for one-shot property loads and stores. This CL replaces the runtime
    > calls with new bytecodes for named property load stores in one-shot code.
    > the runtime calls needed extra set of consecutive registers and
    > additional move instructions. This increased the size of
    > bytecode-array and possibly extended the life time of objects.
    > By replacing them with NoFeedback bytecodes we avoid these issues.
    > 
    > Bug: v8:8072
    > Change-Id: I20a38a5ce9940026171d870d354787fe0b7c5a6f
    > Reviewed-on: https://chromium-review.googlesource.com/1196725
    > Reviewed-by: Ross McIlroy <rmcilroy@chromium.org>
    > Reviewed-by: Jaroslav Sevcik <jarin@chromium.org>
    > Reviewed-by: Yang Guo <yangguo@chromium.org>
    > Reviewed-by: Georg Neis <neis@chromium.org>
    > Commit-Queue: Chandan Reddy <chandanreddy@google.com>
    > Cr-Commit-Position: refs/heads/master@{#56211}
    
    Bug: v8:8072
    Change-Id: Ie8e52b37daf35c7bc08bb910d7b15a9b783354e4
    Reviewed-on: https://chromium-review.googlesource.com/1245742
    Commit-Queue: Chandan Reddy <chandanreddy@google.com>
    Reviewed-by: 's avatarRoss McIlroy <rmcilroy@chromium.org>
    Reviewed-by: 's avatarGeorg Neis <neis@chromium.org>
    Reviewed-by: 's avatarCamillo Bruni <cbruni@chromium.org>
    Reviewed-by: 's avatarYang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#56266}
    0714bd9f
bytecode-array-builder.cc 52.3 KB