Skip to content

Changed subgroup size from 8 to 16 in Publications/GPU-Opt-Guide/exec-model/vaddsync.cpp #2121

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

Closed
wants to merge 4 commits into from

Conversation

zma2
Copy link
Contributor

@zma2 zma2 commented Dec 19, 2023

Existing Sample Changes

Publications/GPU-Opt-Guide/exec-model/vaddsync.cpp

Description

Changed subgroup size from 8 to 16

Fixes Issue#

$ ./exec-model/vaddsync
Running on device: Intel(R) Data Center GPU Max 1100
Vector size: 27525120
terminate called after throwing an instance of 'sycl::_V1::exception'
what(): Sub-group size 8 is not supported on the device
Aborted (core dumped)

External Dependencies

None

Type of change

Please delete options that are not relevant. Add a 'X' to the one that is applicable.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • [X ] Implement fixes for ONSAM Jiras

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • [ X] Command Line
  • oneapi-cli
  • Visual Studio
  • Eclipse IDE
  • VSCode
  • When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

@jimmytwei
Copy link
Contributor

@zma2 Can you resubmit this to the development branch? Thanks.

@zma2
Copy link
Contributor Author

zma2 commented Jan 16, 2024

@zma2 Can you resubmit this to the development branch? Thanks.

@jimmytwei New PR in development branch #2138

Close this one.

@zma2 zma2 closed this Jan 16, 2024
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.

2 participants