Skip to content

Build libdxcompiler from Traverse fork (spirv-allow-import)#9

Open
Jasper-Bekkers wants to merge 3 commits into
mainfrom
spirv-allow-import-build
Open

Build libdxcompiler from Traverse fork (spirv-allow-import)#9
Jasper-Bekkers wants to merge 3 commits into
mainfrom
spirv-allow-import-build

Conversation

@Jasper-Bekkers
Copy link
Copy Markdown
Member

Summary

  • Switches the three clone URLs in `.github/workflows/build-dxc.yaml` from `microsoft/DirectXShaderCompiler` to `Traverse-Research/DirectXShaderCompiler`.
  • Bumps `commit-hash` to `073bf7057` — the head of the `spirv-allow-import` branch, which adds `-fspv-allow-import` for SPIR-V Import linkage in `lib_*` targets ([SPIR-V] Add -fspv-allow-import for Import linkage in lib targets DirectXShaderCompiler#1).
  • README: clarifies that this repo builds from the fork.

Artifact names (`universal-apple-darwin`, `x86_64-unknown-linux-gnu`, `x86_64-pc-windows-msvc`, `aarch64-pc-windows-msvc`) are unchanged — `breda` consumers drop them in `crates/breda-shader-compiler/dxc//` as before.

Test plan

  • CI build succeeds on macos-15-xlarge, ubuntu-22.04, windows-2025
  • Resulting `libdxcompiler` binaries accept `-fspv-allow-import` (smoke test by re-running the spirv-linker e2e test against them)

🤖 Generated with Claude Code

Switches the three clone URLs in the build workflow to the Traverse
fork (Traverse-Research/DirectXShaderCompiler) and bumps `commit-hash`
to 073bf7057 — the head of the `spirv-allow-import` branch, which adds
the `-fspv-allow-import` SPIR-V flag for Import linkage in `lib_*`
targets (PR Traverse-Research/DirectXShaderCompiler#1).

The artifact names (`*-apple-darwin`, `x86_64-unknown-linux-gnu`, etc.)
are unchanged — downstream `breda` consumers still drop them in
`crates/breda-shader-compiler/dxc/<triple>/`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Jasper-Bekkers
Copy link
Copy Markdown
Member Author

This exists as a test, I just need the binaries for some testing for now.

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.

1 participant