TM: bizarre results for ((1 * (1)) | "")

VERIFIED FIXED in mozilla1.9.1b2

Status

()

Product:
Core
Component:
JavaScript Engine
VERIFIED FIXED
Reported:
10 years ago
Modified:
10 years ago

People

(Reporter: Jesse Ruderman, Assigned: gal)

Tracking

({testcase, verified1.9.1})

Version:
Trunk
Target:
mozilla1.9.1b2
x86
Mac OS X
Keywords:
testcase, verified1.9.1
Points:
---
Bug Flags:
blocking1.9.1 +
in-testsuite +
in-litmus -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

patch
1.81 KB, patch
Edwin Smith
: review+
dvander
: review+
Details | Diff | Splinter Review
(Reporter)

Description

•
10 years ago 
e.js:

empty = [];
out = [];
for (var j=0;j<10;++j) { empty[42]; out.push((1 * (1)) | ""); }
print(uneval(out));

./js -j ~/e.js
[1, 1, 1965522873, 9, 16, 25, 36, 49, 64, 81]
(Reporter)

Comment 1

•
10 years ago 
Why do I need the "empty[42]" part?  That's an artifact of how the fuzzer works, rather than something the fuzzer generated :(
(Assignee)

Comment 2

•
10 years ago 
Created attachment 348523 [details] [diff] [review]
patch
Assignee: general → gal
Status: NEW → ASSIGNED
Attachment #348523 - Flags: review?(edwsmith)
(Assignee)

Comment 3

•
10 years ago 
This is a super evil bug. We can't do CSE across in-trace branch labels. Will push with r=me until someone + or - it.
 (Assignee)

Updated

•
10 years ago
Attachment #348523 - Flags: review?(danderson)
Attachment #348523 - Flags: review?(danderson) → review+
 (Assignee)

Comment 4

•
10 years ago 
http://hg.mozilla.org/tracemonkey/rev/55e30117ecee
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
 (Assignee)

Comment 5

•
10 years ago 
ccing ed and graydon

Updated

•
10 years ago
Attachment #348523 - Flags: review?(edwsmith) → review+

Comment 6

•
10 years ago 
reopening, marking blocking beta2, will close once landed on m-c.
Status: RESOLVED → REOPENED
Flags: blocking1.9.1+
Resolution: FIXED → ---

Updated

•
10 years ago
Target Milestone: --- → mozilla1.9.1b2

Comment 7

•
10 years ago 
Fixed in the merge pushed by vlad on Nov 18 14:11:14 2008 -0800:
http://hg.mozilla.org/mozilla-central/rev/e8ed5d4bf531
Status: REOPENED → RESOLVED
Last Resolved: 10 years ago → 10 years ago
Resolution: --- → FIXED

Comment 8

•
10 years ago 
Checking in js1_5/extensions/regress-465276.js;
/cvsroot/mozilla/js/tests/js1_5/extensions/regress-465276.js,v  <--  regress-465276.js
initial revision: 1.1
Flags: in-testsuite+
Flags: in-litmus-
Keywords: fixed1.9.1

Comment 10

•
10 years ago 
v 1.9.1, 1.9.2
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
You need to log in before you can comment on or make changes to this bug.