Skip to content

[SYCL][NFC] Move khr_free_function_commands extension to common khr directory#18556

Merged
sarnex merged 1 commit intointel:syclfrom
dyniols:move_khr_free_function_commands_impl_to_common_dir
May 20, 2025
Merged

[SYCL][NFC] Move khr_free_function_commands extension to common khr directory#18556
sarnex merged 1 commit intointel:syclfrom
dyniols:move_khr_free_function_commands_impl_to_common_dir

Conversation

@dyniols
Copy link
Copy Markdown
Contributor

@dyniols dyniols commented May 20, 2025

Purpose of this PR is to move khr_free_function_commands extension to common khr directory based on discussion #17595 (comment)
Extension spec: KhronosGroup/SYCL-Docs#644

@dyniols dyniols temporarily deployed to WindowsCILock May 20, 2025 08:28 — with GitHub Actions Inactive
@dyniols dyniols requested a review from 0x12CC May 20, 2025 08:30
@dyniols dyniols marked this pull request as ready for review May 20, 2025 08:32
@dyniols dyniols requested a review from a team as a code owner May 20, 2025 08:32
@dyniols dyniols requested a review from uditagarwal97 May 20, 2025 08:32
@dyniols dyniols temporarily deployed to WindowsCILock May 20, 2025 08:55 — with GitHub Actions Inactive
@dyniols dyniols temporarily deployed to WindowsCILock May 20, 2025 08:55 — with GitHub Actions Inactive
Copy link
Copy Markdown
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. Can we also add a [NFC] tag in the title of the PR, since it's a non-functional change?

@dyniols
Copy link
Copy Markdown
Contributor Author

dyniols commented May 20, 2025

LGTM. Can we also add a [NFC] tag in the title of the PR, since it's a non-functional change?

Sure

@dyniols dyniols changed the title [SYCL] Move khr_free_function_commands extension to common khr directory [SYCL][NFC] Move khr_free_function_commands extension to common khr directory May 20, 2025
@dyniols
Copy link
Copy Markdown
Contributor Author

dyniols commented May 20, 2025

@intel/llvm-gatekeepers Can we merge this PR?

@sarnex sarnex merged commit ccb079c into intel:sycl May 20, 2025
21 of 24 checks passed
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