Thanks to visit codestin.com
Credit goes to github.com

Skip to content

cuda: Emit error on subscript operator usage for textures #6813

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mkeshavaNV
Copy link
Contributor

Cuda does not support subscript operators for textures.

Fixes #6782

@mkeshavaNV mkeshavaNV added the pr: non-breaking PRs without breaking changes label Apr 15, 2025
@mkeshavaNV mkeshavaNV marked this pull request as ready for review April 15, 2025 11:32
@mkeshavaNV mkeshavaNV requested a review from a team as a code owner April 15, 2025 11:32
Cuda does not support subscript operators for textures.

Fixes shader-slang#6782
@mkeshavaNV
Copy link
Contributor Author

/format

@slangbot
Copy link
Contributor

🌈 Formatted, please merge the changes from this PR

@mkeshavaNV mkeshavaNV requested a review from Copilot April 15, 2025 12:00
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@mkeshavaNV mkeshavaNV requested a review from Copilot April 15, 2025 12:02
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (3)
  • source/slang/slang-check-expr.cpp: Language not supported
  • source/slang/slang-diagnostic-defs.h: Language not supported
  • tests/compute/texture-subscript-cuda.slang: Language not supported

@skallweitNV
Copy link
Collaborator

Ideally we would support subscript and Load on textures in CUDA though :) see #6781

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: non-breaking PRs without breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CUDA: Do not silently emit invalid code
3 participants