KhronosGroup / KhronosGroup/SPIRV-LLVM-Translator

Support All Spirv or just opencl?

Open
#369 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
LLVM
Stars
625
Forks
279
Avg merge
3d 5h
Merged PRs (30d)
34

Description

I'm getting a crash trying to dump a vulkan spirv compute shader compiled with glslang.

> glangValidator -V shader.comp -o shader.comp.spv
> llvm-spirv shader.comp.spv -r -o shader.comp.bc

> Read string: "GLSL.std.450"
InvalidBuiltinSetName: Expects OpenCL.std. Actual is GLSL.std.450 [Src: /home/steven/Downloads/SPIRV-LLVM-Translator/lib/SPIRV/libSPIRV/SPIRVModule.cpp:748 SPIRVBuiltinSetNameMap::rfind(BuiltinSetName, &BuiltinSet) ]

This tool is advertised as a generic SPIRV-LLVM translator? If it only works with specific spirv modules and pipelines, maybe that should be specified in the description.

Contributor guide

Open the contributing guide

Research direction

Reproduce the reported glslangValidator and llvm-spirv command sequence, then inspect SPIRVModule.cpp around line 748 where GLSL.std.450 is rejected. Review the project description to determine whether generic SPIR-V support or a narrower documented scope is intended. Done means the supported builtin-set scope is resolved and reflected in behavior or documentation.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.