Skip to content

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

Merged
dm-vodopyanov merged 1 commit into
intel:syclfrom
Seanst98:sean/bindless-e2e-fixes
May 12, 2026
Merged

[SYCL][Bindless][E2E] Fix bindless test bugs#21936
dm-vodopyanov merged 1 commit into
intel: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!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 8, 2026

@intel/llvm-gatekeepers please consider merging

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

@intel/llvm-gatekeepers please consider merging

@dm-vodopyanov dm-vodopyanov merged commit 2214f5b into intel:sycl May 12, 2026
31 checks passed
@Seanst98 Seanst98 deleted the sean/bindless-e2e-fixes branch May 12, 2026 14:58
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