1615546 - Remove optimization tracking code from the profiler

Status: RESOLVED FIXED

(Core :: Gecko Profiler, task, P3)

Description

[Jan de Mooij [:jandem]]

We're removing the JS engine code in bug 1614622, but there's also code elsewhere we can remove or clean up:

• TrackOptimizations
• JitOptimizations

Comment 1

[Iain Ireland [:iain]]

This code no longer exists. It looks like it was removed as part of bug 1668219.

Comment 2

[Julien Wajsberg [:julienw]]

I think we still have related code in the profiler, I think this is what [this field][https://searchfox.org/mozilla-central/rev/43cb6eca1c3069d46d589d52ab4949257e630d19/tools/profiler/core/ProfiledThreadData.cpp#69) is about. It's difficult for me to follow the C++ code where we'd actually write a value. Nazim, can you please have a look and file another bug if needed?

Comment 3

[Nazım Can Altınova [:canova][:canaltinova on phabricator]]

Yes, it looks like we still have some code left on the profiler backend side. Let's reopen this so we can cleanup that part. Currently we output all null values for the optimizations on frameTable which is not ideal.

Comment 4

[Nazım Can Altınova [:canova][:canaltinova on phabricator]]

Attachment: Bug 1615546 - Remove the optimizations field from profiler frame table r?julienw,mstange,florian

This requires https://github.com/firefox-devtools/profiler/pull/4432 to be
landed and deployed. We bump the profile version in this patch.

Afaik we never used the optimizations field of the frame table in the profiler
frontend and this pfield was null for more than 3 years (since Bug 1614622).
We don't have any plans to make use of it any time soon. It's good to cleanup
this field so it doesn't clutter the profile data and we can reduce the profile
data size.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a78559e1f67b
Remove the optimizations field from profiler frame table r=julienw

Comment 6

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/a78559e1f67b