• Santiago Aboy Solanes's avatar
    [compiler] Have one unique PersistentHandles container · 64828a54
    Santiago Aboy Solanes authored
    The (now unique)PersistentHandles container follows this path:
      1) PersistentHandles created via PersistentHandlesScope inside of
         CompilationHandleScope
      2) Owned by OptimizedCompilationInfo
      3) Owned by JSHeapBroker
      4) Owned by the broker's LocalHeap
      5) Back to the broker for a brief moment (after tearing down the
        LocalHeap as part of exiting LocalHeapScope)
      6) Back to OptimizedCompilationInfo when exiting the LocalHeapScope.
    
    There is a special case in GenerateCodeForTesting where the JSHeapBroker
    will not be retired in that same method. In this case, we need to
    re-attach the PersistentHandles container to the JSHeapBroker.
    
    The identity map of the persistent & canonical handles also gets passed
    around like the persistent handles. The only difference is that is
    created in the CanonicalHandleScope (i.e step 1) is different).
    
    Bug: v8:7790
    Change-Id: I2da77a7e08f3fd360a46b606c5fbda08c0af27df
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2332811
    Commit-Queue: Santiago Aboy Solanes <solanes@chromium.org>
    Reviewed-by: 's avatarGeorg Neis <neis@chromium.org>
    Reviewed-by: 's avatarDominik Inführ <dinfuehr@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#69360}
    64828a54
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...