Last Comment Bug 754557 - JS fails to build with --enable-methodjit-spew
: JS fails to build with --enable-methodjit-spew
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla15
Assigned To: Mike Hommey [:glandium]
:
: Jason Orendorff [:jorendorff]
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-12 03:23 PDT by Mike Hommey [:glandium]
Modified: 2012-05-15 06:28 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Fix building with --enable-methodjit-spew without --enable-debug (3.00 KB, patch)
2012-05-12 03:40 PDT, Mike Hommey [:glandium]
no flags Details | Diff | Splinter Review
Fix building with --enable-methodjit-spew without --enable-debug (4.63 KB, patch)
2012-05-12 06:47 PDT, Mike Hommey [:glandium]
dmandelin: review+
Details | Diff | Splinter Review

Description Mike Hommey [:glandium] 2012-05-12 03:23:18 PDT
/home/mh/mozilla-central/js/src/methodjit/Compiler.cpp: In member function ‘js::mjit::CompileStatus js::mjit::Compiler::pushActiveFrame(JSScript*, uint32_t)’:
/home/mh/mozilla-central/js/src/methodjit/Compiler.cpp:442:45: error: ‘const struct js::analyze::LifetimeVariable’ has no member named ‘print’
/home/mh/mozilla-central/js/src/methodjit/Compiler.cpp:449:45: error: ‘const struct js::analyze::LifetimeVariable’ has no member named ‘print’
make[1]: *** [Compiler.o] Error 1
Comment 1 Mike Hommey [:glandium] 2012-05-12 03:40:26 PDT
Created attachment 623414 [details] [diff] [review]
Fix building with --enable-methodjit-spew without --enable-debug
Comment 2 Mike Hommey [:glandium] 2012-05-12 04:07:16 PDT
Comment on attachment 623414 [details] [diff] [review]
Fix building with --enable-methodjit-spew without --enable-debug

This breaks --enable-debug :(
Comment 3 Mike Hommey [:glandium] 2012-05-12 06:47:54 PDT
Created attachment 623421 [details] [diff] [review]
Fix building with --enable-methodjit-spew without --enable-debug
Comment 5 Ed Morley [:emorley] 2012-05-15 06:28:01 PDT
https://hg.mozilla.org/mozilla-central/rev/835cdc6f333f

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