Skip to content

Conversation

@bso-intel
Copy link
Contributor

No description provided.

Signed-off-by: Byoungro So <byoungro.so@intel.com>
Signed-off-by: Byoungro So <byoungro.so@intel.com>
Signed-off-by: Byoungro So <byoungro.so@intel.com>
@bso-intel bso-intel changed the title test code for discussion [DO NOT MERGE][SYCL][ABI-Break] ABI-neutralize has-kernel Apr 18, 2024
Signed-off-by: Byoungro So <byoungro.so@intel.com>
Signed-off-by: Byoungro So <byoungro.so@intel.com>
Signed-off-by: Byoungro So <byoungro.so@intel.com>
Signed-off-by: Byoungro So <byoungro.so@intel.com>
Copy link
Contributor

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

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

Overall it looks good, but we should not export the symbols that are inline in the headers. Please address these, then I'm alright with merging this.

Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
bso-intel and others added 2 commits July 18, 2024 09:30
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
@bso-intel
Copy link
Contributor Author

@intel/llvm-gatekeepers
Please merge.
Today is the last day of ABI-breaking window.
Thanks.

@bso-intel bso-intel changed the title [ABI-Break] ABI-neutralize has-kernel [SYCL][ABI-Break] ABI-neutralize has-kernel Jul 19, 2024
@againull againull closed this Jul 19, 2024
@againull againull reopened this Jul 19, 2024
@steffenlarsen steffenlarsen merged commit 9fdef43 into intel:sycl Jul 19, 2024
@bso-intel bso-intel deleted the has-kernel branch July 19, 2024 06:41
@bso-intel bso-intel restored the has-kernel branch August 5, 2024 22:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

abi-break change that's breaking abi and waiting for the next window to be able to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants