Repository navigation
[CI] Add nightly ELD code coverage workflow with regression detection - #1405
Open
Saharsh Burra (bsaharsh) wants to merge 1 commit into
Open
Saharsh Burra (bsaharsh) wants to merge 1 commit into
Saharsh Burra (bsaharsh) wants to merge 1 commit into
Conversation
Saharsh Burra (bsaharsh)
force-pushed
the
add-coverage-regression-detection
branch
from
July 7, 2026 04:10
7ab5f2f to
5cb95b0
Compare
Shankar Easwaran (quic-seaswara)
requested changes
Jul 7, 2026
| git config user.name 'github-actions' | ||
| git config user.email 'github-actions@github.com' | ||
|
|
||
| git checkout --orphan coverage-update |
Contributor
There was a problem hiding this comment.
can we have this branch as code-coverage-reports ?
Contributor
Author
There was a problem hiding this comment.
Updated the name
Parth (parth-07)
requested review from
Steven Ramirez Rosa (Steven6798),
Parth (parth-07) and
quic-areg
July 7, 2026 18:14
| jobs: | ||
| coverage: | ||
| if: github.repository == 'qualcomm/eld' | ||
| runs-on: self-hosted |
Contributor
There was a problem hiding this comment.
Shankar Easwaran (@quic-seaswara) is it okay to run this workflow on self-hosted? self-hosted here will make adding this workflow easier because we would not need to install/setup the cross-toolchains and qemu.
| repository: ${{ github.repository }} | ||
| ref: ${{ github.ref_name }} | ||
| path: llvm-project/llvm/tools/eld | ||
| fetch-depth: 0 |
Contributor
There was a problem hiding this comment.
We can likely remove fetch-depth: 0 from here.
Contributor
Author
There was a problem hiding this comment.
Updated
| ../llvm-project/llvm \ | ||
| -DCMAKE_BUILD_TYPE=Debug \ | ||
| -DCMAKE_DISABLE_PRECOMPILE_HEADERS=ON \ | ||
| -DLLVM_ENABLE_PROJECTS="llvm;clang;clang-tools-extra" \ |
Contributor
There was a problem hiding this comment.
Would it be possible to use the clang and the other tools from the nightly eld toolchain and only build eld with code-coverage enabled?
Contributor
Author
There was a problem hiding this comment.
Updated, but need to test it as fork won't be having the self hosted, may be we can use the workflow dispatch.
Saharsh Burra (bsaharsh)
force-pushed
the
add-coverage-regression-detection
branch
from
July 8, 2026 03:50
5cb95b0 to
0149db3
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Saharsh Burra (bsaharsh)
force-pushed
the
add-coverage-regression-detection
branch
2 times, most recently
from
July 10, 2026 06:41
531af9e to
60cec40
Compare
Parth (parth-07)
force-pushed
the
add-coverage-regression-detection
branch
3 times, most recently
from
July 14, 2026 15:07
aa5537f to
eb451ab
Compare
Saharsh Burra (bsaharsh)
force-pushed
the
add-coverage-regression-detection
branch
from
July 14, 2026 16:33
eb451ab to
10b8ddc
Compare
Parth (parth-07)
force-pushed
the
add-coverage-regression-detection
branch
from
July 15, 2026 11:01
49d7a22 to
4cd34c6
Compare
Saharsh Burra (bsaharsh)
force-pushed
the
add-coverage-regression-detection
branch
from
July 15, 2026 11:32
4cd34c6 to
8fcbdc7
Compare
Parth (parth-07)
force-pushed
the
add-coverage-regression-detection
branch
from
July 15, 2026 15:33
85f166b to
445434e
Compare
Saharsh Burra (bsaharsh)
force-pushed
the
add-coverage-regression-detection
branch
2 times, most recently
from
July 16, 2026 05:33
f7bb343 to
7e09cbc
Compare
Parth (parth-07)
force-pushed
the
add-coverage-regression-detection
branch
from
July 17, 2026 09:26
7e09cbc to
15b56b4
Compare
Parth (parth-07)
approved these changes
Jul 17, 2026
Parth (parth-07)
self-requested a review
July 17, 2026 09:32
Add nightly-eld-code-coverage.yml which runs daily after the nightly build. The workflow: - Builds LLVM+ELD with source-based coverage instrumentation (ELD_COVERAGE=ON) - Runs check-eld under LLVM_PROFILE_FILE to collect per-process .profraw files - Merges profiles and generates an HTML report with llvm-profdata / llvm-cov - Compares Line/Function/Region/Branch coverage against a baseline stored on the code-coverage-reports branch and fails if any metric regresses - Records the baseline on the first run when none exists - Publishes the HTML report and updated baseline to the code-coverage-reports branch Also updates CMakeLists.txt: ELD_COVERAGE now sets the llvm-cov source-based instrumentation flags (-fprofile-instr-generate -fcoverage-mapping). Signed-off-by: bsaharsh <bsaharsh@qti.qualcomm.com>
Saharsh Burra (bsaharsh)
force-pushed
the
add-coverage-regression-detection
branch
from
July 22, 2026 11:23
15b56b4 to
96e0a2c
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added a workflow which runs nightly and computes the code coverage, and when ran at first creates a baseline text file on orphan branch called code-coverage, and from the next time, it checks with the previous coverage numbers and if any metric goes down, it fails, and if no regression is detected then we push the html report and update the baseline text file.