Open Bug 1543849 Opened 5 years ago Updated 6 months ago

[meta] Evaluate and update our PGO training set

Categories

(Firefox Build System :: General, enhancement)

enhancement

Tracking

(Not tracked)

REOPENED

People

(Reporter: erahm, Unassigned)

References

(Depends on 9 open bugs)

Details

(Keywords: meta)

2 years ago we added speedometer to our PGO run in bug 1356652, it's time to start looking at tests that exercise newer pieces of our codebase. We have PGO for rust in the works in bug 1437452, which means we could see improvements by adding tests that thoroughly exercise our rust codebase.

Initial conversations have suggested the following perf tests that we already run in automation might make sense to try out:

There are concerns of over training, so we'll want to evaluate each test set first prior to committing to adding it to our overall PGO training set.

Depends on: 1543852
Depends on: 1543853
Type: defect → enhancement
Depends on: 1543854

It's interesting to note that one possible advantage of the 3-tier PGO builds we're setting up across platforms, is that we don't necessarily have to have one, linear, profiling script anymore. We could spawn multiple tasks running different profiling scripts, and merge the profiling data before feeding it to the last tier of the PGO build.

Depends on: 1543855
Depends on: 1543856
Depends on: 1596172
Depends on: 1596182

We don't have the bandwidth to work on this currently. So, closing.
If someone wants to help, please reopen this bug and don't hesitate to contact dmajor or myself

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INACTIVE

I did a short talk about the framework of PGO at a perf meeting today, and several people made some suggestions in this area that I think are worth following up on, so I'm gonna reopen per Sylvestre's earlier comment, and add some dep bugs and maybe ping some of the existing deps.

Status: RESOLVED → REOPENED
Resolution: INACTIVE → ---
Depends on: 1666884
Depends on: 1666886
Depends on: 1666890
Depends on: 1503559
See Also: 1503559
Severity: normal → S3
Depends on: 1862309
You need to log in before you can comment on or make changes to this bug.