Skip to content

Conversation

@smanna12
Copy link
Contributor

@smanna12 smanna12 commented Jun 8, 2022

This patch updates tests for [[intel::reqd_sub_group_size()]] attribute:

  1. separates AST and diagnostics to make it easier to read/follow-up.
  2. removes duplicate test cases.
  3. adds missing tests.
  4. no compiler changes.

Signed-off-by: Soumi Manna [email protected]

smanna12 added 2 commits June 8, 2022 07:18
…intel#6243)

This patch updates tests for [[intel::reqd_sub_group_size()]] attribute:

* separates AST and diagnostics to make it easier to read/follow-up.
* removes duplicate test cases.
* adds missing tests.
* no compiler changes.

Signed-off-by: Soumi Manna [email protected]
Signed-off-by: Soumi Manna <[email protected]>
@smanna12 smanna12 changed the title Refactor tests [SYCL][NFC] Refactor [[intel::reqd_sub_group_size()]] attribute tests Jun 8, 2022
@smanna12
Copy link
Contributor Author

smanna12 commented Jun 8, 2022

This patch reapplies #6243 after fixing post-commit failures due to check-clang failure in #6220 (comment)?

@smanna12 smanna12 marked this pull request as ready for review June 8, 2022 15:04
@smanna12 smanna12 requested a review from a team as a code owner June 8, 2022 15:04
@smanna12
Copy link
Contributor Author

smanna12 commented Jun 9, 2022

Thanks for reviews @premanandrao.

@intel/llvm-gatekeepers, this PR is ready to be merged. Thank you

@pvchupin pvchupin merged commit fe2b28e into intel:sycl Jun 9, 2022
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.

3 participants