• 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
compilation-dependencies.cc 5.69 KB