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.

Sign up for GitHub

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

Jump to bottom

Fix a number of issues from the latest coverity scan. #2352

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

Fix a number of issues from the latest coverity scan. #2352

wants to merge 1 commit into from

Conversation

aarongreig
Copy link
Contributor

No description provided.

@github-actions github-actions bot added loader Loader related feature/bug conformance Conformance test suite issues. cuda CUDA adapter specific issues native-cpu Native CPU adapter specific issues sanitizer Sanitizer layer issues/changes/specification labels Nov 19, 2024
@aarongreig aarongreig force-pushed the aaron/fixCoverity15-11 branch 4 times, most recently from 3fb40a9 to e3261ff Compare November 20, 2024 15:11
@github-actions github-actions bot added level-zero L0 adapter specific issues hip HIP adapter specific issues labels Nov 20, 2024
@aarongreig aarongreig marked this pull request as ready for review November 21, 2024 13:39
@aarongreig aarongreig requested review from a team as code owners November 21, 2024 13:39
PietroGhg
PietroGhg approved these changes Nov 21, 2024
View reviewed changes
Copy link
Contributor

@PietroGhg PietroGhg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Native CPU lgtm, thank you

frasercrmck
frasercrmck reviewed Nov 21, 2024
View reviewed changes
Copy link
Contributor

@frasercrmck frasercrmck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CUDA/HIP LGTM other than a nit about that comment

@github-actions github-actions bot added the opencl OpenCL adapter specific issues label Nov 22, 2024
@aarongreig
Copy link
Contributor Author

LLVM PR intel/llvm#16158

@aarongreig
Copy link
Contributor Author

ping @oneapi-src/unified-runtime-level-zero-write

@aarongreig aarongreig added the ready to merge Added to PR's which are ready to merge label Nov 29, 2024
@kbenzie kbenzie added the v0.11.x Include in the v0.11.x release label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pbalcer pbalcer pbalcer left review comments

@kbenzie kbenzie kbenzie approved these changes

@frasercrmck frasercrmck frasercrmck left review comments

@PietroGhg PietroGhg PietroGhg approved these changes

@nrspruit nrspruit nrspruit approved these changes

@omarahmed1111 omarahmed1111 Awaiting requested review from omarahmed1111 omarahmed1111 is a code owner automatically assigned from oneapi-src/unified-runtime-hip-write

Assignees
No one assigned
Labels
conformance Conformance test suite issues. cuda CUDA adapter specific issues hip HIP adapter specific issues level-zero L0 adapter specific issues loader Loader related feature/bug native-cpu Native CPU adapter specific issues opencl OpenCL adapter specific issues ready to merge Added to PR's which are ready to merge sanitizer Sanitizer layer issues/changes/specification v0.11.x Include in the v0.11.x release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@aarongreig @frasercrmck @kbenzie @nrspruit @pbalcer @PietroGhg