Closed Bug 1753272 Opened 4 months ago Closed 3 months ago

Enable profiler stackwalking in Mac builds which don't have --enable-profiling

Categories

(Core :: Gecko Profiler, task, P2)

All
macOS
task

Tracking

()

RESOLVED FIXED
98 Branch
Tracking Status
firefox98 --- fixed

People

(Reporter: mstange, Assigned: mstange)

Details

Attachments

(1 file)

On macOS, the profiler requires frame pointers in order to get C++ stacks. One way to force frame pointers is to use --enable-profiling, which is currently set for Nightly builds (by way of --enable-instruments), and which sets the MOZ_PROFILING define. But we can also have frame pointers in other ways, for example bug 1753107 just enabled them on all channels for macOS arm64 builds even if --enable-profiling is not set. And system libraries always have frame pointers.
So we should just enable stackwalking on macOS always. This will make the profiler's "stackwalk" feature available even in Firefox x86_64 Release builds, which currently don't have frame pointers. There it will only give us stacks for system libraries and not for XUL, but that's better than nothing.

Severity: -- → N/A
Priority: -- → P2
Pushed by mstange@themasta.com:
https://hg.mozilla.org/integration/autoland/rev/6b62a8e4b721
On macOS, always enable frame pointer stack walking, even in builds which may not have frame pointers. r=gerald
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 98 Branch
You need to log in before you can comment on or make changes to this bug.