Skip to content

Update Slang to 2026.16.1 - #1128

Merged
kaizhangNV merged 1 commit into
mainfrom
update-slang-2026.16.1
Aug 28, 2026
Merged

Update Slang to 2026.16.1#1128
kaizhangNV merged 1 commit into
mainfrom
update-slang-2026.16.1

Conversation

@kaizhangNV

Copy link
Copy Markdown
Contributor

Summary

  • update the bundled Slang compiler from 2026.12 to 2026.16.1

Testing

  • built Slang 2026.16.1 from source with native build parallelism capped at 8
  • built SlangPy against Slang 2026.16.1 with native build parallelism capped at 8
  • verified SLANG_BUILD_TAG=2026.16.1
  • pytest slangpy/tests/device/test_standard_modules.py -v (3 passed)
  • pre-commit run --all-files
  • git diff --check

Slang release: https://github.com/shader-slang/slang/releases/tag/v2026.16.1

@kaizhangNV
kaizhangNV requested a review from a team as a code owner August 28, 2026 22:20
@kaizhangNV
kaizhangNV requested review from bmillsNV and removed request for a team August 28, 2026 22:20
@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • external/CMakeLists.txt is excluded by !external/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 0eac6dde-a94b-409f-8fa8-1b60a1fd987c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jhelferty-nv
jhelferty-nv removed the request for review from bmillsNV August 28, 2026 22:20

@jhelferty-nv jhelferty-nv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to/should we bump slangpy's slang-rhi submodule pin? It looks like it might be 37 commits behind what's in slang 2026.16.1.

@jhelferty-nv

Copy link
Copy Markdown
Contributor

Correction, looks like my agent gaslit me on the numbers. Checking manually, slangpy's pin for slang-rhi is two ahead of where slang 2026.16.1 had slang-rhi pinned. I think it should be fine to approve this.

@jhelferty-nv jhelferty-nv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks fine to me.

@kaizhangNV
kaizhangNV merged commit 47f06a1 into main Aug 28, 2026
22 checks passed
@kaizhangNV
kaizhangNV deleted the update-slang-2026.16.1 branch August 28, 2026 23:20
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.

2 participants