Skip to content

coverage(qnx): build the baseline notes of untested files unoptimised - #704

Merged
4og merged 1 commit into
mainfrom
dcalavrezo_qnx_host_o0
Oct 19, 2026
Merged

4og merged 1 commit into
mainfrom
dcalavrezo_qnx_host_o0

Conversation

@dcalavrezo-qorix

Copy link
Copy Markdown
Contributor

One line in coverage:qnx_cov: --host_cxxopt=-O0.

The scope's baseline .gcno notes of untested files are built in the exec configuration, which takes the host options and defaults to opt. GCC then optimises those translation units and their line tables differ from the test builds; a folded return 0 loses its line, so the 0 % entries of untested files have fewer lines than they should. Found while fixing eclipse-score/coverage_tool#40 and documented in the 0.4.2 release notes and user manual step 4b. The LLVM backend is unaffected (front-end mapping).

Independent of the score_coverage version; Renovate's bump to 0.4.2 comes separately.

The scope's .gcno notes of untested files are built in the exec
configuration, which takes the host options and defaults to opt, so GCC
folded lines that the -O0 test builds have (score_coverage 0.4.2).

Signed-off-by: Dan Calavrezo <195309321+dcalavrezo-qorix@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown
Contributor

Documentation preview for this pull request is available at:
pr-704: https://eclipse-score.github.io/baselibs/pr-704/

@dcalavrezo-qorix

Copy link
Copy Markdown
Contributor Author

Effect on the QNX on-target report (this PR's run vs. main's last run, same 702 gcno notes, 276 tests):

main this PR
Files described by the baseline notes 593 699
Raw / effective line coverage 89.38 % 88.25 %
Unjustified uncovered lines 1,948 2,182

With the host options at -O0 the baseline notes of untested units keep every line and every inlined header function that the optimised build had folded away, so 106 more files get their 0 % entries and the denominator grows accordingly. Tests themselves are unchanged. The Linux (LLVM) report is unaffected.

@github-project-automation github-project-automation Bot moved this from In Progress to On Hold in BAS - Baselibs FT Oct 19, 2026
@4og
4og added this pull request to the merge queue Oct 19, 2026
Merged via the queue into main with commit d1b4822 Oct 19, 2026
29 checks passed
@4og
4og deleted the dcalavrezo_qnx_host_o0 branch October 19, 2026 14:55
@github-project-automation github-project-automation Bot moved this from On Hold to Done in BAS - Baselibs FT Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants