• Dan Elphick's avatar
    [builtins] Reduce table size for bytecode mappings · f35495ca
    Dan Elphick authored
    This replaces kBytecodeToBuiltinsMapping (an array with currently 549
    32-bit integers = 2196 bytes) with kWideBytecodeToBuiltinsMapping which
    is an array of uint8_t with only 183 values. The new array contains just
    the mappings from wide handlers to builtins but only once since the
    mapping is the same for extra wide handlers. (No mapping array is
    required for normal handlers since they map 1:1).
    
    This reduces d8's binary size by 2008 bytes on x64.
    
    As a result Interpreter::GetBytecodeHandler will be slightly slower than
    before, but its only use in non-test code is in
    Runtime_DebugBreakOnBytecode which does not need to be fast.
    
    Bug: v8:11066
    Change-Id: Iafc28fba2d1b62c1d49ceabe731d8b52a82dd2fd
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2502291
    Commit-Queue: Ross McIlroy <rmcilroy@chromium.org>
    Reviewed-by: 's avatarRoss McIlroy <rmcilroy@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#70836}
    f35495ca
interpreter.cc 15 KB