• Yang Guo's avatar
    Revert "[turbofan] Rewrite CompilationDependencies" · b1cf1e1e
    Yang Guo authored
    This reverts commit 52a10e50.
    
    Reason for revert: https://test-results.appspot.com/data/layout_results/V8-Blink_Linux_64__dbg_/12434/layout-test-results/results.html
    
    Crash e.g. in http/tests/devtools/oopif/oopif-performance-cpu-profiles.js
    
    crash log for devtools (pid <unknown>):
    STDOUT: <empty>
    STDERR: 
    STDERR: 
    STDERR: #
    STDERR: # Fatal error in ../../v8/src/compilation-dependencies.cc, line 281
    STDERR: # Debug check failed: descriptor == owner->LastAdded() (10 vs. 22).
    STDERR: #
    STDERR: #
    STDERR: #
    STDERR: #FailureMessage Object: 0x7fff86878630#0 0x0000031c642c base::debug::StackTrace::StackTrace()
    STDERR: #1 0x0000046a56bb gin::(anonymous namespace)::PrintStackTrace()
    STDERR: #2 0x00000469c528 V8_Fatal()
    STDERR: #3 0x00000469c285 v8::base::(anonymous namespace)::DefaultDcheckHandler()
    STDERR: #4 0x000001cc5253 v8::internal::CompilationDependencies::DependOnFieldType()
    STDERR: #5 0x000001cdcc46 v8::internal::compiler::AccessInfoFactory::ComputePropertyAccessInfo()
    STDERR: #6 0x000001cde661 v8::internal::compiler::AccessInfoFactory::ComputePropertyAccessInfos()
    STDERR: #7 0x000001dd982b v8::internal::compiler::JSNativeContextSpecialization::ReduceNamedAccess()
    STDERR: #8 0x000001ddb715 v8::internal::compiler::JSNativeContextSpecialization::ReduceNamedAccessFromNexus()
    STDERR: #9 0x000001dd656d v8::internal::compiler::JSNativeContextSpecialization::ReduceJSLoadNamed()
    STDERR: #10 0x000001d53872 v8::internal::compiler::GraphReducer::Reduce()
    STDERR: #11 0x000001d534a5 v8::internal::compiler::GraphReducer::ReduceTop()
    STDERR: #12 0x000001d52e58 v8::internal::compiler::GraphReducer::ReduceNode()
    STDERR: #13 0x000001e4c201 v8::internal::compiler::InliningPhase::Run()
    STDERR: #14 0x000001e44f79 v8::internal::compiler::PipelineImpl::Run<>()
    STDERR: #15 0x000001e41058 v8::internal::compiler::PipelineImpl::CreateGraph()
    STDERR: #16 0x000001e40c75 v8::internal::compiler::PipelineCompilationJob::PrepareJobImpl()
    STDERR: #17 0x000001ccd437 v8::internal::OptimizedCompilationJob::PrepareJob()
    STDERR: #18 0x000001cd071e v8::internal::(anonymous namespace)::GetOptimizedCode()
    STDERR: #19 0x000001cd0c6f v8::internal::Compiler::CompileOptimized()
    STDERR: #20 0x00000231fb62 v8::internal::__RT_impl_Runtime_CompileOptimized_Concurrent()
    STDERR: #21 0x00000288e535 <unknown>
    
    Original change's description:
    > [turbofan] Rewrite CompilationDependencies
    > 
    > Instead of installing code dependencies during graph reduction,
    > install them after code generation.
    > 
    > Bug: v8:7902, v8:7790
    > Change-Id: I8a3798254abb5b9ec7c295a1592aeb6b51f24c7a
    > Reviewed-on: https://chromium-review.googlesource.com/1119913
    > Commit-Queue: Georg Neis <neis@chromium.org>
    > Reviewed-by: Jaroslav Sevcik <jarin@chromium.org>
    > Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#54170}
    
    TBR=mstarzinger@chromium.org,jarin@chromium.org,neis@chromium.org
    
    Change-Id: Ic58c2bfadbd34bb6ba7dc0d2b74871cc90b0a74f
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:7902, v8:7790
    Reviewed-on: https://chromium-review.googlesource.com/1125680Reviewed-by: 's avatarYang Guo <yangguo@chromium.org>
    Commit-Queue: Yang Guo <yangguo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#54192}
    b1cf1e1e
Name
Last commit
Last update
..
api Loading commit data...
asmjs Loading commit data...
base Loading commit data...
compiler Loading commit data...
compiler-dispatcher Loading commit data...
heap Loading commit data...
interpreter Loading commit data...
libplatform Loading commit data...
parser Loading commit data...
wasm Loading commit data...
zone Loading commit data...
BUILD.gn Loading commit data...
DEPS Loading commit data...
allocation-unittest.cc Loading commit data...
bigint-unittest.cc Loading commit data...
cancelable-tasks-unittest.cc Loading commit data...
char-predicates-unittest.cc Loading commit data...
code-stub-assembler-unittest.cc Loading commit data...
code-stub-assembler-unittest.h Loading commit data...
counters-unittest.cc Loading commit data...
detachable-vector-unittest.cc Loading commit data...
eh-frame-iterator-unittest.cc Loading commit data...
eh-frame-writer-unittest.cc Loading commit data...
locked-queue-unittest.cc Loading commit data...
object-unittest.cc Loading commit data...
register-configuration-unittest.cc Loading commit data...
run-all-unittests.cc Loading commit data...
source-position-table-unittest.cc Loading commit data...
strings-storage-unittest.cc Loading commit data...
test-helpers.cc Loading commit data...
test-helpers.h Loading commit data...
test-utils.cc Loading commit data...
test-utils.h Loading commit data...
testcfg.py Loading commit data...
unicode-unittest.cc Loading commit data...
unittests.status Loading commit data...
utils-unittest.cc Loading commit data...
value-serializer-unittest.cc Loading commit data...