Skip to content

Commit a54c11e

Browse files
committed
wgsl: add spirv-unknown-naga-wgsl target, transpiling with naga 26
1 parent 84e43b8 commit a54c11e

File tree

9 files changed

+122
-0
lines changed

9 files changed

+122
-0
lines changed

Cargo.lock

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/rustc_codegen_spirv-target-specs/src/include_str.rs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,9 @@ impl SpirvTargetEnv {
5959
SpirvTargetEnv::Vulkan_1_4 => {
6060
include_str!("../target-specs/spirv-unknown-vulkan1.4.json")
6161
}
62+
SpirvTargetEnv::Naga_Wgsl => {
63+
include_str!("../target-specs/spirv-unknown-naga-wgsl.json")
64+
}
6265
}
6366
}
6467
}

crates/rustc_codegen_spirv-target-specs/src/lib.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,8 @@ pub enum SpirvTargetEnv {
6060
Vulkan_1_3,
6161
#[strum(to_string = "vulkan1.4")]
6262
Vulkan_1_4,
63+
#[strum(to_string = "naga-wgsl")]
64+
Naga_Wgsl,
6365
}
6466

6567
#[derive(Clone, Error, Eq, PartialEq)]
Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,26 @@
1+
{
2+
"allows-weak-linkage": false,
3+
"arch": "spirv",
4+
"crt-objects-fallback": "false",
5+
"crt-static-allows-dylibs": true,
6+
"crt-static-respected": true,
7+
"data-layout": "e-m:e-p:32:32:32-i64:64-n8:16:32:64",
8+
"dll-prefix": "",
9+
"dll-suffix": ".spv.json",
10+
"dynamic-linking": true,
11+
"emit-debug-gdb-scripts": false,
12+
"env": "naga-wgsl",
13+
"linker-flavor": "unix",
14+
"linker-is-gnu": false,
15+
"llvm-target": "spirv-unknown-naga-wgsl",
16+
"main-needs-argc-argv": false,
17+
"metadata": {
18+
"description": null,
19+
"host_tools": null,
20+
"std": null,
21+
"tier": null
22+
},
23+
"panic-strategy": "abort",
24+
"simd-types-indirect": false,
25+
"target-pointer-width": "32"
26+
}

crates/rustc_codegen_spirv/Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ itertools = "0.14.0"
6060
tracing.workspace = true
6161
tracing-subscriber.workspace = true
6262
tracing-tree = "0.4.0"
63+
naga = { version = "26.0.0", features = ["spv-in", "wgsl-out"] }
6364

6465
# required for cargo gpu to resolve the needed target specs
6566
rustc_codegen_spirv-target-specs.workspace = true

crates/rustc_codegen_spirv/src/lib.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,7 @@ mod custom_decorations;
133133
mod custom_insts;
134134
mod link;
135135
mod linker;
136+
mod naga_transpile;
136137
mod spirv_type;
137138
mod spirv_type_constraints;
138139
mod symbols;

crates/rustc_codegen_spirv/src/link.rs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
use crate::maybe_pqp_cg_ssa as rustc_codegen_ssa;
33

44
use crate::codegen_cx::{CodegenArgs, SpirvMetadata};
5+
use crate::naga_transpile::should_transpile;
56
use crate::{SpirvCodegenBackend, SpirvModuleBuffer, linker};
67
use ar::{Archive, GnuBuilder, Header};
78
use rspirv::binary::Assemble;
@@ -322,6 +323,10 @@ fn post_link_single_module(
322323

323324
drop(save_modules_timer);
324325
}
326+
327+
if let Some(transpile) = should_transpile(sess) {
328+
transpile(sess, cg_args, &spv_binary, out_filename).ok();
329+
}
325330
}
326331

327332
fn do_spirv_opt(
Lines changed: 79 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,79 @@
1+
use crate::codegen_cx::CodegenArgs;
2+
use rustc_codegen_spirv_target_specs::SpirvTargetEnv;
3+
use rustc_session::Session;
4+
use rustc_span::ErrorGuaranteed;
5+
use std::path::Path;
6+
7+
pub type NagaTranspile = fn(
8+
sess: &Session,
9+
cg_args: &CodegenArgs,
10+
spv_binary: &[u32],
11+
out_filename: &Path,
12+
) -> Result<(), ErrorGuaranteed>;
13+
14+
pub fn should_transpile(sess: &Session) -> Option<NagaTranspile> {
15+
let target = SpirvTargetEnv::parse_triple(sess.opts.target_triple.tuple())
16+
.expect("parsing should fail earlier");
17+
match target {
18+
SpirvTargetEnv::Naga_Wgsl => Some(transpile::wgsl_transpile),
19+
_ => None,
20+
}
21+
}
22+
23+
mod transpile {
24+
use crate::codegen_cx::CodegenArgs;
25+
use naga::error::ShaderError;
26+
use naga::valid::Capabilities;
27+
use rustc_session::Session;
28+
use rustc_span::ErrorGuaranteed;
29+
use std::path::Path;
30+
31+
pub fn wgsl_transpile(
32+
sess: &Session,
33+
_cg_args: &CodegenArgs,
34+
spv_binary: &[u32],
35+
out_filename: &Path,
36+
) -> Result<(), ErrorGuaranteed> {
37+
// these should be params via spirv-builder
38+
let opts = naga::front::spv::Options::default();
39+
let capabilities = Capabilities::default();
40+
let writer_flags = naga::back::wgsl::WriterFlags::empty();
41+
42+
let module = naga::front::spv::parse_u8_slice(bytemuck::cast_slice(spv_binary), &opts)
43+
.map_err(|err| {
44+
sess.dcx().err(format!(
45+
"Naga failed to parse spv: \n{}",
46+
ShaderError {
47+
source: String::new(),
48+
label: None,
49+
inner: Box::new(err),
50+
}
51+
))
52+
})?;
53+
let mut validator =
54+
naga::valid::Validator::new(naga::valid::ValidationFlags::default(), capabilities);
55+
let info = validator.validate(&module).map_err(|err| {
56+
sess.dcx().err(format!(
57+
"Naga validation failed: \n{}",
58+
ShaderError {
59+
source: String::new(),
60+
label: None,
61+
inner: Box::new(err),
62+
}
63+
))
64+
})?;
65+
66+
let wgsl_dst = out_filename.with_extension("wgsl");
67+
let wgsl = naga::back::wgsl::write_string(&module, &info, writer_flags).map_err(|err| {
68+
sess.dcx()
69+
.err(format!("Naga failed to write wgsl : \n{err}"))
70+
})?;
71+
72+
std::fs::write(&wgsl_dst, wgsl).map_err(|err| {
73+
sess.dcx()
74+
.err(format!("failed to write wgsl to file: {err}"))
75+
})?;
76+
77+
Ok(())
78+
}
79+
}

crates/rustc_codegen_spirv/src/target.rs

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,8 @@ impl TargetsExt for SpirvTargetEnv {
3333
| SpirvTargetEnv::Vulkan_1_2
3434
| SpirvTargetEnv::Vulkan_1_3
3535
| SpirvTargetEnv::Vulkan_1_4 => MemoryModel::Vulkan,
36+
37+
SpirvTargetEnv::Naga_Wgsl => MemoryModel::Vulkan,
3638
}
3739
}
3840

@@ -57,6 +59,8 @@ impl TargetsExt for SpirvTargetEnv {
5759
SpirvTargetEnv::Vulkan_1_2 => spirv_tools::TargetEnv::Vulkan_1_2,
5860
SpirvTargetEnv::Vulkan_1_3 => spirv_tools::TargetEnv::Vulkan_1_3,
5961
SpirvTargetEnv::Vulkan_1_4 => spirv_tools::TargetEnv::Vulkan_1_4,
62+
63+
SpirvTargetEnv::Naga_Wgsl => spirv_tools::TargetEnv::Vulkan_1_2,
6064
}
6165
}
6266

0 commit comments

Comments
 (0)