• Nico Hartmann's avatar
    Revert "Reland "[torque] Don't generate k(?:Start|End)Of\w+FieldsOffset constants"" · 6a3dc05f
    Nico Hartmann authored
    This reverts commit a3480b55.
    
    Reason for revert: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20-%20debug%20-%20header%20includes/22234/overview
    
    Original change's description:
    > Reland "[torque] Don't generate k(?:Start|End)Of\w+FieldsOffset constants"
    >
    > This is a reland of 7366f6e2
    >
    > The test that failed after the initial commit was just flaky and has
    > been fixed; see https://bugs.chromium.org/p/v8/issues/detail?id=12341
    >
    > Original change's description:
    > > [torque] Don't generate k(?:Start|End)Of\w+FieldsOffset constants
    > >
    > > Torque currently generates constants like kStartOfWeakFieldsOffset and
    > > kEndOfStrongFieldsOffset, which can be used when writing custom
    > > BodyDescriptors. However, these offsets have some potentially confusing
    > > behaviors:
    > >
    > > * They don't take inheritance into account and describe only the fields
    > >   defined by the current class itself, so there might be (for example)
    > >   strong fields before kStartOfStrongFieldsOffset if they were defined
    > >   by a superclass.
    > > * kStartOfWeakFieldsOffset points to the first field defined in Torque
    > >   using the keyword `weak`, which indicates fields with *custom*
    > >   weakness semantics (those that should be visited with
    > >   IterateCustomWeakPointers), not those that may contain standard weak
    > >   pointers (visited with IterateMaybeWeakPointers). (As a follow-up, I'd
    > >   like to also rename `weak` to `@customWeak`.)
    > >
    > > Given that these constants have very low usage and somewhat bizarre
    > > semantics, I propose that we remove them. This change does so, and
    > > updates the existing usages to either define the required constants
    > > directly in C++ or not use them. I know that defining these constants in
    > > C++ is more brittle, but I think that brittle and clear is better than
    > > automatic and incomprehensible.
    > >
    > > Bug: v8:7793
    > > Change-Id: I87f8c85ccae4027f61ac73d4e7e4e2820e92003b
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3199731
    > > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org>
    > > Reviewed-by: Toon Verwaest <verwaest@chromium.org>
    > > Commit-Queue: Seth Brenith <seth.brenith@microsoft.com>
    > > Cr-Commit-Position: refs/heads/main@{#77411}
    >
    > Bug: v8:7793
    > Change-Id: Iefdd4014ce4b85b48c19ead79a0316774a5ecd45
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3258082
    > Reviewed-by: Toon Verwaest <verwaest@chromium.org>
    > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org>
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Commit-Queue: Seth Brenith <seth.brenith@microsoft.com>
    > Cr-Commit-Position: refs/heads/main@{#77688}
    
    Bug: v8:7793
    Change-Id: I7b9667268901b7aef85a95832d40860056e61050
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3259656Reviewed-by: 's avatarNico Hartmann <nicohartmann@chromium.org>
    Owners-Override: Nico Hartmann <nicohartmann@chromium.org>
    Commit-Queue: Nico Hartmann <nicohartmann@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#77689}
    6a3dc05f
Name
Last commit
Last update
..
baseline Loading commit data...
DEPS Loading commit data...
DIR_METADATA Loading commit data...
OWNERS Loading commit data...
branch-hint-map.h Loading commit data...
c-api.cc Loading commit data...
c-api.h Loading commit data...
code-space-access.cc Loading commit data...
code-space-access.h Loading commit data...
compilation-environment.h Loading commit data...
decoder.h Loading commit data...
function-body-decoder-impl.h Loading commit data...
function-body-decoder.cc Loading commit data...
function-body-decoder.h Loading commit data...
function-compiler.cc Loading commit data...
function-compiler.h Loading commit data...
graph-builder-interface.cc Loading commit data...
graph-builder-interface.h Loading commit data...
init-expr-interface.cc Loading commit data...
init-expr-interface.h Loading commit data...
jump-table-assembler.cc Loading commit data...
jump-table-assembler.h Loading commit data...
leb-helper.h Loading commit data...
local-decl-encoder.cc Loading commit data...
local-decl-encoder.h Loading commit data...
memory-protection-key.cc Loading commit data...
memory-protection-key.h Loading commit data...
memory-tracing.cc Loading commit data...
memory-tracing.h Loading commit data...
module-compiler.cc Loading commit data...
module-compiler.h Loading commit data...
module-decoder.cc Loading commit data...
module-decoder.h Loading commit data...
module-instantiate.cc Loading commit data...
module-instantiate.h Loading commit data...
object-access.h Loading commit data...
signature-map.cc Loading commit data...
signature-map.h Loading commit data...
simd-shuffle.cc Loading commit data...
simd-shuffle.h Loading commit data...
stacks.h Loading commit data...
streaming-decoder.cc Loading commit data...
streaming-decoder.h Loading commit data...
struct-types.h Loading commit data...
sync-streaming-decoder.cc Loading commit data...
value-type.cc Loading commit data...
value-type.h Loading commit data...
wasm-arguments.h Loading commit data...
wasm-code-manager.cc Loading commit data...
wasm-code-manager.h Loading commit data...
wasm-constants.h Loading commit data...
wasm-debug.cc Loading commit data...
wasm-debug.h Loading commit data...
wasm-engine.cc Loading commit data...
wasm-engine.h Loading commit data...
wasm-external-refs.cc Loading commit data...
wasm-external-refs.h Loading commit data...
wasm-feature-flags.h Loading commit data...
wasm-features.cc Loading commit data...
wasm-features.h Loading commit data...
wasm-import-wrapper-cache.cc Loading commit data...
wasm-import-wrapper-cache.h Loading commit data...
wasm-init-expr.cc Loading commit data...
wasm-init-expr.h Loading commit data...
wasm-js.cc Loading commit data...
wasm-js.h Loading commit data...
wasm-limits.h Loading commit data...
wasm-linkage.h Loading commit data...
wasm-module-builder.cc Loading commit data...
wasm-module-builder.h Loading commit data...
wasm-module-sourcemap.cc Loading commit data...
wasm-module-sourcemap.h Loading commit data...
wasm-module.cc Loading commit data...
wasm-module.h Loading commit data...
wasm-objects-inl.h Loading commit data...
wasm-objects.cc Loading commit data...
wasm-objects.h Loading commit data...
wasm-objects.tq Loading commit data...
wasm-opcodes-inl.h Loading commit data...
wasm-opcodes.cc Loading commit data...
wasm-opcodes.h Loading commit data...
wasm-result.cc Loading commit data...
wasm-result.h Loading commit data...
wasm-serialization.cc Loading commit data...
wasm-serialization.h Loading commit data...
wasm-subtyping.cc Loading commit data...
wasm-subtyping.h Loading commit data...
wasm-tier.h Loading commit data...
wasm-value.h Loading commit data...