-
Notifications
You must be signed in to change notification settings - Fork 130
[SYCL] Update LIT to support the latest BEs #752
Conversation
- add support for ext_intel_esimd_emulator BE. Only SYCL/ESIMD test should be run for this BE; - switch to use actual BE names to avoid runtime warning; - updated documentation accordingly.
@dongkyunahn-intel, @kbobrovs the testing on with compiler built from intel/llvm#5058 provides the following pass rate for the emulator:
is that expected? |
For ESIMD emulator backend only tests from SYCL/ESIMD are run.
The failure is unrelated to the patch:
|
I locally tested esimd_emulator backend with this PR and observed failures from ESIMD kernels - e.g. memory intrinsic from esimd_emulator, time-out, etc while 19 tests passed. I'm going to use my previous PR (#751) for marking such kernels accordingly - either |
right, we want to start running regularly those which can run today, address the rest afterwards. |
548bb1b
to
2b6944a
Compare
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, with one comment
- add support for ext_intel_esimd_emulator BE. Only SYCL/ESIMD test should be run for this BE; - switch to use actual BE names to avoid runtime warning; - updated documentation accordingly.
- add support for ext_intel_esimd_emulator BE. Only SYCL/ESIMD test should be run for this BE; - switch to use actual BE names to avoid runtime warning; - updated documentation accordingly.
- add support for ext_intel_esimd_emulator BE. Only SYCL/ESIMD test should be run for this BE; - switch to use actual BE names to avoid runtime warning; - updated documentation accordingly.
should be run for this BE;