Skip to content

Conversation

@szihs
Copy link
Collaborator

@szihs szihs commented Oct 16, 2025

The four optixMakeHitObject wrapper functions were enabled for OptiX 8.1+ but used OptixTraverseData and optixHitObjectGetTraverseData, which only exist in OptiX 9.0+. Added OptiX 8.1-specific implementations that call the native OptiX 8.1 APIs (optixMakeHitObject and optixMakeHitObjectWithRecord) with explicit SBT parameters.

The four optixMakeHitObject wrapper functions were enabled for OptiX 8.1+
but used OptixTraverseData and optixHitObjectGetTraverseData, which only
exist in OptiX 9.0+. Added OptiX 8.1-specific implementations that call
the native OptiX 8.1 APIs (optixMakeHitObject and optixMakeHitObjectWithRecord)
with explicit SBT parameters.
@szihs szihs added the pr: non-breaking PRs without breaking changes label Oct 16, 2025
@szihs szihs marked this pull request as ready for review October 16, 2025 12:31
@szihs szihs requested a review from a team as a code owner October 16, 2025 12:31
Copy link
Collaborator

@jkwak-work jkwak-work left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@csyonghe
Copy link
Collaborator

What's the testing plan?

@szihs
Copy link
Collaborator Author

szihs commented Oct 30, 2025

What's the testing plan?

@jkiviluoto-nv would work on adding it to CI for have coverage for supported optix
#8854

@szihs szihs added this pull request to the merge queue Oct 30, 2025
Merged via the queue into master with commit 6b1105b Oct 30, 2025
37 of 38 checks passed
jvepsalainen-nv pushed a commit to jvepsalainen-nv/slang that referenced this pull request Nov 4, 2025
…er-slang#8723 (shader-slang#8730)

The four optixMakeHitObject wrapper functions were enabled for OptiX
8.1+ but used OptixTraverseData and optixHitObjectGetTraverseData, which
only exist in OptiX 9.0+. Added OptiX 8.1-specific implementations that
call the native OptiX 8.1 APIs (optixMakeHitObject and
optixMakeHitObjectWithRecord) with explicit SBT parameters.
juliusikkala pushed a commit to juliusikkala/slang that referenced this pull request Nov 12, 2025
…er-slang#8723 (shader-slang#8730)

The four optixMakeHitObject wrapper functions were enabled for OptiX
8.1+ but used OptixTraverseData and optixHitObjectGetTraverseData, which
only exist in OptiX 9.0+. Added OptiX 8.1-specific implementations that
call the native OptiX 8.1 APIs (optixMakeHitObject and
optixMakeHitObjectWithRecord) with explicit SBT parameters.
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.

optixMakeHitObject uses OptiX 9.0 intrinsics but is enabled in OptiX 8.1+

4 participants