Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL] removal of sycl-ls-unified-runtime test#1491

Closed
cperkinsintel wants to merge 1 commit into
intel:intelfrom
cperkinsintel:cperkins-remove-sycl-ls-ur-test
Closed

[SYCL] removal of sycl-ls-unified-runtime test#1491
cperkinsintel wants to merge 1 commit into
intel:intelfrom
cperkinsintel:cperkins-remove-sycl-ls-ur-test

Conversation

@cperkinsintel

Copy link
Copy Markdown

this test depends upon SYCL gathering up empty platforms, which we don't do. Platforms without devices are not listed nor reported to the user. I'm not sure of an alternate way of detecting the presence of the UR. Perhaps one of its functions could output when SYCL_PI_TRACE=2 ?

See also intel/llvm#7923

…n't do. Platforms without devices are not listed nor reported to the user. I'm not sure of an alternate way of detecting the presence of the UR. Perhaps one of its functions could output when SYCL_PI_TRACE=2 ?
@smaslov-intel

Copy link
Copy Markdown

@cperkinsintel : could we keep the test disabled? I expect soon devices will be reported by UR and then it can be re-enabled

@cperkinsintel

Copy link
Copy Markdown
Author

@smaslov-intel - yes, I can do that. I'll just close this PR and open a different one.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants