Closed Bug 560300 Opened 10 years ago Closed 10 years ago
Remove dead NJ
From what I can tell, nothing enabled by NJ_PROFILE=1 is ever used in TR or TM.
TR patch. Should not land until nanojit changes are merged in, otherwise removing this flag will enable NJ_PROFILE code in release mode.
Assignee: nobody → edwsmith
Status: NEW → ASSIGNED
Attachment #439993 - Flags: review?(rreitmai)
Removed some accidental unrelated changes.
Comment on attachment 439995 [details] [diff] [review] (v2) Removes NJ_PROFILE and related code, since it's all dead. Looks fine, compiles and passes TM trace-tests.
Attachment #439995 - Flags: review?(nnethercote) → review+
Comment on attachment 439995 [details] [diff] [review] (v2) Removes NJ_PROFILE and related code, since it's all dead. TR: http://hg.mozilla.org/tamarin-redux/rev/3e6f00bb55d4
Comment on attachment 439993 [details] [diff] [review] Don't reference NJ_PROFILE in TR TR: http://hg.mozilla.org/tamarin-redux/rev/f455fb80e50f
Whiteboard: fixed-in-nanojit → fixed-in-nanojit, fixed-in-tamarin, fixed-in-tracemonkey
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.