Skip to content

[SYCL][Cmake] Add SYCL_LIBDEVICE_GCC_TOOLCHAIN variable #7771

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

Merged
merged 1 commit into from
Jan 4, 2023

Conversation

aelovikov-intel
Copy link
Contributor

To be used when we want libdevice built against non-system STL (e.g. if we're using non-system GCC) as clang doesn't use gcc in PATH to determine which STL to use.

@abagusetty
Copy link
Contributor

Would passing the right, non-standard absolute paths to CC= and CXX= to both configure and compile steps alleviate this issue with gcc-toolchain for libdevice instead ?

@aelovikov-intel
Copy link
Contributor Author

Would passing the right, non-standard absolute paths to CC= and CXX= to both configure and compile steps alleviate this issue with gcc-toolchain for libdevice instead ?

No, for several reasons:

  1. We might want to compile with non-GCC compilers yet still use GCC's libstdc++ implementation as C++ standard library, so that would be a wrong variable to use.
  2. We need to pass the information to a freshly built clang and it doesn't care about CC/CXX.

To be used when we want libdevice built against non-system STL (e.g. if we're
using non-system GCC) as clang doesn't use gcc in PATH to determine which STL
to use.
@aelovikov-intel aelovikov-intel force-pushed the libdevice_gcc_toolchain branch from 7f77afd to 62b8e72 Compare January 3, 2023 20:11
@aelovikov-intel aelovikov-intel requested a review from bader January 3, 2023 20:12
@aelovikov-intel aelovikov-intel temporarily deployed to aws January 3, 2023 20:36 — with GitHub Actions Inactive
@aelovikov-intel aelovikov-intel temporarily deployed to aws January 3, 2023 21:08 — with GitHub Actions Inactive
@aelovikov-intel
Copy link
Contributor Author

@intel/llvm-gatekeepers , this PR is ready.

@againull againull merged commit 7eccec3 into intel:sycl Jan 4, 2023
@aelovikov-intel aelovikov-intel deleted the libdevice_gcc_toolchain branch January 4, 2023 00:40
aelovikov-intel added a commit to aelovikov-intel/llvm that referenced this pull request Apr 21, 2023
…its.hpp

Re-commits intel#7628 and intel#7668 that were previously reverted due to build
issues on RHEL systems. The issues were fixed by introducing
`SYCL_LIBDEVICE_GCC_TOOLCHAIN` cmake configuration option in intel#7771.
againull pushed a commit that referenced this pull request Apr 24, 2023
…its.hpp (#9162)

Re-commits #7628 and #7668 that were previously reverted due to build
issues on RHEL systems. The issues were fixed by introducing
`SYCL_LIBDEVICE_GCC_TOOLCHAIN` cmake configuration option in #7771.
aelovikov-intel added a commit to aelovikov-intel/llvm that referenced this pull request Aug 11, 2023
…tests

The same has been done in libdevice in intel#7771. We need this internally
where we have a build on an older RHEL with gcc in non-system location.
aelovikov-intel added a commit that referenced this pull request Aug 14, 2023
…tests (#10791)

The same has been done in libdevice in #7771. We need this internally
where we have a build on an older RHEL with gcc in non-system location.
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.

4 participants