Closed Bug 2033336 Opened 4 months ago Closed 3 hours ago

Code coverage has stopped recording files in UNIFIED_SOURCES

Categories

(Firefox Build System :: General, defect)

defect

Tracking

(firefox-esr140 unaffected, firefox150 unaffected, firefox151 wontfix, firefox152 wontfix, firefox153 fix-optional, firefox158 fixed)

RESOLVED FIXED
158 Branch
Tracking Status
firefox-esr140 --- unaffected
firefox150 --- unaffected
firefox151 --- wontfix
firefox152 --- wontfix
firefox153 --- fix-optional
firefox158 --- fixed

People

(Reporter: darktrojan, Assigned: nicolas.guichard)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

No code coverage data has been recorded after 2026-04-16 for many .cpp files (e.g. the coverage graph for Preferences.cpp stops before the end). It seems that files outside of unified builds are not affected (e.g. deflate.c has more recent data), so I think it's very likely caused by bug 2031965.

:emilio, since you are the author of the regressor, bug 2031965, could you take a look? Also, could you set the severity field?

For more information, please visit BugBot documentation.

Flags: needinfo?(emilio)

I'm not familiar with our code coverage instrumentation. Do you know how we map it to the source tree? It seems believable but I don't know where to start looking :)

Flags: needinfo?(emilio) → needinfo?(mcastelluccio)

Set release status flags based on info from the regressing bug 2031965

There are a couple of places in https://searchfox.org/firefox-main/source/testing/mozharness/mozharness/mozilla/testing/codecoverage.py that might need to be updated (see everything related to prefix, there's both an option passed to grcov and also GCOV_PREFIX and GCOV_PREFIX_STRIP).

I also see other files in the tree that set GCOV_PREFIX.

Unfortunately it's a bit brittle... So you'd have to do a few attemps with try pushes.

Flags: needinfo?(mcastelluccio)

The severity field is not set for this bug.
:ahochheiden, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(ahochheiden)
Severity: -- → S3
Flags: needinfo?(ahochheiden)
See Also: → 2066709
See Also: → 2070023

This new grcov release includes a bugfix for GCNOs pointing to C++
files outside the objdir with relative paths, as we typically get when
building with unified sources.

See https://github.com/mozilla/grcov/pull/1519 for details.

Assignee: nobody → nicolas.guichard
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 3 hours ago
Resolution: --- → FIXED
Target Milestone: --- → 158 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: