Open Bug 1639786 Opened 4 years ago

profiler_feature_active() and others could be inlined

Categories

(Core :: Gecko Profiler, task, P3)

task

Tracking

()

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.