Closed Bug 752121 Opened 9 years ago Closed 9 years ago

IonMonkey: Disable C1/JSON Spewers in debug builds

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: dvander, Assigned: dvander)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Attached patch fix (obsolete) — Splinter Review
Attachment #622981 - Flags: review?
Attachment #622981 - Flags: review? → review?(sstangl)
This patch enables ion.cfg/ion.json logging in the shell, but disables it in the browser. You can get it back in the browser by setting IONFLAGS to any of its options.
Comment on attachment 622981 [details] [diff] [review]
fix

Review of attachment 622981 [details] [diff] [review]:
-----------------------------------------------------------------

The patch is fine, but it doesn't disable spew for the browser: IonSpewNewFunction(), at the top of TestCompiler(), will call the init() function anyway. That needs to be changed to check against inited_, and EnableIonDebugLogging() should be the sole caller of init().

I think this patch should actually go further and disable spew in the shell by default also. That would let the testsuite run more quickly, and will give us less of a surprise on long-running (emscripten) tests where JSON files of hundreds of megabytes are silently built.
Attachment #622981 - Flags: review?(sstangl)
Attached patch v2Splinter Review
Good catch - I agree, let's disable in the shell too. IONFLAGS=logs gets it back.
Attachment #622981 - Attachment is obsolete: true
Attachment #623366 - Flags: review?(sstangl)
Comment on attachment 623366 [details] [diff] [review]
v2

Review of attachment 623366 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/shell/js.cpp
@@ +101,5 @@
>  #include "jsinterpinlines.h"
>  #include "jsobjinlines.h"
>  #include "jsscriptinlines.h"
>  #include "ion/Ion.h"
> +#include "ion/IonSpewer.h"

Not needed anymore :)
Attachment #623366 - Flags: review?(sstangl) → review+
http://hg.mozilla.org/projects/ionmonkey/rev/c0537e1c5e8b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.