Skip to content

[SYCL][SCLA] Check allocated types are trivial #13105

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 2 commits into from
Mar 29, 2024

Conversation

victor-eds
Copy link
Contributor

@victor-eds victor-eds commented Mar 22, 2024

Replace check for cv-unqualified object types with a check for cv-unqualified trivial types to be in line with the sycl_ext_oneapi_private_alloca extension specification:

ElementType must be a cv-unqualified trivial type

@victor-eds victor-eds self-assigned this Mar 22, 2024
@victor-eds victor-eds requested a review from a team as a code owner March 22, 2024 10:01
@victor-eds victor-eds requested a review from premanandrao March 25, 2024 15:41
Replace check for cv-unqualified object types with a check for
cv-unqualified trivial type to be in line with the extension
specification:

> `ElementType` must be a cv-unqualified trivial type

Signed-off-by: Victor Perez <[email protected]>
@victor-eds
Copy link
Contributor Author

@intel/dpcpp-cfe-reviewers can I get a review for this? Test failures unrelated to this change.

Copy link
Contributor

@premanandrao premanandrao left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@smanna12 smanna12 left a comment

Choose a reason for hiding this comment

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

LGTM.

@victor-eds , what is SCLA?

@victor-eds
Copy link
Contributor Author

CI failures unrelated to these changes.

@victor-eds
Copy link
Contributor Author

@intel/llvm-gatekeepers can we get this merged?

@victor-eds
Copy link
Contributor Author

LGTM.

@victor-eds , what is SCLA?

"Specialization Constant-Length Alloca"

@sommerlukas sommerlukas merged commit ac4808a into intel:sycl Mar 29, 2024
@victor-eds victor-eds deleted the alloca-trivial branch March 29, 2024 09:24
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