Open
Bug 1639786
Opened 4 years ago
profiler_feature_active() and others could be inlined
Categories
(Core :: Gecko Profiler, task, P3)
Core
Gecko Profiler
Tracking
()
NEW
People
(Reporter: mozbugz, Unassigned)
Details
Some profiler functions like profiler_feature_active()
are considered "hot" enough to use RacyFeatures
and avoid locking on the profiler mutex, but they are not inline-able (their implementation is not in GeckoProfiler.h), meaning they probably incur a function call cost.
I think all functions using RacyFeatures()
exclusively could be moved into the header for better inlining where possible.
But we should watch the code size to ensure this doesn't add significantly to it.
You need to log in
before you can comment on or make changes to this bug.
Description
•