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

[ESIMD] host device disabled for ESIMD kernels #492

Conversation

dongkyunahn-intel
Copy link

  • esimd_media_block_load/store intrinsic implementation for host
    device is changed to use ESIMD_CPU plug-in in intel/llvm PR#4020

  • The implementation is not compatible with host device mode therefore
    tests calling impacted intrinsics must be marked UNSUPPORTED for host
    device

…/store

- esimd_media_block_load/store intrinsic implementation for host
device is changed to use ESIMD_CPU plug-in in intel/llvm PR#4020

- The implementation is not compatible with host device mode therefore
tests calling impacted intrinsics must be marked UNSUPPORTED for host
device
@v-klochkov
Copy link

The only failed test - slm_gather_scatter_rgba.cpp. It has been fixed yesterday. This PR would need rebase or git-merge.

- ESIMD won't support host mode

- For CPU Emulation, ESIMD_EMU (or ESIMD_CPU) backend will be used instead of host
@dongkyunahn-intel dongkyunahn-intel changed the title [ESIMD] host device disabled for ESIMD kernels using media_block_load/store [ESIMD] host device disabled for ESIMD kernels Oct 7, 2021
@dongkyunahn-intel
Copy link
Author

Will merge after #505.

@kbobrovs
Copy link

Per our discussion, let's not disable HOST_RUN for this tests at this point and follow these steps:

  • split the compiler's 4020 into two parts - __esimd* implementation based on CM lib and the rest (PI ESIMD EMU plugin)
  • merge the latter PR (plugin part) and enable it by default
  • have vadd_1d HOST_RUN_PLACEHOLDER replaced with ESIMD_EMU_RUN_PLACEHOLDER, and it should pass with the EMU plugin
  • over time, convert HOST_RUN_PLACEHOLDER to ESIMD_EMU_RUN_PLACEHOLDER in other ESIMD tests, in accordance with intrinsics emulation implementation readiness

@dongkyunahn-intel
Copy link
Author

Closing as planning for transition to ESIMD_EMU_RUN_PLACEHOLDER is changed.

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.

3 participants