1626905 - JIT Spew in Backtracking Allocator is always enabled in debug builds

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect, P3)

Description

[Christian Holler (:decoder)]

In the backtracking allocator, there are several JitSpew statements that are very costly both in argument evaluation and string allocation. These statements are always on in debug builds, even if we are not logging anything. The number of temporary allocations from these is enough to produce a huge spike in my heap profiling attempts.

After discussing with Jan, we concluded that providing a macro, e.g. JitSpewIfEnabled would be worthwhile and the macro only evaluates the arguments if the particular JitSpew channel is active.

I've made the necessary changes in the backtracking allocator and haven't seen more temporary allocations from other places, so I assume they are simply not stressed enough to make a significant difference (then again, once we have this, we might as well use it everywhere).

Comment 1

[Christian Holler (:decoder)]

Attachment: Bug 1626905 - Evaluate JitSpew in BacktrackingAllocator only if required. r?jandem

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c47be0a726ba
Evaluate JitSpew in BacktrackingAllocator only if required. r=jandem

Comment 3

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/c47be0a726ba

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:decoder, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.