diff options
| author | Tim Foley <tfoleyNV@users.noreply.github.com> | 2018-08-21 08:40:25 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-08-21 08:40:25 -0700 |
| commit | 0ce1131ba12f777fbaa40004e0e3e7af89ccf4f0 (patch) | |
| tree | 053d94527cab22111060087c53f18c4bd066ecd2 /source/slang/ir.h | |
| parent | 56d8a752d84e984afab20de5980edf10fe6c06f5 (diff) | |
Add support for more RasterizerOrdered types (#628)
Fixes #627
The front-end has support for `RasterizerOrderedBuffer` and `RasterizerOrderedTexture*`, but left out support for:
* `RasterizerOrderedByteAddressBuffer`
* `RasterizerOrderedStructuredBuffer`
[Nitpick: these tyeps are all amazingly annoying to type. It is easy to want to write `RasterOrdered` instead of the bulkier `RasterizerOrdered`, and almost everybody does in casual speech. There's already the issue of wanting to type `StructureBuffer` (a buffer of structures) instead of `StructuredBuffer` (a buffer that is... structured?). Then you have `ByteAddressBuffer` which is just adding to the confusion because it is nominally a "byte addressable" buffer (so that `ByteAddressedBuffer` would actually make sense), but then actually *isn't* byte addressable in practice.]
There were a few `TODO` comments related to this already, and this change was mostly a matter of doing a find-in-files for `RWByteAddressBuffer` and `RWStructuredBuffer` and adding matching `RasterizerOrdered` cases.
The test I added just checks that these types make it through the front-end, and doesn't do any actual confirmation that they work as intended. It is worth noting that the handling of ordering in GLSL/VK is different from in HLSL ("pixel shader interlock" instead of "rasterizer ordered views"), so coming up with a cross-compilation story would need to be a later step.
Diffstat (limited to 'source/slang/ir.h')
| -rw-r--r-- | source/slang/ir.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/source/slang/ir.h b/source/slang/ir.h index b23e26e5e..4c601ce5d 100644 --- a/source/slang/ir.h +++ b/source/slang/ir.h @@ -709,11 +709,12 @@ SIMPLE_IR_PARENT_TYPE(PointerLikeType, BuiltinGenericType); SIMPLE_IR_PARENT_TYPE(HLSLStructuredBufferTypeBase, BuiltinGenericType) SIMPLE_IR_TYPE(HLSLStructuredBufferType, HLSLStructuredBufferTypeBase) SIMPLE_IR_TYPE(HLSLRWStructuredBufferType, HLSLStructuredBufferTypeBase) -// TODO: need raster-ordered case here +SIMPLE_IR_TYPE(HLSLRasterizerOrderedStructuredBufferType, HLSLStructuredBufferTypeBase) SIMPLE_IR_PARENT_TYPE(UntypedBufferResourceType, Type) SIMPLE_IR_TYPE(HLSLByteAddressBufferType, UntypedBufferResourceType) SIMPLE_IR_TYPE(HLSLRWByteAddressBufferType, UntypedBufferResourceType) +SIMPLE_IR_TYPE(HLSLRasterizerOrderedByteAddressBufferType, UntypedBufferResourceType) SIMPLE_IR_TYPE(HLSLAppendStructuredBufferType, HLSLStructuredBufferTypeBase) SIMPLE_IR_TYPE(HLSLConsumeStructuredBufferType, HLSLStructuredBufferTypeBase) |