Comment #4 on issue 4132 by bugdro...@chromium.org: Improve performance when break points are set.
https://code.google.com/p/v8/issues/detail?id=4132#c4

The following revision refers to this bug:
https://chromium.googlesource.com/v8/v8.git/+/c1669450834436508e0007885eb7ac266cbcf083

commit c1669450834436508e0007885eb7ac266cbcf083
Author: yangguo <yang...@chromium.org>
Date: Fri Jun 19 14:40:22 2015

Keep a canonical list of shared function infos.

Each Script object now keeps a WeakFixedArray of SharedFunctionInfo
objects created from this script.

This way, when compiling a function, we do not create duplicate shared
function info objects when recompiling with either compiler.

This fixes a class of issues in the debugger, where we set break points
on one shared function info, but functions from duplicate shared function
infos are not affected.

LOG=N
BUG=v8:4132

Review URL: https://codereview.chromium.org/1183733006

Cr-Commit-Position: refs/heads/master@{#29151}

[modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/arm/full-codegen-arm.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/arm64/full-codegen-arm64.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/ast.cc
[modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/ast.h
[modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/bootstrapper.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/codegen.h [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/compiler.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/compiler.h [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/compiler/ast-graph-builder.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/debug.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/factory.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/full-codegen.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/hydrogen.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/ia32/full-codegen-ia32.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/liveedit.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/mips/full-codegen-mips.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/mips64/full-codegen-mips64.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/objects-inl.h [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/objects-printer.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/objects.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/objects.h [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/ppc/full-codegen-ppc.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/runtime/runtime-function.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/snapshot/serialize.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/typing.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/x64/full-codegen-x64.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/src/x87/full-codegen-x87.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/test/cctest/test-heap.cc [modify] http://crrev.com/c1669450834436508e0007885eb7ac266cbcf083/test/mjsunit/regress/regress-4121.js


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to