[v8-dev] Re: Fix very strange bug in FindBreakLocationFromAddress algorithm (issue 15737023)

2013-06-07 Thread prybin
Committed patchset #4 manually as r15014. https://codereview.chromium.org/15737023/ -- -- 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 gro

[v8-dev] Re: Fix very strange bug in FindBreakLocationFromAddress algorithm (issue 15737023)

2013-06-07 Thread prybin
On 2013/06/07 12:14:51, Yang wrote: On 2013/06/07 11:34:01, peter.rybin wrote: LGTM. Please add a comment on "pc - 1", something about the pc pointing to the instruction after the break point. Alternatively, rename FindBreakLocationFromAddress to something like FindClosestBreakLocationBef

[v8-dev] Re: Fix very strange bug in FindBreakLocationFromAddress algorithm (issue 15737023)

2013-06-07 Thread yangguo
On 2013/06/07 11:34:01, peter.rybin wrote: LGTM. Please add a comment on "pc - 1", something about the pc pointing to the instruction after the break point. Alternatively, rename FindBreakLocationFromAddress to something like FindClosestBreakLocationBeforeAddress and leave the current impleme

[v8-dev] Re: Fix very strange bug in FindBreakLocationFromAddress algorithm (issue 15737023)

2013-06-07 Thread prybin
https://codereview.chromium.org/15737023/ -- -- 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

[v8-dev] Re: Fix very strange bug in FindBreakLocationFromAddress algorithm (issue 15737023)

2013-05-23 Thread prybin
On 2013/05/23 09:33:48, Yang wrote: On 2013/05/23 08:05:08, Jakob wrote: > Drive-by comment: > Does this pass tests? I've stumbled over this oddity before, but had to conclude > that fixing it is a can of worms. See > https://code.google.com/p/v8/issues/detail?id=2047 for my findings. It looks

[v8-dev] Re: Fix very strange bug in FindBreakLocationFromAddress algorithm (issue 15737023)

2013-05-23 Thread yangguo
On 2013/05/23 08:05:08, Jakob wrote: Drive-by comment: Does this pass tests? I've stumbled over this oddity before, but had to conclude that fixing it is a can of worms. See https://code.google.com/p/v8/issues/detail?id=2047 for my findings. It looks correct, but as Jakob said, there seem to

[v8-dev] Re: Fix very strange bug in FindBreakLocationFromAddress algorithm (issue 15737023)

2013-05-23 Thread jkummerow
Drive-by comment: Does this pass tests? I've stumbled over this oddity before, but had to conclude that fixing it is a can of worms. See https://code.google.com/p/v8/issues/detail?id=2047 for my findings. https://codereview.chromium.org/15737023/ -- -- v8-dev mailing list v8-dev@googlegroups.