Skip to content

Conversation

@kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Jul 19, 2024

Split out from #14145 this patch replaces the generated features.hpp header for the CUDA and HIP plugins, which only defined single feature test macro, with #cmakedefine01 definitions controlled by the build system.

@kbenzie kbenzie requested a review from a team as a code owner July 19, 2024 16:16
@kbenzie kbenzie requested a review from uditagarwal97 July 19, 2024 16:16
@kbenzie kbenzie changed the title [SYCL][UR] Replace CUDA & HIP features.hpp headers with macros [SYCL] Replace CUDA & HIP features.hpp headers with macros Jul 19, 2024
@kbenzie kbenzie temporarily deployed to WindowsCILock July 19, 2024 16:18 — with GitHub Actions Inactive
@kbenzie kbenzie requested a review from AlexeySachkov July 19, 2024 16:45
@kbenzie kbenzie temporarily deployed to WindowsCILock July 19, 2024 17:21 — with GitHub Actions Inactive
@kbenzie kbenzie force-pushed the benie/pi2ur-feature-tests branch from 240d034 to 06e1bdf Compare July 23, 2024 12:21
Split out from intel#14145 this patch
replaces the generated `features.hpp` header for the CUDA and HIP
plugins, which only defined single feature test macro, with
`#cmakedefine01` definitions controlled by the build system.
@kbenzie kbenzie force-pushed the benie/pi2ur-feature-tests branch from 06e1bdf to 81286c9 Compare July 23, 2024 12:22
@kbenzie kbenzie temporarily deployed to WindowsCILock July 23, 2024 12:23 — with GitHub Actions Inactive
@kbenzie kbenzie temporarily deployed to WindowsCILock July 23, 2024 13:04 — with GitHub Actions Inactive
@kbenzie
Copy link
Contributor Author

kbenzie commented Jul 23, 2024

@uditagarwal97 anything needing changed or is this in a good state for apporoval?

Copy link
Contributor

@uditagarwal97 uditagarwal97 left a comment

Choose a reason for hiding this comment

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

LGTM

@AlexeySachkov AlexeySachkov merged commit 5bf5442 into intel:sycl Jul 24, 2024
endif()
if ("level_zero" IN_LIST SYCL_ENABLE_PLUGINS)
set(SYCL_BUILD_PI_LEVEL_ZERO ON)
set(SYCL_BUILD_BACKENDLEVEL_ZERO ON)
Copy link
Contributor

Choose a reason for hiding this comment

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

SYCL_BUILD_BACKEND_LEVEL_ZERO

@kbenzie kbenzie deleted the benie/pi2ur-feature-tests branch December 18, 2024 13:28
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