Skip to content

[UR][L0] Enable Immediate Command List by default given Intel DG2 #15031

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

Closed
wants to merge 1 commit into from

Conversation

nrspruit
Copy link
Contributor

@nrspruit nrspruit commented Aug 9, 2024

-pre-commit PR for oneapi-src/unified-runtime#1951

@sarnex
Copy link
Contributor

sarnex commented Aug 9, 2024

@nrspruit sorry i just happened to see notice this PR and I know it's a still draft, but since this looks like a DG2 specific change, DG2 testing isn't run in precommit besides trivial matrix tests. if you want to do full DG2 testing we need to run postcommit as part of PR testing, which you can do by creating a branch in intel/llvm prefixed with sycl-devops-pr/ and making a PR from there.

@nrspruit
Copy link
Contributor Author

nrspruit commented Aug 13, 2024

Thanks @sarnex

devops PR here:
#15054

@sarnex
Copy link
Contributor

sarnex commented Aug 13, 2024

Thanks, looks like postcommit is running there as expected!

@omarahmed1111
Copy link
Contributor

This will be included in this PR

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.

3 participants