Skip to content
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

[SYCL][NFC] Remove deprecated variables from LIT #2794

Merged
merged 4 commits into from
Nov 24, 2020

Conversation

vladimirlaz
Copy link
Contributor

@vladimirlaz vladimirlaz commented Nov 19, 2020

Remove use of deprecated environment variables (SYCL_BE,
SYCL_DEVICE_TYPE) from LIT framework. Use SYCL_DEVICE_FILTER instead:

  • rename CMake parameter SYCL_BE to SYCL_PLUGIN to avoid mix-up with
    deprecated environment variable;
  • update [CPU|GPU|ACC]_RUN_PLACEHOLDER to use SYCL_DEVICE_FILTER;
  • change values format used to define a backend and a device type to
    match SYCL_DEVICE_FILTER requirements;
  • add BE_RUN_PLACEHOLDER to set backend only for tests which require
    that;
  • add RUN_ON_HOST substitution to clearly state target device in RUN
    command.

Remove use of deprecated environment variables (SYCL_BE,
SYCL_DEVICE_TYPE) from LIT framework. Use SYCL_DEVICE_FILTER instead:

 - update [CPU|GPU|ACC]_RUN_PLACEHOLDER to use SYCL_DEVICE_FILTER;
 - change values format used to define a backend and a device type to
match SYCL_DEVICE_FILTER requirements;
 - add BE_RUN_PLACEHOLDER to set backend only for tests which require
that;
 - add HOST_RUN_PLACEHOLDER to align configuration with other devices;
 - on-device/basic_tests/diagnostics/device-check.cpp is not changed
because it checks SYCL_DEVICE_FILTER itself and will be removed once
the variable is removed.
@vladimirlaz vladimirlaz requested a review from a team as a code owner November 19, 2020 05:42
sycl/test/CMakeLists.txt Outdated Show resolved Hide resolved
sycl/test/lit.cfg.py Outdated Show resolved Hide resolved
sycl/test/basic_tests/esimd/vadd.cpp Outdated Show resolved Hide resolved
sycl/test/on-device/config/select_device.cpp Show resolved Hide resolved
@bader
Copy link
Contributor

bader commented Nov 19, 2020

LGTM. Thanks.

@vladimirlaz
Copy link
Contributor Author

@rbegam, could you please have a look?

Copy link
Contributor

@rbegam rbegam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@vladimirlaz vladimirlaz merged commit d0b4172 into intel:sycl Nov 24, 2020
@vladimirlaz vladimirlaz deleted the remove_deprecated_env branch November 24, 2020 05:11
jsji pushed a commit that referenced this pull request Nov 7, 2024
#1174 implements translating integer dot product OCL builtins to SPIR-V builtins. This pull request is to do the reverse translation.

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@925255cb1982896
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants