Closed
Bug 2008110
Opened 8 months ago
Closed 8 months ago
Linux code coverage is missing C/C++ data
Categories
(Testing :: Code Coverage, defect)
Testing
Code Coverage
Tracking
(firefox-esr115 wontfix, firefox-esr140 wontfix, firefox146 wontfix, firefox147 wontfix, firefox148 fixed)
RESOLVED
FIXED
148 Branch
People
(Reporter: nicolas.guichard, Assigned: nicolas.guichard)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression)
Attachments
(1 file)
The raw code coverage data (.gcno and .gcda files) are generated by Clang, but grcov tries parsing them with GCC's gcov.
There are 2 solutions to this issue:
- make grcov use llvm-cov instead
- pass the --llvm flag to grcov to have it parse the gcno files directly
The 2nd option is what is already done on Windows and macOS, so I think we should just extend that to Linux.
|
Assignee | |
Comment 1•8 months ago
|
||
grcov calls GCC's gcov by default to parse gcno files, this tells it
instead that the files were generated by LLVM and that it should parse
them as such.
Without this, Linux code coverage reports were missing all C/C++ files.
|
||
Comment 2•8 months ago
|
||
Set release status flags based on info from the regressing bug 1499663
status-firefox146:
--- → affected
status-firefox147:
--- → affected
status-firefox148:
--- → affected
status-firefox-esr140:
--- → affected
|
||
Updated•8 months ago
|
Pushed by nicolas.guichard@kdab.com:
https://github.com/mozilla-firefox/firefox/commit/645c25302753
https://hg.mozilla.org/integration/autoland/rev/ced5b1759b81
Always call grcov with --llvm flag. r=marco
|
||
Comment 4•8 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 148 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•