Most profilers (VTune and Quantify) are much happier if we pass /FIXED:NO to LDFLAGS in browser/app/Makefile.in.
Should we do it with a new config flag or just always do it with --enable-debugger-info-modules set?
Test _DEBUG_CFLAGS, and if that is set add the flag.
Does that work with: --enable-optimize --disable-debug --enable-debugger-info-modules
yes, see http://lxr.mozilla.org/mozilla/source/config/config.mk#193
Will this help in the case of a static executable as well? It's tough to make sense of Quantify results when I profile a static build.
If the static build has --enable-debugger-info-modules=all, yes
Hey, this is a very old bug, and I wonder if this is still needed, or if we can close this?
I am looking at this documentation on VTune support for Firefox: https://developer.mozilla.org/en-US/docs/Profiling_with_VTune , and I was able to build a VTune-enabled build of Firefox on Windows with the following .mozconfig (not all flags below are relevant, but pasting it in full to be exact):
ac_add_options --enable-profiling # This is implied by --enable-vtune, so was redundant.
Looking at the generated VTune profile, I am seeing JS code interleaved with C++ code, which is nice (although there's something that looks a bit fishy, but that's probably story for another bug).
The reason I am asking is that I want to bring the VTune MDN page up to date with new flags, and I am wondering whether I can remove the reference on that page to this bug? I'm thinking yes, but just wanted to confirm.
If VTune works for you then that seems fine.