• Igor Sheludko's avatar
    Reland "[csa] Fix semantics of PopAndReturn" · 3593ee83
    Igor Sheludko authored
    This is a reland of 5e5eaf79
    
    This CL fixes the "function returns address of local variable" issue
    which GCC was complaining about by using inline assembly instead of
    address of a local for getting stack pointer approximation.
    
    Original change's description:
    > [csa] Fix semantics of PopAndReturn
    >
    > This CL prohibits using PopAndReturn from the builtins that
    > have calling convention with arguments on the stack.
    >
    > This CL also updates the PopAndReturn tests so that even off-by-one
    > errors in the number of poped arguments are caught which was not the
    > case before.
    >
    > Motivation:
    >
    > PopAndReturn is supposed to be using ONLY in CSA/Torque builtins for
    > dropping ALL JS arguments that are currently located on the stack.
    > Disallowing PopAndReturn in builtins with stack arguments simplifies
    > semantics of this instruction because in case of presence of declared
    > stack parameters it's impossible to distinguish the following cases:
    > 1) stack parameter is included in JS arguments (and therefore it will
    >    be dropped as a part of 'pop' number of arguments),
    > 2) stack parameter is NOT included in JS arguments (and therefore it
    >    should be dropped in ADDITION to the 'pop' number of arguments).
    >
    > This issue wasn't noticed before because builtins with stack parameters
    > relied on adapter frames machinery to ensure that the expected
    > parameters are present on the stack, but on the same time the adapter
    > frame tearing down code was effectively recovering the stack pointer
    > potentially broken by the CSA builtin.
    >
    > Once we get rid of the arguments adapter frames keeping stack pointer
    > in a valid state becomes crucial.
    >
    > Bug: v8:5269, v8:10201
    > Change-Id: Id3ea9730bb0d41d17999c73136c4dfada374a822
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2460819
    > Commit-Queue: Igor Sheludko <ishell@chromium.org>
    > Reviewed-by: Tobias Tebbi <tebbi@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#70454}
    
    Tbr: tebbi@chromium.org
    Bug: v8:5269
    Bug: v8:10201
    Change-Id: Ic1a05fcc4efd2068538bff28189545cfd2617d9b
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2465839Reviewed-by: 's avatarIgor Sheludko <ishell@chromium.org>
    Reviewed-by: 's avatarVictor Gomes <victorgomes@chromium.org>
    Commit-Queue: Igor Sheludko <ishell@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#70483}
    3593ee83
Name
Last commit
Last update
..
arm Loading commit data...
arm64 Loading commit data...
ia32 Loading commit data...
mips Loading commit data...
mips64 Loading commit data...
ppc Loading commit data...
s390 Loading commit data...
x64 Loading commit data...
DEPS Loading commit data...
OWNERS Loading commit data...
assembler-arch.h Loading commit data...
assembler-inl.h Loading commit data...
assembler.cc Loading commit data...
assembler.h Loading commit data...
bailout-reason.cc Loading commit data...
bailout-reason.h Loading commit data...
callable.h Loading commit data...
code-comments.cc Loading commit data...
code-comments.h Loading commit data...
code-desc.cc Loading commit data...
code-desc.h Loading commit data...
code-factory.cc Loading commit data...
code-factory.h Loading commit data...
code-reference.cc Loading commit data...
code-reference.h Loading commit data...
code-stub-assembler.cc Loading commit data...
code-stub-assembler.h Loading commit data...
compilation-cache.cc Loading commit data...
compilation-cache.h Loading commit data...
compiler.cc Loading commit data...
compiler.h Loading commit data...
constant-pool.cc Loading commit data...
constant-pool.h Loading commit data...
constants-arch.h Loading commit data...
cpu-features.h Loading commit data...
external-reference-encoder.cc Loading commit data...
external-reference-encoder.h Loading commit data...
external-reference-table.cc Loading commit data...
external-reference-table.h Loading commit data...
external-reference.cc Loading commit data...
external-reference.h Loading commit data...
flush-instruction-cache.cc Loading commit data...
flush-instruction-cache.h Loading commit data...
handler-table.cc Loading commit data...
handler-table.h Loading commit data...
interface-descriptors.cc Loading commit data...
interface-descriptors.h Loading commit data...
label.h Loading commit data...
machine-type.cc Loading commit data...
machine-type.h Loading commit data...
macro-assembler-inl.h Loading commit data...
macro-assembler.h Loading commit data...
optimized-compilation-info.cc Loading commit data...
optimized-compilation-info.h Loading commit data...
pending-optimization-table.cc Loading commit data...
pending-optimization-table.h Loading commit data...
register-arch.h Loading commit data...
register-configuration.cc Loading commit data...
register-configuration.h Loading commit data...
register.cc Loading commit data...
register.h Loading commit data...
reglist.h Loading commit data...
reloc-info.cc Loading commit data...
reloc-info.h Loading commit data...
safepoint-table.cc Loading commit data...
safepoint-table.h Loading commit data...
signature.h Loading commit data...
source-position-table.cc Loading commit data...
source-position-table.h Loading commit data...
source-position.cc Loading commit data...
source-position.h Loading commit data...
string-constants.cc Loading commit data...
string-constants.h Loading commit data...
tick-counter.cc Loading commit data...
tick-counter.h Loading commit data...
tnode.cc Loading commit data...
tnode.h Loading commit data...
turbo-assembler.cc Loading commit data...
turbo-assembler.h Loading commit data...
unoptimized-compilation-info.cc Loading commit data...
unoptimized-compilation-info.h Loading commit data...