Skip to content

Conversation

@romanovvlad
Copy link
Contributor

It seems that without -fsycl the default version of c++ is less than c++17 which is required for SYCL headers.

It seems that without -fsycl the default version of c++ is less
than c++17 which is required for SYCL headers.
@romanovvlad romanovvlad requested a review from a team as a code owner October 13, 2022 16:43
@pvchupin
Copy link
Contributor

That's very strange because community switched to C++17 by default and we should have already this change pulled down from upstream. We need to follow up on the current default.
@mdtoguchi, can you comment please?

@pvchupin pvchupin merged commit 6de561e into intel:sycl Oct 13, 2022
@pvchupin
Copy link
Contributor

That's very strange because community switched to C++17 by default and we should have already this change pulled down from upstream. We need to follow up on the current default. @mdtoguchi, can you comment please?

On the other hand... probably that was for Linux only?
llvm/llvm-project@e321c8d

@mdtoguchi
Copy link
Contributor

Looks like linux based targets only:

sycl-run:1197> clang++ -target x86_64-unknown-windows-msvc -xc++ -E -dM /dev/null | grep plusplus
#define __cplusplus 201402L
sycl-run:1198> clang++ -target x86_64-unknown-linux-gnu -xc++ -E -dM /dev/null | grep plusplus
#define __cplusplus 201703L

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.

5 participants