-
Notifications
You must be signed in to change notification settings - Fork 131
[SYCL] Check that fp16 aspect is supported before using half #1487
Conversation
passed &= test<sycl::half>(); | ||
passed &= test<float>(queue); | ||
if (queue.get_device().has(sycl::aspect::fp16)) | ||
passed &= test<sycl::half>(queue); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the test compilation options need -fsycl-device-code-split=per_kernel to handle the aspects properly at runtime.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't know that, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
per_kernel
device code split is not actually needed anymore since intel/llvm#7302: the compiler is now able to automatically split kernels into separate modules if optional features are used
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
SYCL :: SubGroupMask/Basic.cpp is known and tracked. |
This is needed to fix CI failure caused by new test, example: intel/llvm#7919