Closed Bug 495907 Opened 15 years ago Closed 15 years ago

upvar incorrectness with "let"

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: jruderman, Assigned: dmandelin)

References

Details

(Keywords: testcase, verified1.9.1, Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

Patch with test case
3.36 KB, patch
gal
: review+
Details | Diff | Splinter Review 
var o = [];
for (let a = 0; a < 4; ++a) {
    (function () {for (var b = 0; b < 2; ++b) {o.push(a);}}());
}
print(o.join(""));

00112233 without JIT (correct)
00112222 with JIT
I can reproduce this bug on the 1.9.1 branch.
Flags: blocking1.9.1?
Flags: blocking1.9.1? → blocking1.9.1+
Whiteboard: [needs assignee]
Assignee: general → dmandelin
Whiteboard: [needs assignee]

        Attachment #381119 -
        Flags: review?(gal)

    
  

        Attachment #381119 -
        Flags: review?(gal) → review+

    
    

    
  
Pushed to TM as b9e104ec562f.

    
  
Whiteboard: fixed-in-tracemonkey

    
    

    
  
http://hg.mozilla.org/mozilla-central/rev/b9e104ec562f
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED

    
    

    
  
js1_8_1/regress/regress-495907.js
v 1.9.1, 1.9.2
Status: RESOLVED → VERIFIED
Flags: in-testsuite+
Keywords: fixed1.9.1 → verified1.9.1

    
    

    
  
/cvsroot/mozilla/js/tests/js1_8_1/regress/regress-495907.js,v  <--  regress-495907.js
initial revision: 1.1

          You need to log in
          before you can comment on or make changes to this bug.