Skip to content

[SYCL][NFC] Update some ESIMD tests - #22944

Open
KornevNikita wants to merge 1 commit into
intel:syclfrom
KornevNikita:upd-esimd-e2e
Open

[SYCL][NFC] Update some ESIMD tests#22944
KornevNikita wants to merge 1 commit into
intel:syclfrom
KornevNikita:upd-esimd-e2e

Conversation

@KornevNikita

Copy link
Copy Markdown
Contributor

These tests are disabled on PVC because there are seperate versions of these tests for PVC.

These tests are disabled on PVC because there are seperate versions of these
tests for PVC.
@KornevNikita
KornevNikita marked this pull request as ready for review August 14, 2026 16:53
@KornevNikita
KornevNikita requested review from a team as code owners August 14, 2026 16:53

@sarnex sarnex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks!

//===----------------------------------------------------------------------===//
//
// UNSUPPORTED: arch-intel_gpu_pvc
// REQUIRES: !arch-intel_gpu_pvc

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: can we put the new comment above the REQUIRES? its not worth manually updating, see if you can get an LLM to do it. if not, just ignore this comment

//
// UNSUPPORTED: arch-intel_gpu_pvc
// REQUIRES: !arch-intel_gpu_pvc
// There is a separate version of this test for PVC.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The guide says you should use UNSUPPORTED + UNSUPPORTED-INTENDED for such cases.

Suggested change
// There is a separate version of this test for PVC.
// UNSUPPORTED-INTENDED: There is a separate version of this test for PVC.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you link the guide? I didn't know we had a rule for this

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thanks for the link, i will make sure to honor this in the future

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