454163 - Failed to compilejstracer on Linux-64, DEBUG build

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Igor Bukanov]

Attachment: fix

the current code contains:

#if defined(DEBUG) && defined(NANOJIT_IA32)
    uint64 cycles = rdtsc() - start;
#else
    debug_only_v(uint64 cycles = 0;)
#endif

    debug_only_v(printf("leaving trace at %s:%u@%u, op=%s, lr=%p, exitType=%d, sp=%d, ip=%p, "
                        "cycles=%llu\n",
...
                        cycles));


That uint64 cycles = 0; does not work given definition of debug_only_v:

#define debug_only_v(x) if (verbose_debug) { x; }

The attached patch fixes that by printing cycles only for ia32.

Comment 1

[Igor Bukanov]

landed - http://hg.mozilla.org/mozilla-central/rev/d87cf8e2bd02