crash, regression, top100, verified188.8.131.52, verified1.8.1
or just var r3="-1"; r3++;
backing out bug 316885 stops the crash Apparently I can't make the dependency because I'm not in the security group.
Assignee: general → brendan
Created attachment 204148 [details] [diff] [review] fix I'm about to check this in. /be
Attachment #204148 - Flags: review?(shaver)
Comment on attachment 204148 [details] [diff] [review] fix r=shaver. (This looks like code I misreviewed before, alas.)
Attachment #204148 - Flags: review?(shaver) → review+
Created attachment 204149 [details] [diff] [review] the right fix We need that extra stack slot for all post-increment operator forms except name ops (which consume no stack slots, and produce one slot, so we can "pre-use" that result slot for the pre-increment result). /be
Attachment #204149 - Flags: review?(shaver)
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Comment on attachment 204149 [details] [diff] [review] the right fix r=shaver, makes sense. (Though after my review history on this bug, one wonders what value I'm adding here!)
Attachment #204149 - Flags: review?(shaver) → review+
*** Bug 317697 has been marked as a duplicate of this bug. ***
*** Bug 318066 has been marked as a duplicate of this bug. ***
/cvsroot/mozilla/js/tests/js1_5/Regress/regress-317714-01.js,v <-- regress-317714-01.js initial revision: 1.1 /cvsroot/mozilla/js/tests/js1_5/Regress/regress-317714-02.js,v <-- regress-317714-02.js initial revision: 1.1
Whiteboard: required for 316885
This is required for nominated blocker bug 316885 -- I assume the first patch is obsolete? Please put approval requests on the right patch
Whiteboard: required for 316885 → required for 316885, which patch?
It looks like only the second patch landed on the trunk.
Comment on attachment 204149 [details] [diff] [review] the right fix a=dveditz for drivers
v 2006-01-11 184.108.40.206, 1.8.1, trunk windows/linux/mac
Status: RESOLVED → VERIFIED
Keywords: fixed220.127.116.11, fixed1.8.1 → verified18.104.22.168, verified1.8.1
Not needed on aviary101/moz17 branches per caillon in bug 316885
Whiteboard: required for 316885, which patch? → required for 316885 in 1.8, qawanted for 1.7
No crash on Firefox 1.0.x/Mozilla 1.7.x from 2006-02-02 on winxp or linux.
Whiteboard: required for 316885 in 1.8, qawanted for 1.7 → required for 316885 in 1.8
You need to log in before you can comment on or make changes to this bug.