Skip to content

[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

Merged
merged 29 commits into from
Oct 10, 2024

Conversation

KseniyaTikhomirova
Copy link
Contributor

@KseniyaTikhomirova KseniyaTikhomirova commented Sep 2, 2024

Similar checks for extensions will be covered in a separate commit.

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]>
Copy link
Contributor

@steffenlarsen steffenlarsen left a 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]>
@KseniyaTikhomirova KseniyaTikhomirova changed the title [SYCL] Add property validation to sycl::queue ctors [SYCL] Add property validation to sycl object ctors Sep 3, 2024
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]>
@KseniyaTikhomirova
Copy link
Contributor Author

@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]>
@KseniyaTikhomirova
Copy link
Contributor Author

@intel/llvm-gatekeepers hi, this PR is ready to be merged

@sergey-semenov sergey-semenov merged commit dff3d00 into intel:sycl Oct 10, 2024
12 checks passed
@sarnex
Copy link
Contributor

sarnex commented Oct 10, 2024

@KseniyaTikhomirova

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

@KseniyaTikhomirova
Copy link
Contributor Author

@KseniyaTikhomirova

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

@sarnex
Copy link
Contributor

sarnex commented Oct 10, 2024

Thank you!

sarnex pushed a commit that referenced this pull request Oct 10, 2024
fixes post commit build for #15253

Signed-off-by: Tikhomirova, Kseniya <[email protected]>
@KseniyaTikhomirova
Copy link
Contributor Author

Thank you!

this commit is a fix too: #15657. Build failure without it is observed with gcc12 only. gcc11 is ok, "algorithm" header includes "functional".

steffenlarsen pushed a commit that referenced this pull request Jan 8, 2025
…raph extension (#15647)

extra changes, follow up for #15253

---------

Signed-off-by: Tikhomirova, Kseniya <[email protected]>
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.

5 participants