• Sathya Gunasekaran's avatar
    Revert "Reland "[runtime] Move Context::native_context to the map"" · 38301e7b
    Sathya Gunasekaran authored
    This reverts commit c7c47c68.
    
    Reason for revert: breaks TSAN
    https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20TSAN/28738
    
    Original change's description:
    > Reland "[runtime] Move Context::native_context to the map"
    > 
    > This is a reland of f05bae1e
    > 
    > Previously I presumed that the context read from a frame in the profiler was
    > a valid context. Turns out that on non-intel we're not guaranteed that the
    > frame is properly set up. In the case we looked at, the profiler took a
    > sample right before writing the frame marker indicating a builtin frame,
    > causing the "context" pointer from that frame to be a bytecode array. Since
    > we'll read random garbage on the stack as a possible context pointer, I made
    > the code reading the native context from it a little more defensive.
    > 
    > Bug: v8:9860
    > 
    > Original change's description:
    > > [runtime] Move Context::native_context to the map
    > >
    > > Remove the native context slot from contexts by making context maps
    > > native-context-specific. Now we require 2 loads to go from a context to the
    > > native context, but we have 1 field fewer to store when creating contexts.
    > >
    > > Change-Id: I3c0d7c50c94060c4129db684f46a567de6f30e8d
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1859629
    > > Commit-Queue: Toon Verwaest <verwaest@chromium.org>
    > > Reviewed-by: Igor Sheludko <ishell@chromium.org>
    > > Reviewed-by: Peter Marshall <petermarshall@chromium.org>
    > > Reviewed-by: Maya Lekova <mslekova@chromium.org>
    > > Reviewed-by: Georg Neis <neis@chromium.org>
    > > Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
    > > Reviewed-by: Toon Verwaest <verwaest@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#64296}
    > 
    > Change-Id: If9461e9b21d35a260d71c79d7f95e518cc429e09
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1864930
    > Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
    > Reviewed-by: Peter Marshall <petermarshall@chromium.org>
    > Reviewed-by: Igor Sheludko <ishell@chromium.org>
    > Reviewed-by: Georg Neis <neis@chromium.org>
    > Commit-Queue: Toon Verwaest <verwaest@chromium.org>
    > Auto-Submit: Toon Verwaest <verwaest@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#64314}
    
    TBR=ulan@chromium.org,neis@chromium.org,petermarshall@chromium.org,ishell@chromium.org,verwaest@chromium.org,mslekova@chromium.org,victorgomes@google.com
    
    Change-Id: I4f9edc62ea6f9f5857619ff0ad1a63cab4b33cc3
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:9860
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1864937Reviewed-by: 's avatarSathya Gunasekaran  <gsathya@chromium.org>
    Commit-Queue: Sathya Gunasekaran  <gsathya@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#64316}
    38301e7b
Name
Last commit
Last update
..
c-signature.h Loading commit data...
call-tester.h Loading commit data...
code-assembler-tester.h Loading commit data...
codegen-tester.cc Loading commit data...
codegen-tester.h Loading commit data...
function-tester.cc Loading commit data...
function-tester.h Loading commit data...
graph-and-builders.h Loading commit data...
serializer-tester.cc Loading commit data...
serializer-tester.h Loading commit data...
test-basic-block-profiler.cc Loading commit data...
test-branch-combine.cc Loading commit data...
test-code-assembler.cc Loading commit data...
test-code-generator.cc Loading commit data...
test-gap-resolver.cc Loading commit data...
test-graph-visualizer.cc Loading commit data...
test-instruction-scheduler.cc Loading commit data...
test-instruction.cc Loading commit data...
test-js-constant-cache.cc Loading commit data...
test-js-context-specialization.cc Loading commit data...
test-js-typed-lowering.cc Loading commit data...
test-jump-threading.cc Loading commit data...
test-linkage.cc Loading commit data...
test-loop-analysis.cc Loading commit data...
test-machine-operator-reducer.cc Loading commit data...
test-multiple-return.cc Loading commit data...
test-node.cc Loading commit data...
test-operator.cc Loading commit data...
test-representation-change.cc Loading commit data...
test-run-bytecode-graph-builder.cc Loading commit data...
test-run-calls-to-external-references.cc Loading commit data...
test-run-deopt.cc Loading commit data...
test-run-intrinsics.cc Loading commit data...
test-run-jsbranches.cc Loading commit data...
test-run-jscalls.cc Loading commit data...
test-run-jsexceptions.cc Loading commit data...
test-run-jsobjects.cc Loading commit data...
test-run-jsops.cc Loading commit data...
test-run-load-store.cc Loading commit data...
test-run-machops.cc Loading commit data...
test-run-native-calls.cc Loading commit data...
test-run-retpoline.cc Loading commit data...
test-run-stackcheck.cc Loading commit data...
test-run-tail-calls.cc Loading commit data...
test-run-unwinding-info.cc Loading commit data...
test-run-variables.cc Loading commit data...
value-helper.cc Loading commit data...
value-helper.h Loading commit data...