-
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: Ross McIlroy <rmcilroy@chromium.org> Cr-Commit-Position: refs/heads/master@{#70836}
f35495ca