• Clemens Hammacher's avatar
    Revert "Reland: [Compiler] Use CompilationCache for StreamedScript compilation." · d7e59efa
    Clemens Hammacher authored
    This reverts commit 25427203.
    
    Reason for revert: code-coverage failures on gc-stress bot: https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/builds/17956
    
    Original change's description:
    > Reland: [Compiler] Use CompilationCache for StreamedScript compilation.
    > 
    > Previously GetSharedFunctionInfoForStreamedScript didn't either check the
    > compilation cache or put the result of compilation into the compilation
    > cache. This would mean future compiles would need to re-parse / compile
    > the same script even if the isolate had already seen it. This CL
    > fixes this.
    > 
    > Also refactors the compilation pipelines to ensure we call debug->OnAfterCompile()
    > for all script compiles even when loading from a cache.
    > 
    > BUG=v8:5203
    > Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    > 
    > Change-Id: I0a74c5b67bfaca5e50511d5f72da0ab53d8457f6
    > Reviewed-on: https://chromium-review.googlesource.com/937724
    > Commit-Queue: Ross McIlroy <rmcilroy@chromium.org>
    > Reviewed-by: Mythri Alle <mythria@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#51594}
    
    TBR=rmcilroy@chromium.org,yangguo@chromium.org,mythria@chromium.org
    
    Change-Id: I784b9eeff75a677b9f2276fa05a0d1af09772baa
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:5203
    Cq-Include-Trybots: master.tryserver.chromium.linux:linux_chromium_rel_ng
    Reviewed-on: https://chromium-review.googlesource.com/939401Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#51596}
    d7e59efa
compiler.cc 73.6 KB