Skip to content

[SYCL] Rework of SYCL poperties #2196

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

Conversation

romanovvlad
Copy link
Contributor

Changed property_list class to support properties of unknown
type. This is achieved by splitting properties to two types:
simple and complex. Simple are those which do not have any
date - just represent that something is enabled or disabled.
Compilex properties have some data.

Simple properties have compile time known id, so they are
always stored in the property_list even if they are not set.
Complex are allocated dynamically and accessed through the
common base class.

Changed property_list class to support properties of unknown
type. This is achieved by splitting properties to two types:
simple and complex. Simple are those which do not have any
date - just represent that something is enabled or disabled.
Compilex properties have some data.

Simple properties have compile time known id, so they are
always stored in the property_list even if they are not set.
Complex are allocated dynamically and accessed through the
common base class.
@romanovvlad romanovvlad requested a review from a team as a code owner July 28, 2020 18:52
@romanovvlad romanovvlad requested a review from v-klochkov July 28, 2020 18:52
@romanovvlad romanovvlad merged commit ca2c5bb into intel:sycl Jul 31, 2020
jsji pushed a commit that referenced this pull request Nov 9, 2023
This commit adds StridesizeINTEL, WordsizeINTEL and TrueDualPortINTEL
support to translator.

Original commit:
KhronosGroup/SPIRV-LLVM-Translator@5c1e2c7
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