Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now) Reporter
	Description • 15 years ago

(function() {
  eval("\
    (function() {\
      for each(let w in[new Number,[''],new Number,new Number(.5)]) {\
        print(w|\"\");\
        print()\
      }\
    }\
  )")()
})()


asserts 64-bit js debug shell on JM tip with -m and -j at Assertion failed: LIR type error (start of writer pipeline): arg 1 of 'ldq' is 'int' which has type int32 (expected int64): 0 (../nanojit/LIR.cpp:2610)

This occurs both on Mac and Linux, if I'm not wrong.

	Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now) Reporter
	Comment 1 • 15 years ago

I'm quite sure this doesn't occur on TM tip:

$ ./js-dbg-64-tm-darwin -j w6636-reduced.js 
0

0

0

0

$ ./js-dbg-64-jm-darwin -m -j w6636-reduced.js 
0

0

0

0
Assertion failed: LIR type error (start of writer pipeline): arg 1 of 'ldq' is 'int' which has type int32 (expected int64): 0 (../nanojit/LIR.cpp:2610)
Abort trap

	David Anderson [:dvander] - inactive, e-mail if emergency
	Comment 2 • 15 years ago

Hrm, I can't reproduce this. Could you show me the full backtrace? Usually that's enough to fix a LIR type error.

	Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now) Reporter
	Comment 3 • 15 years ago

Backtraces, normal and full.

	Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now) Reporter
	Comment 4 • 14 years ago

I can still reproduce this with the following changeset, on 64-bit Mac debug shell:

http://hg.mozilla.org/users/danderson_mozilla.com/jaegermonkey/rev/0075ab7ae8fe

	Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now) Reporter
	Comment 5 • 14 years ago

This revision seemingly fixed this issue:

The first good revision is:
changeset:   41033:a8780485cc7b
user:        David Anderson
date:        Wed Apr 14 13:51:45 2010 -0700
summary:     [JAEGER] Added JMFLAG for disabling trace hints.

	Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now) Reporter
	Comment 6 • 14 years ago

(In reply to comment #5)
> This revision seemingly fixed this issue:

Or that might have been the wrong revision. Whatever the case, it's now WFM.