Skip to content

deps: cherry-pick 1420e44db0 from upstream V8

Rodrigo Muino Tomonari requested to merge github/fork/TimothyGu/c8 into master

Original commit message:

[coverage] Correctly free DebugInfo in the absence of breakpoints

It's quite possible for DebugInfos to exist without the presence of a
bytecode array, since DebugInfos are created for all functions for which
we have a CoverageInfo. Free such objects properly.

Also move the corresponding deletion of CoverageInfos on unload up
before the early exit.

Bug: v8:6000
Change-Id: Idde45b222290aa8b6828b61ff2251918b8ed2aed
Reviewed-on: https://chromium-review.googlesource.com/664811
Reviewed-by: Yang Guo <yangguo@chromium.org>
Commit-Queue: Jakob Gruber <jgruber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#48024}

Fixes crash when passing Profiler.startPreciseCoverage before Debug.paused is received.

Refs: https://github.com/bcoe/c8/pull/6#discussion_r153121287

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

deps (V8)

Merge request reports