Skip to content

[SYCL][Bindless][E2E] Fix bindless test bugs#21936

Open
Seanst98 wants to merge 1 commit intointel:syclfrom
Seanst98:sean/bindless-e2e-fixes
Open

[SYCL][Bindless][E2E] Fix bindless test bugs#21936
Seanst98 wants to merge 1 commit intointel:syclfrom
Seanst98:sean/bindless-e2e-fixes

Conversation

@Seanst98
Copy link
Copy Markdown
Contributor

@Seanst98 Seanst98 commented May 5, 2026

  • Change addressing mode from repeat to clamp in fetch tests (OpenCL repeat/mirrored repeat addressing modes are undefined with unnormalized coordinates - per the OpenCL C Kernel Language Specification §6.15.15.1.)
  • Add missing VERBOSE_PRINT comments for debugging

@Seanst98 Seanst98 requested a review from a team as a code owner May 5, 2026 12:18
@Seanst98 Seanst98 force-pushed the sean/bindless-e2e-fixes branch from 25f9aba to 174a6cd Compare May 5, 2026 12:20
@Seanst98
Copy link
Copy Markdown
Contributor Author

Seanst98 commented May 5, 2026

@sergey-semenov apologies for pinging you with this. Nobody has been automatically assigned to review this. Could you please add any relevant reviewer?

Thanks!

- Change addressing mode from repeat to clamp in fetch tests (OpenCL
  repeat/mirrored repeat addressing modes are undefined with unnormalized
  coordinates - per the OpenCL C Kernel Language Specification
  §6.15.15.1.)
- Add missing VERBOSE_PRINT comments for debugging
@Seanst98 Seanst98 force-pushed the sean/bindless-e2e-fixes branch from 174a6cd to 334f95d Compare May 5, 2026 15:27
@sergey-semenov
Copy link
Copy Markdown
Contributor

@intel/bindless-images-reviewers doesn't have auto-assignment turned on, not sure if that's intentional.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

@intel/llvm-gatekeepers please consider merging

@Seanst98
Copy link
Copy Markdown
Contributor Author

Seanst98 commented May 7, 2026

Hi @cperkinsintel, thanks for the review! Could you merge this? Thanks!

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