-
Notifications
You must be signed in to change notification settings - Fork 788
[SYCL] Add property validation to sycl object ctors #15253
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
[SYCL] Add property validation to sycl object ctors #15253
Conversation
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
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.
Small missing template terminators, but otherwise LGTM!
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
@steffenlarsen may I ask you for one more round of review? I added checks to images and accessors. thank you. |
Signed-off-by: Tikhomirova, Kseniya <[email protected]>
@intel/llvm-gatekeepers hi, this PR is ready to be merged |
Postcommit build is failing, seems to be due to this PR. Can you please fix ASAP? Thanks https://github.com/intel/llvm/actions/runs/11275009165/job/31355446133 |
#15656 to fix that |
Thank you! |
fixes post commit build for #15253 Signed-off-by: Tikhomirova, Kseniya <[email protected]>
this commit is a fix too: #15657. Build failure without it is observed with gcc12 only. gcc11 is ok, "algorithm" header includes "functional". |
…raph extension (#15647) extra changes, follow up for #15253 --------- Signed-off-by: Tikhomirova, Kseniya <[email protected]>
Similar checks for extensions will be covered in a separate commit.