Skip to content

[SER] Diagnose payload in HitObject::TraceRay|Invoke #7356

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

Merged
merged 1 commit into from
Apr 22, 2025

Conversation

simoll
Copy link
Contributor

@simoll simoll commented Apr 16, 2025

  • Generalize raypayload validation to HitObject::TraceRay|Invoke
  • Reject non-numeric payload types in [HitObject::]TraceRay|Invoke

Specification: https://github.com/microsoft/hlsl-specs/blob/main/proposals/0027-shader-execution-reordering.md

Bug: #7234 [SER] Diagnose and validate illegal use of HitObject in unsupported contexts

- Generalize raypayload detection to HitObject::TraceRay|Invoke
- Reject non-numeric payload types in [HitObject::]TraceRay|Invoke

Specification: https://github.com/microsoft/hlsl-specs/blob/main/proposals/0027-shader-execution-reordering.md

Bug: microsoft#7234    [SER] Diagnose and validate illegal use of HitObject in unsupported contexts
@tex3d tex3d merged commit 8988e47 into microsoft:main Apr 22, 2025
13 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in HLSL Roadmap Apr 22, 2025
@simoll simoll deleted the ser_diagtraceinvoke_patch branch April 23, 2025 12:04
@damyanp damyanp moved this to Closed in HLSL Support Apr 25, 2025
@damyanp damyanp removed this from HLSL Support Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants