453674 - TM: jstracer.cpp build failure on 64bit Linux error: 'cycles' was not declared in this scope

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Bob Clary [:bc] (inactive)]

mozilla-central centos5 64 xeon 5450

error: 'cycles' was not declared in this scope

full warnings/error log

jstracer.cpp: In constructor 'TraceRecorder::TraceRecorder(JSContext*, nanojit::GuardRecord*, nanojit::Fragment*, TreeInfo*, unsigned int, uint8*, uint8*, nanojit::GuardRecord*)':
jstracer.cpp:803: warning: format '%u' expects type 'unsigned int', but argument 4 has type 'long int'
jstracer.cpp: In member function 'bool TraceRecorder::checkType(jsval&, uint8, bool&)':
jstracer.cpp:1564: warning: format '%d' expects type 'int', but argument 2 has type 'ptrdiff_t'
jstracer.cpp: In member function 'void TraceRecorder::prepareTreeCall(nanojit::Fragment*)':
jstracer.cpp:1722: warning: format '%d' expects type 'int', but argument 2 has type 'ptrdiff_t'
jstracer.cpp:1722: warning: format '%d' expects type 'int', but argument 3 has type 'ptrdiff_t'
jstracer.cpp:1722: warning: format '%d' expects type 'int', but argument 4 has type 'ptrdiff_t'
jstracer.cpp: In function 'bool js_RecordLoopEdge(JSContext*, TraceRecorder*, jsbytecode*, uintN&)':
jstracer.cpp:2191: warning: format '%d' expects type 'int', but argument 2 has type 'long int'
jstracer.cpp:2191: warning: format '%d' expects type 'int', but argument 3 has type 'long int'
jstracer.cpp:2191: warning: format '%d' expects type 'int', but argument 4 has type 'long int'
jstracer.cpp: In function 'nanojit::GuardRecord* js_ExecuteTree(JSContext*, nanojit::Fragment**, uintN&, nanojit::GuardRecord**)':
jstracer.cpp:2217: warning: format '%u' expects type 'unsigned int', but argument 4 has type 'long int'
jstracer.cpp:2365: warning: unused variable 'cycles'
jstracer.cpp:2368: error: 'cycles' was not declared in this scope
jstracer.cpp: In function 'void js_AbortRecording(JSContext*, jsbytecode*, const char*)':
jstracer.cpp:2508: warning: format '%d' expects type 'int', but argument 3 has type 'long int'
jstracer.cpp: In function 'void js_FinishJIT(JSTraceMonitor*)':
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 2 has type 'uint64'
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 3 has type 'uint64'
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 4 has type 'uint64'
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 5 has type 'uint64'
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 6 has type 'uint64'
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 7 has type 'uint64'
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 8 has type 'uint64'
jstracer.cpp:2583: warning: format '%llu' expects type 'long long unsigned int', but argument 9 has type 'uint64'
jstracer.cpp:2586: warning: format '%llu' expects type 'long long unsigned int', but argument 2 has type 'uint64'
jstracer.cpp:2586: warning: format '%llu' expects type 'long long unsigned int', but argument 3 has type 'uint64'
jstracer.cpp:2586: warning: format '%llu' expects type 'long long unsigned int', but argument 4 has type 'uint64'
jstracer.cpp:2586: warning: format '%llu' expects type 'long long unsigned int', but argument 5 has type 'uint64'
jstracer.cpp: In member function 'bool TraceRecorder::guardDenseArrayIndex(JSObject*, jsint, nanojit::LIns*, nanojit::LIns*, nanojit::LIns*)':
jstracer.cpp:3569: warning: overflow in implicit constant conversion
jstracer.cpp: At global scope:
jstracer.cpp:123: warning: 'did_we_check_sse2' defined but not used

Comment 1

[Bob Clary [:bc] (inactive)]

PS. debug failure only.

Comment 2

[David Anderson [:dvander] - inactive, e-mail if emergency]

I don't recall enabling the JIT yet for x86_64, did you manually enable it?

Comment 3

[Bob Clary [:bc] (inactive)]

not intentionally. there is nothing in my mozconfig that mentions jit. note this was with mozilla-central and not tracemonkey repo. I thought TM: was reserved for the tracemonkey repo.

Comment 4

[Bob Clary [:bc] (inactive)]

ignore the mozconfig comment. the build was a generic debug shell build with Makefile.ref

Comment 5

[Andreas Gal :gal]

TM is for anything that concerns the JIT. Feel free to use it broadly. Its easier for me to remove it than find bugs that don't have it.

Comment 6

[David Anderson [:dvander] - inactive, e-mail if emergency]

Pushed fixes as changeset ee704fbeb74f.  JIT turned off until I can track some runtime regressions down.

Comment 7

[Bob Clary [:bc] (inactive)]

David, don't forget mozilla-central.

Comment 8

[David Anderson [:dvander] - inactive, e-mail if emergency]

My account doesn't have permission to push to m-c.

Comment 9

[Bob Clary [:bc] (inactive)]

marcia, brendan: comment 8.