This captures the target shader model and pipeline stage into the DXIL
metadata for consumption by the DirectX runtime.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/test/CodeGen/DirectX/Metadata/shaderModel-vs.ll | ||
---|---|---|
6 | Should we report an error for vs_0_0? |
llvm/test/CodeGen/DirectX/Metadata/shaderModel-vs.ll | ||
---|---|---|
6 | Not in the backend. I think we should catch this error in the driver before we start compilation, and in the validator later. The test cases here cover some odd edge cases to ensure the compiler doesn’t crash, but we shouldn’t rely on the backend for producing errors to users unless there are no other alternatives. |
llvm/lib/Target/DirectX/DXILMetadata.cpp | ||
---|---|---|
68 | Shall we add "dx.version" in the function too since it is just shader model with offset on major. |
llvm/lib/Target/DirectX/DXILMetadata.cpp | ||
---|---|---|
68 | Yea, that’s probably fair. There is a possibility the two versions may not always be in sync, but we can cross that when we get there. |
llvm/lib/Target/DirectX/DXILMetadata.cpp | ||
---|---|---|
68 | I'd like to have that come in through a separate metadata utility function just for code organization. |
llvm/lib/Target/DirectX/DXILMetadata.cpp | ||
---|---|---|
64 | assert for invalid shader profile? |
assert for invalid shader profile?