Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Require SER support on any SM 6.9 device with raytracing support. #379

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Require SER support on any SM 6.9 device with raytracing support. #379

wants to merge 2 commits into from

Conversation

amarpMSFT
Copy link
Collaborator

  • Require devices that support SM 6.9 and raytracing to support SER, whether it can reorder or not. If not, ReorderThread() is just a no-op.
  • Expose D3D runtime cap indicating BOOL ShaderExecutionReorderingActuallyReorders
  • Example of using SER to just TraceRay without Invoke() as a way to just get T value from the HitObject without running ClosestHit/Miss. This works on any device.

@amarpMSFT
Require SER support on any SM 6.9 device with raytracing support.
e284354 
- Require devices that support SM 6.9 and raytracing to support SER, whether it can reorder or not.  If not, ReorderThread() is just a no-op.
- Expose D3D runtime cap indicating BOOL ShaderExecutionReorderingActuallyReorders
- Example of using SER to just TraceRay without Invoke() as a way to just get T value from the HitObject without running ClosestHit/Miss. This works on any device.
@amarpMSFT amarpMSFT requested review from tex3d and damyanp January 31, 2025 20:03
@amarpMSFT
Reiterated ability to just get a HitObject and not ask for reorder or…
c6fa87b 
… invoke in the Detailed Design section, per request in PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@damyanp damyanp damyanp approved these changes

@jenatali jenatali jenatali approved these changes

@tex3d tex3d Awaiting requested review from tex3d

At least 2 approving reviews are required to merge this pull request.

Assignees

@tex3d tex3d

@llvm-beanz llvm-beanz

@pow2clk pow2clk

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@amarpMSFT @damyanp @jenatali @tex3d @llvm-beanz @pow2clk