• Jaideep Bajwa's avatar
    PPC/s390: Reland "[arm] Restrict grouping pushes before a TailCall to registers only" · a2388599
    Jaideep Bajwa authored
    Port 79bcb454
    
    Original Commit Message:
    
        This is a reland of a72b2f88
        Original change's description:
        > [arm] Restrict grouping pushes before a TailCall to registers only
        >
        > We optimize parallel moves performed before a TailCall by grouping adjacent
        > pushes. This way, we may use a single instruction to push multiple registers at
        > once. However, we also have support for pushing immediates and stack slots for
        > which the benefit is questionnable therefore this patch removes support for
        > them.
        >
        > Concerning immediate pushes, it looks like a mistake since we do not have
        > support for this case in `AssembleMove` so this patch removes it. Furthermore,
        > if we add a test for this case, we see that a `push ip` instruction is
        > generated, effectively pushing whatever was in `ip` at the time instead of
        > pushing a constant.
        >
        > Concerning stack slot pushes, we generate a more or less equivalent sequence of
        > instructions.
        >
        > Finally, grouping floating point pushes is not used anywhere so this patch
        > removes support for this also.
        >
        > Bug: v8:6553
        > Change-Id: I9b820d33361fc442dd813f66e1f96cda41009110
        > Reviewed-on: https://chromium-review.googlesource.com/567191
        > Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
        > Commit-Queue: Pierre Langlois <pierre.langlois@arm.com>
        > Cr-Commit-Position: refs/heads/master@{#46718}
    
    R=pierre.langlois@arm.com, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com
    BUG=
    LOG=N
    
    Change-Id: I8790c7a72f92803ea8fda3c6dc7e6b013e2e09e9
    Reviewed-on: https://chromium-review.googlesource.com/588471Reviewed-by: 's avatarJunliang Yan <jyan@ca.ibm.com>
    Commit-Queue: Jaideep Bajwa <bjaideep@ca.ibm.com>
    Cr-Commit-Position: refs/heads/master@{#46949}
    a2388599
Name
Last commit
Last update
benchmarks Loading commit data...
build_overrides Loading commit data...
docs Loading commit data...
gni Loading commit data...
gypfiles 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...
.clang-format Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
Makefile.android Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...
snapshot_toolchain.gni Loading commit data...