Skip to content

Commit 3c1237f

Browse files
authored
[SYCL] Simplify check whether a header file is available for inclusion (#19759)
`__has_include` is part of the C++ standard since C++17: https://en.cppreference.com/w/cpp/preprocessor/include We require C++17 if `sycl.hpp` is included, so we can use `__has_include` without extra checks for its presence.
1 parent 93ab143 commit 3c1237f

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

sycl/include/sycl/khr/group_interface.hpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,9 @@
1313

1414
#define SYCL_KHR_GROUP_INTERFACE 1
1515

16-
#if __cplusplus >= 202302L && defined(__has_include)
17-
#if __has_include(<mdspan>)
16+
#if __cplusplus >= 202302L && __has_include(<mdspan>)
1817
#include <mdspan>
1918
#endif
20-
#endif
2119

2220
namespace sycl {
2321
inline namespace _V1 {

sycl/source/detail/os_util.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,10 @@
1313
#include <limits>
1414

1515
// For GCC versions less than 8, use experimental/filesystem.
16-
#if defined(__has_include) && __has_include(<filesystem>)
16+
#if __has_include(<filesystem>)
1717
#include <filesystem>
1818
namespace fs = std::filesystem;
19-
#elif defined(__has_include) && __has_include(<experimental/filesystem>)
19+
#elif __has_include(<experimental/filesystem>)
2020
#include <experimental/filesystem>
2121
namespace fs = std::experimental::filesystem;
2222
#else

0 commit comments

Comments
 (0)