mirrors/Odin
1
0
Fork 0
mirror of https://github.com/odin-lang/Odin.git synced 2026-07-18 05:31:05 +00:00
Code Issues Packages Projects Releases Wiki Activity

rexcode/ir/spirv: printer -- symbolic ValueEnum names + computed bound in header

Browse Source 
fmt_enum prints the common ValueEnum operands by name (Capability Shader,
Addressing/Memory model Logical GLSL450, ExecutionModel GLCompute, StorageClass,
Decoration, ...); BitEnums stay numeric (their bit_set %v form is too verbose to
read inline). The header comment now reads version/generator/bound from the
encoded words, so Bound reflects the computed value, not the pre-encode 0.
This commit is contained in:
Brendan Punsky
2026-06-26 11:17:39 -04:00
committed by Flāvius
parent 35fac68c7c
commit 4de6a7e6b6
1 changed files with 25 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
core/rexcode/ir/spirv/printer.odin
View File

@@ -45,10 +45,10 @@ print :: proc(m: Module, sb: ^strings.Builder, options: ^Print_Options = nil) {
return
}
fmt.sbprintf(sb, "; SPIR-V\n; Version: %d.%d\n; Generator: 0x%08x\n; Bound: %d\n; Schema: 0\n",
(m.version >> 16) & 0xFF, (m.version >> 8) & 0xFF, m.generator, m.bound)
words := mem.slice_data_cast([]u32, buf[:n])
ver := words[1] // header fields as actually encoded (bound is computed there)
fmt.sbprintf(sb, "; SPIR-V\n; Version: %d.%d\n; Generator: 0x%08x\n; Bound: %d\n; Schema: 0\n",
(ver >> 16) & 0xFF, (ver >> 8) & 0xFF, words[2], words[3])
wi := int(HEADER_WORDS)
for wi < len(words) {
head := words[wi]
@@ -105,10 +105,32 @@ fmt_operand :: proc(sb: ^strings.Builder, kind: Spec_Kind, w: []u32, opts: ^Prin
s, nwords := str_from_words(w)
fmt.sbprintf(sb, "%q", s); return nwords
case:
if fmt_enum(sb, kind, w[0]) { return 1 }
fmt.sbprintf(sb, "%d", w[0]); return 1
}
}
// Symbolic name for the common enum operand kinds (others fall back to numeric).
// ValueEnums print their member name, BitEnums their set; an out-of-range value
// prints numerically via fmt's enum handling.
@(private="file")
fmt_enum :: proc(sb: ^strings.Builder, kind: Spec_Kind, v: u32) -> bool {
#partial switch kind {
case .SourceLanguage: fmt.sbprintf(sb, "%v", Source_Language(v))
case .ExecutionModel: fmt.sbprintf(sb, "%v", Execution_Model(v))
case .AddressingModel: fmt.sbprintf(sb, "%v", Addressing_Model(v))
case .MemoryModel: fmt.sbprintf(sb, "%v", Memory_Model(v))
case .ExecutionMode: fmt.sbprintf(sb, "%v", Execution_Mode(v))
case .StorageClass: fmt.sbprintf(sb, "%v", Storage_Class(v))
case .Decoration: fmt.sbprintf(sb, "%v", Decoration(v))
case .Capability: fmt.sbprintf(sb, "%v", Capability(v))
// BitEnums (FunctionControl, MemoryAccess, ...) stay numeric -- their bit_set
// %v form is too verbose to read inline.
case: return false
}
return true
}
@(private="file")
str_from_words :: proc(w: []u32) -> (s: string, nwords: int) {
buf := strings.builder_make(context.temp_allocator)