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] Build sycl/tools with same CXX STD settings as set for sycl #2225

Merged
merged 1 commit into from
Jul 31, 2020

Conversation

v-klochkov
Copy link
Contributor

Having differnet CMAKE_CXX_STANDARD/CMAKE_CXX_STANDARD_REQUIRED/CMAKE_CXX_EXTENSIONS
may lead to errors during build of sycl/tools.
Removing explicit setting of those 3 parameters in sycl/tool/CMakeLists.txt
in this patch syncs the build settings of sycl/tools and sycl libraries.

Signed-off-by: Vyacheslav N Klochkov [email protected]

Having differnet CMAKE_CXX_STANDARD/CMAKE_CXX_STANDARD_REQUIRED/CMAKE_CXX_EXTENSIONS
may lead to errors during build of sycl/tools.
Removing explicit setting of those 3 parameters in sycl/tool/CMakeLists.txt
in this patch syncs the build settings of sycl/tools and sycl libraries.

Signed-off-by: Vyacheslav N Klochkov <[email protected]>
@v-klochkov v-klochkov requested a review from a team as a code owner July 30, 2020 22:12
@v-klochkov
Copy link
Contributor Author

v-klochkov commented Jul 30, 2020

Without this fix I have build errors like the one below during the sycl/tools build (with my changes being in development):
llvm/sycl/include/CL/sycl/handler.hpp:892:50: error: use of ‘auto’ in lambda parameter declaration only available with -std=c++14 or -std=gnu++14

@bader bader merged commit dd53104 into intel:sycl Jul 31, 2020
@v-klochkov v-klochkov deleted the public_tools_cpp14 branch July 31, 2020 20:23
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