1120924 - TraceLogger: deduplicate emitEpilogue and emitPrologue

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, defect)

Description

[Hannes Verschore [:h4writer]]

see bug 1119220 comment 3

Comment 1

[Hannes Verschore [:h4writer]]

Attachment: Patch

Comment 2

[Benjamin Bouvier [:bbouvier] (inactive)]

Comment on attachment 8548172 [details] [diff] [review]
Patch

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

How about also putting the #ifdef in emitTracelogIonStart/Stop()? They'll get optimized away by the compiler anyways

Comment 3

[Hannes Verschore [:h4writer]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/d8467e9bbae8

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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