Comment #13 on issue 4132 by [email protected]: Improve performance when break points are set.
https://code.google.com/p/v8/issues/detail?id=4132#c13

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

commit 35c28ce0a742e58346d2dea009428cacd442040d
Author: yangguo <[email protected]>
Date: Mon Jul 20 14:53:28 2015

Debugger: prepare code for debugging on a per-function basis.

Prior to this patch, we enter a global debug mode whenever a break point
is set. By entering this mode, all code is deoptimized and activated
frames are recompiled and redirected to newly compiled debug code.

After this patch, we only deoptimize/redirect for functions we want to
debug. Trigger for this is Debug::EnsureDebugInfo, and having DebugInfo
object attached to the SFI prevents optimization/inlining.

The result is that we can have optimized code for functions without break
points alongside functions that do have break points, which are not
optimized.

[email protected], [email protected]
BUG=v8:4132
LOG=Y

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

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

[modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/arm/assembler-arm-inl.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/arm64/assembler-arm64-inl.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/background-parsing-task.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/bailout-reason.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/compiler.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/compiler.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/compiler/js-inlining.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/debug.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/debug.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/heap/mark-compact.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/hydrogen.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/ia32/assembler-ia32-inl.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/ic/ic.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/liveedit.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/mips/assembler-mips-inl.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/mips64/assembler-mips64-inl.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/objects-inl.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/objects.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/objects.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/runtime/runtime-debug.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/src/x64/assembler-x64-inl.h [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/test/cctest/test-debug.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/test/cctest/test-func-name-inference.cc [modify] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/test/mjsunit/debug-evaluate.js [add] http://crrev.com/35c28ce0a742e58346d2dea009428cacd442040d/test/mjsunit/debug-optimize.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
[email protected]
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 [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to