876366 - Avoid pointer to long casts in PerfSpewer::writeProfile.

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jacek Caban]

Attachment: fix

unsigned long is 32-bit on 64-bit Windows, so those casts are problematic there. The patch uses uintptr_t instead and avoids casts in printf call by using "%p" format.

Comment 1

[Niko Matsakis [:nmatsakis]]

Comment on attachment 754395 [details] [diff] [review]
fix

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

::: js/src/ion/PerfSpewer.cpp
@@ +135,5 @@
>      uint32_t thisFunctionIndex = nextFunctionIndex++;
>  
>      if (PerfFuncEnabled()) {
>          fprintf(fp_,
> +                "%p %p %s:%d: Func%02d\n",

I specifically chose not to use `%p` because it outputs a string like `0xABCDEFGH`, but I want `ABCDEFGH`. Perhaps rewrite the patch to use `PRIxPTR`? Alternatively, is there a way to make the cast not warn (reinterpret_cast, perhaps?)

Comment 2

[Jacek Caban]

I pushed a fix using reinterpret_cast. Thanks for review.

https://hg.mozilla.org/integration/mozilla-inbound/rev/7c0c49dd4e91

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7c0c49dd4e91