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

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

commit 75350f1ef0ebf189ef3583183eae694a25597fc0
Author: yangguo <[email protected]>
Date: Tue Jun 16 07:11:11 2015

Debugger: require debugger to be active when dealing with breaks.

This invariant will save us some head ache.
The changes to test-debug/DebugStub is due to the fact that it abuses
the ability to set break points in code that has no debug break slots.
This is now no longer possible.

[email protected]
BUG=v8:4132
LOG=N

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

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

[modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/src/debug.cc [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/src/debug.h [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/src/runtime/runtime-debug.cc [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/cctest/cctest.h [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/cctest/test-api.cc [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/cctest/test-debug.cc [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/cctest/test-heap.cc [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/debug-liveedit-breakpoints.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/debug-liveedit-patch-positions.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/debug-liveedit-stack-padding.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/debug-script-breakpoints.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/debug-script.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/debug-stepin-positions.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/deserialize-script-id.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/es6/debug-evaluate-blockscopes.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/es6/generators-relocation.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/regress/regress-1586.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/regress/regress-2318.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/regress/regress-crbug-424142.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/regress/regress-crbug-474297.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/regress/regress-debug-code-recompilation.js [modify] http://crrev.com/75350f1ef0ebf189ef3583183eae694a25597fc0/test/mjsunit/regress/regress-prepare-break-while-recompile.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