Skip to content

[SYCL][ABI-break] Switch to struct information descriptors #6467

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

Merged
merged 6 commits into from
Aug 5, 2022

Conversation

sergey-semenov
Copy link
Contributor

This patch changes information descriptors from enum class elements to
structs in accordance with SYCL 2020 spec. Additionally, it removes some
deprecated information queries that were dropped in either SYCL 2020 or
the corresponding extension and deprecates those listed as such in SYCL
2020 spec.

This patch changes information descriptors from enum class elements to
structs in accordance with SYCL 2020 spec. Additionally, it removes some
deprecated information queries that were dropped in either SYCL 2020 or
the corresponding extension and deprecates those listed as such in SYCL
2020 spec.
@sergey-semenov sergey-semenov requested a review from a team as a code owner July 22, 2022 14:54
@sergey-semenov sergey-semenov requested a review from againull July 22, 2022 14:54
@sergey-semenov
Copy link
Contributor Author

Need to update windows ABI test as well, but I'm running into some issues on my Windows machine. Will follow up.

@sergey-semenov
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1107

@sergey-semenov
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1107

@sergey-semenov
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1107

@sergey-semenov
Copy link
Contributor Author

sergey-semenov commented Jul 27, 2022

@againull Could you please review this one and the llvm-test-suite change?

@sergey-semenov
Copy link
Contributor Author

@againull @intel/llvm-reviewers-runtime ping

@steffenlarsen
Copy link
Contributor

Since this breaks ABI, this PR should increment SYCL_DEV_ABI_VERSION.

@sergey-semenov
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1107

@sergey-semenov
Copy link
Contributor Author

/verify with intel/llvm-test-suite#1107

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