• Michael Lippautz's avatar
    cppgc: Optimize GCInfo setup · 479bfdb1
    Michael Lippautz authored
    In Blink's version of Oilpan, GCInfo objects would reside in .bss and
    a table would translate between an index and the .bss address. Upon
    retrieving a GCInfoIndex, the slow path merely passes a .bss pointer
    to a slow path setup method to create the table mapping.
    
    In cppgc, we set up GCInfo entries directly in the table. This is
    slightly faster for actually using GCInfo objects as there's no
    indirection between table and .bss, and it also saves one pointer (the
    indirection) per type that is set up. The downside of this approach is
    that individual components of a GCInfo objects, that are all
    type-dependent, need to be passed to the conditional setup method.
    Since GCInfo indices must be retrieved on each allocation, this
    pollutes the fast path with additional instructions.
    
    However, GCInfo components are actually known at compile-time for many
    objects. In such cases, we can use a compile-time static dispatch to
    encode the known parameters in different functions. This saves around
    40KiB of memory on ChromePublic.apk and also creates a more compact
    fast path for allocation.
    
    Bug: chromium:1238884, chromium:1056170
    Change-Id: Iedd809a8baefcc02f131d2b2c77d341b0abe43bb
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3094007Reviewed-by: 's avatarAnton Bikineev <bikineev@chromium.org>
    Reviewed-by: 's avatarOmer Katz <omerkatz@chromium.org>
    Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#76291}
    479bfdb1
Name
Last commit
Last update
.github Loading commit data...
bazel Loading commit data...
build_overrides Loading commit data...
custom_deps Loading commit data...
docs Loading commit data...
gni Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.bazelrc Loading commit data...
.clang-format Loading commit data...
.clang-tidy Loading commit data...
.editorconfig Loading commit data...
.flake8 Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.mailmap Loading commit data...
.vpython Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.bazel Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
COMMON_OWNERS Loading commit data...
DEPS Loading commit data...
DIR_METADATA Loading commit data...
ENG_REVIEW_OWNERS Loading commit data...
INFRA_OWNERS Loading commit data...
INTL_OWNERS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
MIPS_OWNERS Loading commit data...
OWNERS Loading commit data...
PPC_OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
RISCV_OWNERS Loading commit data...
S390_OWNERS Loading commit data...
WATCHLISTS Loading commit data...
WORKSPACE Loading commit data...
codereview.settings Loading commit data...