Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SYCL][CI] Reduce scope of Coverity scan #17355

Open
wants to merge 1 commit into
base: sycl
Choose a base branch
from

Conversation

AlexeySachkov
Copy link
Contributor

We are not interested in project/binaries/libraries which are not a part of the sycl-toolchain target - they are already being scanned by the upstream LLVM.

@AlexeySachkov
Copy link
Contributor Author

AlexeySachkov commented Mar 7, 2025

Workflow change is being tested here: https://github.com/intel/llvm/actions/runs/13760707906

I see ~5.5GB build reduction comparing to the latest regular run: https://github.com/intel/llvm/actions/runs/13742894384

We are not interested in project/binaries/libraries which are not a part
of the `sycl-toolchain` target - they are already being scanned by the
upstream LLVM.
@AlexeySachkov AlexeySachkov force-pushed the private/asachkov/reduce-coverity-scope branch from c57f972 to d366c7f Compare March 10, 2025 09:02
@AlexeySachkov AlexeySachkov marked this pull request as ready for review March 10, 2025 14:17
@AlexeySachkov AlexeySachkov requested a review from a team as a code owner March 10, 2025 14:17
@AlexeySachkov
Copy link
Contributor Author

CI results are irrelevant for this PR and the whole pre-commit was launched due to misconfiguration of our workflow files: changes to coverity scan workflow cannot affect our functional validation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants