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

[L0] coverity fixes #1179

Merged
merged 1 commit into from
Dec 13, 2023
Merged

[L0] coverity fixes #1179

merged 1 commit into from
Dec 13, 2023

Conversation

pbalcer
Copy link
Contributor

@pbalcer pbalcer commented Dec 12, 2023

No description provided.

@pbalcer pbalcer added the v0.8.x Include in the v0.8.x release label Dec 12, 2023
@pbalcer pbalcer requested a review from a team as a code owner December 12, 2023 14:40
@pbalcer
Copy link
Contributor Author

pbalcer commented Dec 12, 2023

This fixes all but one coverity issues for the L0 adapter. I think that remaining one is a false positive, but I'd like someone to double check.

@pbalcer
Copy link
Contributor Author

pbalcer commented Dec 12, 2023

intel/llvm#12152

@kbenzie kbenzie mentioned this pull request Dec 12, 2023
13 tasks
@kbenzie kbenzie merged commit f8fc936 into oneapi-src:main Dec 13, 2023
51 checks passed
kbenzie added a commit to kbenzie/unified-runtime that referenced this pull request Dec 13, 2023
sarnex pushed a commit to intel/llvm that referenced this pull request Dec 13, 2023
Corresponding UR changes
oneapi-src/unified-runtime#1179

---------

Co-authored-by: Kenneth Benzie (Benie) <k.benzie@codeplay.com>
kbenzie added a commit to kbenzie/unified-runtime that referenced this pull request Dec 15, 2023
callumfare pushed a commit to kbenzie/llvm that referenced this pull request Dec 18, 2023
Corresponding UR changes
oneapi-src/unified-runtime#1179

---------

Co-authored-by: Kenneth Benzie (Benie) <k.benzie@codeplay.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v0.8.x Include in the v0.8.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants