Skip to content

Conversation

@Bensuo
Copy link
Contributor

@Bensuo Bensuo commented Jun 17, 2024

  • Fix incorrect conditions for copy engine usage that were reported on coverity.

@Bensuo Bensuo requested a review from a team as a code owner June 17, 2024 13:51
@github-actions github-actions bot added level-zero L0 adapter specific issues command-buffer Command Buffer feature addition/changes/specification labels Jun 17, 2024
@Bensuo Bensuo mentioned this pull request Jun 17, 2024
@Bensuo
Copy link
Contributor Author

Bensuo commented Jun 17, 2024

DPC++ PR: intel/llvm#14194

@Bensuo Bensuo force-pushed the ben/coverity-fix branch from 8225f39 to 9341627 Compare June 20, 2024 10:01
@EwanC EwanC added the ready to merge Added to PR's which are ready to merge label Jun 21, 2024
@Bensuo Bensuo force-pushed the ben/coverity-fix branch 2 times, most recently from be5f90c to 2d8125a Compare June 25, 2024 11:53
@Bensuo
Copy link
Contributor Author

Bensuo commented Jun 25, 2024

E2E CI job failure looks unrelated to this PR.

@Bensuo Bensuo force-pushed the ben/coverity-fix branch from 2d8125a to 41ec385 Compare June 26, 2024 11:54
- Fix incorrect conditions for copy engine usage that were reported on coverity.
@Bensuo Bensuo force-pushed the ben/coverity-fix branch from 41ec385 to e0bb845 Compare June 27, 2024 14:47
@hdelan hdelan mentioned this pull request Jun 28, 2024
@callumfare
Copy link
Contributor

This was included in #1804 which has now merged

@callumfare callumfare closed this Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command-buffer Command Buffer feature addition/changes/specification level-zero L0 adapter specific issues ready to merge Added to PR's which are ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants