-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL] Don't return empty platforms #7923
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
Conversation
Does spec mandate (or even allow) that platforms with no devices be filtered? |
/verify with intel/llvm-test-suite#1491 |
Probably we should report that empty platform doesn't have any device aspects, because it doesn't have any devices. |
With this change empty platforms are not reported, so there is no way to query aspects from such. |
Matching update to test suite here: intel/llvm-test-suite#1493 |
Please, test them together. |
/verify with intel/llvm-test-suite#1493 |
collecting empty platforms leads to confusion for things like
platform.has(aspect)