-
Notifications
You must be signed in to change notification settings - Fork 790
[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
Conversation
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]>
6f87e34
to
091f432
Compare
@intel/dpcpp-cfe-reviewers can I get a review for this? Test failures unrelated to this change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this 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?
CI failures unrelated to these changes. |
@intel/llvm-gatekeepers can we get this merged? |
"Specialization Constant-Length Alloca" |
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: