• 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
benchmarks Loading commit data...
build_overrides Loading commit data...
custom_deps Loading commit data...
docs Loading commit data...
gni Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.editorconfig Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.vpython Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...
snapshot_toolchain.gni Loading commit data...