From 1fc9f36114b3e919ee0ed499df4430babcf06a90 Mon Sep 17 00:00:00 2001 From: baldurk Date: Mon, 12 Aug 2019 17:50:39 +0100 Subject: [PATCH] Add support for VK_EXT_index_type_uint8 --- renderdoc/driver/vulkan/vk_common.h | 2 + renderdoc/driver/vulkan/vk_core.cpp | 3 + renderdoc/driver/vulkan/vk_next_chains.cpp | 3 +- renderdoc/driver/vulkan/vk_overlay.cpp | 13 +- renderdoc/driver/vulkan/vk_postvs.cpp | 56 +++-- renderdoc/driver/vulkan/vk_rendermesh.cpp | 6 + renderdoc/driver/vulkan/vk_serialise.cpp | 24 ++- renderdoc/driver/vulkan/vk_state.cpp | 10 +- .../driver/vulkan/wrappers/vk_cmd_funcs.cpp | 16 +- .../vulkan/wrappers/vk_device_funcs.cpp | 7 + util/test/data/VK_Int8_IBuffer/backbuffer.png | Bin 0 -> 9693 bytes util/test/demos/CMakeLists.txt | 1 + util/test/demos/demos.vcxproj | 1 + util/test/demos/demos.vcxproj.filters | 3 + util/test/demos/vk/vk_int8_ibuffer.cpp | 195 ++++++++++++++++++ util/test/tests/Vulkan/VK_Int8_IBuffer.py | 52 +++++ 16 files changed, 367 insertions(+), 25 deletions(-) create mode 100644 util/test/data/VK_Int8_IBuffer/backbuffer.png create mode 100644 util/test/demos/vk/vk_int8_ibuffer.cpp create mode 100644 util/test/tests/Vulkan/VK_Int8_IBuffer.py diff --git a/renderdoc/driver/vulkan/vk_common.h b/renderdoc/driver/vulkan/vk_common.h index dad87d7d2..3f29e95ef 100644 --- a/renderdoc/driver/vulkan/vk_common.h +++ b/renderdoc/driver/vulkan/vk_common.h @@ -782,6 +782,7 @@ DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceGroupProperties); DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceHostQueryResetFeaturesEXT); DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceIDProperties); DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceImageFormatInfo2); +DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceIndexTypeUint8FeaturesEXT); DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceMaintenance3Properties); DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceMemoryBudgetPropertiesEXT); DECLARE_REFLECTION_STRUCT(VkPhysicalDeviceMemoryPriorityFeaturesEXT); @@ -1020,6 +1021,7 @@ DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceGroupProperties); DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceHostQueryResetFeaturesEXT); DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceIDProperties); DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceImageFormatInfo2); +DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceIndexTypeUint8FeaturesEXT); DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceMaintenance3Properties); DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceMemoryBudgetPropertiesEXT); DECLARE_DESERIALISE_TYPE(VkPhysicalDeviceMemoryPriorityFeaturesEXT); diff --git a/renderdoc/driver/vulkan/vk_core.cpp b/renderdoc/driver/vulkan/vk_core.cpp index b9a65b98b..d37fbe9d6 100644 --- a/renderdoc/driver/vulkan/vk_core.cpp +++ b/renderdoc/driver/vulkan/vk_core.cpp @@ -705,6 +705,9 @@ static const VkExtensionProperties supportedExtensions[] = { { VK_EXT_HOST_QUERY_RESET_EXTENSION_NAME, VK_EXT_HOST_QUERY_RESET_SPEC_VERSION, }, + { + VK_EXT_INDEX_TYPE_UINT8_EXTENSION_NAME, VK_EXT_INDEX_TYPE_UINT8_SPEC_VERSION, + }, { VK_EXT_MEMORY_BUDGET_EXTENSION_NAME, VK_EXT_MEMORY_BUDGET_SPEC_VERSION, }, diff --git a/renderdoc/driver/vulkan/vk_next_chains.cpp b/renderdoc/driver/vulkan/vk_next_chains.cpp index 85c98d6ae..dd5820e9b 100644 --- a/renderdoc/driver/vulkan/vk_next_chains.cpp +++ b/renderdoc/driver/vulkan/vk_next_chains.cpp @@ -202,6 +202,8 @@ static void AppendModifiedChainedStruct(byte *&tempMem, VkStruct *outputStruct, VkPhysicalDeviceImageFormatInfo2); \ COPY_STRUCT(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_IMAGE_VIEW_IMAGE_FORMAT_INFO_EXT, \ VkPhysicalDeviceImageViewImageFormatInfoEXT); \ + COPY_STRUCT(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INDEX_TYPE_UINT8_FEATURES_EXT, \ + VkPhysicalDeviceIndexTypeUint8FeaturesEXT); \ COPY_STRUCT(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_MAINTENANCE_3_PROPERTIES, \ VkPhysicalDeviceMaintenance3Properties); \ COPY_STRUCT(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_MEMORY_BUDGET_PROPERTIES_EXT, \ @@ -517,7 +519,6 @@ static void AppendModifiedChainedStruct(byte *&tempMem, VkStruct *outputStruct, case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_GROUP_PROPERTIES: \ case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_IMAGE_DRM_FORMAT_MODIFIER_INFO_EXT: \ case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_IMAGELESS_FRAMEBUFFER_FEATURES_KHR: \ - case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INDEX_TYPE_UINT8_FEATURES_EXT: \ case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INLINE_UNIFORM_BLOCK_FEATURES_EXT: \ case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INLINE_UNIFORM_BLOCK_PROPERTIES_EXT: \ case VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_LINE_RASTERIZATION_FEATURES_EXT: \ diff --git a/renderdoc/driver/vulkan/vk_overlay.cpp b/renderdoc/driver/vulkan/vk_overlay.cpp index 969afb2ba..5b7226259 100644 --- a/renderdoc/driver/vulkan/vk_overlay.cpp +++ b/renderdoc/driver/vulkan/vk_overlay.cpp @@ -332,6 +332,7 @@ void VulkanDebugManager::PatchLineStripIndexBuffer(const DrawcallDescription *dr bytebuf indices; + uint8_t *idx8 = NULL; uint16_t *idx16 = NULL; uint32_t *idx32 = NULL; @@ -341,16 +342,18 @@ void VulkanDebugManager::PatchLineStripIndexBuffer(const DrawcallDescription *dr rs.ibuffer.offs + uint64_t(draw->indexOffset) * draw->indexByteWidth, uint64_t(draw->numIndices) * draw->indexByteWidth, indices); - if(rs.ibuffer.bytewidth == 2) - idx16 = (uint16_t *)indices.data(); - else + if(rs.ibuffer.bytewidth == 4) idx32 = (uint32_t *)indices.data(); + else if(rs.ibuffer.bytewidth == 1) + idx8 = (uint8_t *)indices.data(); + else + idx16 = (uint16_t *)indices.data(); } // we just patch up to 32-bit since we'll be adding more indices and we might overflow 16-bit. std::vector patchedIndices; - ::PatchLineStripIndexBuffer(draw, NULL, idx16, idx32, patchedIndices); + ::PatchLineStripIndexBuffer(draw, idx8, idx16, idx32, patchedIndices); indexBuffer.Create(m_pDriver, m_Device, patchedIndices.size() * sizeof(uint32_t), 1, GPUBuffer::eGPUBufferIBuffer); @@ -2271,6 +2274,8 @@ ResourceId VulkanReplay::RenderOverlay(ResourceId texid, CompType typeHint, Floa VkIndexType idxtype = VK_INDEX_TYPE_UINT16; if(fmt.indexByteStride == 4) idxtype = VK_INDEX_TYPE_UINT32; + else if(fmt.indexByteStride == 1) + idxtype = VK_INDEX_TYPE_UINT8_EXT; if(fmt.indexResourceId != ResourceId()) { diff --git a/renderdoc/driver/vulkan/vk_postvs.cpp b/renderdoc/driver/vulkan/vk_postvs.cpp index 1321f4dd7..794382e57 100644 --- a/renderdoc/driver/vulkan/vk_postvs.cpp +++ b/renderdoc/driver/vulkan/vk_postvs.cpp @@ -1616,9 +1616,11 @@ void VulkanReplay::FetchVSOut(uint32_t eventId) if(drawcall->flags & DrawFlags::Indexed) { - bool index16 = (idxsize == 2); + const bool restart = + pipeCreateInfo.pInputAssemblyState->primitiveRestartEnable && IsStrip(drawcall->topology); bytebuf idxdata; std::vector indices; + uint8_t *idx8 = NULL; uint16_t *idx16 = NULL; uint32_t *idx32 = NULL; @@ -1666,12 +1668,15 @@ void VulkanReplay::FetchVSOut(uint32_t eventId) // do ibuffer rebasing/remapping - idx16 = (uint16_t *)&idxdata[0]; - idx32 = (uint32_t *)&idxdata[0]; + if(idxsize == 4) + idx32 = (uint32_t *)&idxdata[0]; + else if(idxsize == 1) + idx8 = (uint8_t *)&idxdata[0]; + else + idx16 = (uint16_t *)&idxdata[0]; // only read as many indices as were available in the buffer - uint32_t numIndices = - RDCMIN(uint32_t(index16 ? idxdata.size() / 2 : idxdata.size() / 4), drawcall->numIndices); + uint32_t numIndices = RDCMIN(uint32_t(idxdata.size() / idxsize), drawcall->numIndices); uint32_t idxclamp = 0; if(drawcall->baseVertex < 0) @@ -1680,7 +1685,13 @@ void VulkanReplay::FetchVSOut(uint32_t eventId) // grab all unique vertex indices referenced for(uint32_t i = 0; i < numIndices; i++) { - uint32_t i32 = index16 ? uint32_t(idx16[i]) : idx32[i]; + uint32_t i32 = 0; + if(idx32) + i32 = idx32[i]; + else if(idx16) + i32 = uint32_t(idx16[i]); + else if(idx8) + i32 = uint32_t(idx8[i]); // apply baseVertex but clamp to 0 (don't allow index to become negative) if(i32 < idxclamp) @@ -1798,10 +1809,16 @@ void VulkanReplay::FetchVSOut(uint32_t eventId) // vertex buffer for(uint32_t i = 0; i < numIndices; i++) { - uint32_t i32 = index16 ? uint32_t(idx16[i]) : idx32[i]; + uint32_t i32 = 0; + if(idx32) + i32 = idx32[i]; + else if(idx16) + i32 = uint32_t(idx16[i]); + else if(idx8) + i32 = uint32_t(idx8[i]); // preserve primitive restart indices - if(i32 == (index16 ? 0xffff : 0xffffffff)) + if(restart && i32 == (0xffffffff >> ((4 - idxsize) * 8))) continue; // apply baseVertex but clamp to 0 (don't allow index to become negative) @@ -1812,10 +1829,12 @@ void VulkanReplay::FetchVSOut(uint32_t eventId) else if(drawcall->baseVertex > 0) i32 += drawcall->baseVertex; - if(index16) - idx16[i] = uint16_t(indexRemap[i32]); - else + if(idx32) idx32[i] = uint32_t(indexRemap[i32]); + else if(idx16) + idx16[i] = uint16_t(indexRemap[i32]); + else if(idx8) + idx8[i] = uint8_t(indexRemap[i32]); } bufInfo.size = RDCMAX((VkDeviceSize)64, (VkDeviceSize)idxdata.size()); @@ -2555,9 +2574,15 @@ void VulkanReplay::FetchVSOut(uint32_t eventId) m_PostVS.Data[eventId].vsout.idxbuf = VK_NULL_HANDLE; if(m_PostVS.Data[eventId].vsout.useIndices && state.ibuffer.buf != ResourceId()) { + VkIndexType type = VK_INDEX_TYPE_UINT16; + if(idxsize == 4) + type = VK_INDEX_TYPE_UINT32; + else if(idxsize == 1) + type = VK_INDEX_TYPE_UINT8_EXT; + m_PostVS.Data[eventId].vsout.idxbuf = rebasedIdxBuf; m_PostVS.Data[eventId].vsout.idxbufmem = rebasedIdxBufMem; - m_PostVS.Data[eventId].vsout.idxFmt = idxsize == 2 ? VK_INDEX_TYPE_UINT16 : VK_INDEX_TYPE_UINT32; + m_PostVS.Data[eventId].vsout.idxFmt = type; } m_PostVS.Data[eventId].vsout.hasPosOut = @@ -3251,7 +3276,12 @@ MeshFormat VulkanReplay::GetPostVSBuffers(uint32_t eventId, uint32_t instID, uin if(s.useIndices && s.idxbuf != VK_NULL_HANDLE) { ret.indexResourceId = GetResID(s.idxbuf); - ret.indexByteStride = s.idxFmt == VK_INDEX_TYPE_UINT16 ? 2 : 4; + if(s.idxFmt == VK_INDEX_TYPE_UINT32) + ret.indexByteStride = 4; + else if(s.idxFmt == VK_INDEX_TYPE_UINT8_EXT) + ret.indexByteStride = 1; + else + ret.indexByteStride = 2; } else { diff --git a/renderdoc/driver/vulkan/vk_rendermesh.cpp b/renderdoc/driver/vulkan/vk_rendermesh.cpp index f96181ced..0a44508c8 100644 --- a/renderdoc/driver/vulkan/vk_rendermesh.cpp +++ b/renderdoc/driver/vulkan/vk_rendermesh.cpp @@ -498,6 +498,8 @@ void VulkanReplay::RenderMesh(uint32_t eventId, const std::vector &s VkIndexType idxtype = VK_INDEX_TYPE_UINT16; if(fmt.indexByteStride == 4) idxtype = VK_INDEX_TYPE_UINT32; + else if(fmt.indexByteStride == 1) + idxtype = VK_INDEX_TYPE_UINT8_EXT; if(fmt.indexResourceId != ResourceId()) { @@ -624,6 +626,8 @@ void VulkanReplay::RenderMesh(uint32_t eventId, const std::vector &s VkIndexType idxtype = VK_INDEX_TYPE_UINT16; if(cfg.position.indexByteStride == 4) idxtype = VK_INDEX_TYPE_UINT32; + else if(cfg.position.indexByteStride == 1) + idxtype = VK_INDEX_TYPE_UINT8_EXT; if(cfg.position.indexResourceId != ResourceId()) { @@ -671,6 +675,8 @@ void VulkanReplay::RenderMesh(uint32_t eventId, const std::vector &s VkIndexType idxtype = VK_INDEX_TYPE_UINT16; if(cfg.position.indexByteStride == 4) idxtype = VK_INDEX_TYPE_UINT32; + else if(cfg.position.indexByteStride == 1) + idxtype = VK_INDEX_TYPE_UINT8_EXT; if(cfg.position.indexResourceId != ResourceId()) { diff --git a/renderdoc/driver/vulkan/vk_serialise.cpp b/renderdoc/driver/vulkan/vk_serialise.cpp index 3e200cf53..682103bfd 100644 --- a/renderdoc/driver/vulkan/vk_serialise.cpp +++ b/renderdoc/driver/vulkan/vk_serialise.cpp @@ -538,6 +538,10 @@ SERIALISE_VK_HANDLES(); PNEXT_STRUCT(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_HOST_QUERY_RESET_FEATURES_EXT, \ VkPhysicalDeviceHostQueryResetFeaturesEXT) \ \ + /* VK_EXT_index_type_uint8 */ \ + PNEXT_STRUCT(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INDEX_TYPE_UINT8_FEATURES_EXT, \ + VkPhysicalDeviceIndexTypeUint8FeaturesEXT) \ + \ /* VK_EXT_memory_budget */ \ PNEXT_STRUCT(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_MEMORY_BUDGET_PROPERTIES_EXT, \ VkPhysicalDeviceMemoryBudgetPropertiesEXT) \ @@ -915,9 +919,6 @@ SERIALISE_VK_HANDLES(); PNEXT_UNSUPPORTED(VK_STRUCTURE_TYPE_IMAGE_DRM_FORMAT_MODIFIER_EXPLICIT_CREATE_INFO_EXT) \ PNEXT_UNSUPPORTED(VK_STRUCTURE_TYPE_IMAGE_DRM_FORMAT_MODIFIER_PROPERTIES_EXT) \ \ - /* VK_EXT_index_type_uint8 */ \ - PNEXT_UNSUPPORTED(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INDEX_TYPE_UINT8_FEATURES_EXT) \ - \ /* VK_EXT_inline_uniform_block */ \ PNEXT_UNSUPPORTED(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INLINE_UNIFORM_BLOCK_FEATURES_EXT) \ PNEXT_UNSUPPORTED(VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INLINE_UNIFORM_BLOCK_PROPERTIES_EXT) \ @@ -5885,6 +5886,22 @@ void Deserialise(const VkPhysicalDeviceMemoryBudgetPropertiesEXT &el) DeserialiseNext(el.pNext); } +template +void DoSerialise(SerialiserType &ser, VkPhysicalDeviceIndexTypeUint8FeaturesEXT &el) +{ + RDCASSERT(ser.IsReading() || + el.sType == VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INDEX_TYPE_UINT8_FEATURES_EXT); + SerialiseNext(ser, el.sType, el.pNext); + + SERIALISE_MEMBER(indexTypeUint8); +} + +template <> +void Deserialise(const VkPhysicalDeviceIndexTypeUint8FeaturesEXT &el) +{ + DeserialiseNext(el.pNext); +} + template void DoSerialise(SerialiserType &ser, VkPhysicalDeviceMemoryPriorityFeaturesEXT &el) { @@ -6987,6 +7004,7 @@ INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceGroupProperties); INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceHostQueryResetFeaturesEXT); INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceIDProperties); INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceImageFormatInfo2); +INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceIndexTypeUint8FeaturesEXT); INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceMaintenance3Properties); INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceMemoryBudgetPropertiesEXT); INSTANTIATE_SERIALISE_TYPE(VkPhysicalDeviceMemoryPriorityFeaturesEXT); diff --git a/renderdoc/driver/vulkan/vk_state.cpp b/renderdoc/driver/vulkan/vk_state.cpp index 8daa0f1fc..2bcaaf8c7 100644 --- a/renderdoc/driver/vulkan/vk_state.cpp +++ b/renderdoc/driver/vulkan/vk_state.cpp @@ -282,9 +282,17 @@ void VulkanRenderState::BindPipeline(VkCommandBuffer cmd, PipelineBinding bindin } if(ibuffer.buf != ResourceId()) + { + VkIndexType type = VK_INDEX_TYPE_UINT16; + if(ibuffer.bytewidth == 4) + type = VK_INDEX_TYPE_UINT32; + else if(ibuffer.bytewidth == 1) + type = VK_INDEX_TYPE_UINT8_EXT; + ObjDisp(cmd)->CmdBindIndexBuffer( Unwrap(cmd), Unwrap(GetResourceManager()->GetCurrentHandle(ibuffer.buf)), - ibuffer.offs, ibuffer.bytewidth == 4 ? VK_INDEX_TYPE_UINT32 : VK_INDEX_TYPE_UINT16); + ibuffer.offs, type); + } for(size_t i = 0; i < vbuffers.size(); i++) { diff --git a/renderdoc/driver/vulkan/wrappers/vk_cmd_funcs.cpp b/renderdoc/driver/vulkan/wrappers/vk_cmd_funcs.cpp index 93917f37e..c29922fb7 100644 --- a/renderdoc/driver/vulkan/wrappers/vk_cmd_funcs.cpp +++ b/renderdoc/driver/vulkan/wrappers/vk_cmd_funcs.cpp @@ -2248,15 +2248,25 @@ bool WrappedVulkan::Serialise_vkCmdBindIndexBuffer(SerialiserType &ser, { m_RenderState.ibuffer.buf = GetResID(buffer); m_RenderState.ibuffer.offs = offset; - m_RenderState.ibuffer.bytewidth = indexType == VK_INDEX_TYPE_UINT32 ? 4 : 2; + + if(indexType == VK_INDEX_TYPE_UINT32) + m_RenderState.ibuffer.bytewidth = 4; + else if(indexType == VK_INDEX_TYPE_UINT8_EXT) + m_RenderState.ibuffer.bytewidth = 1; + else + m_RenderState.ibuffer.bytewidth = 2; } } } else { // track while reading, as we need to bind current topology & index byte width in AddDrawcall - m_BakedCmdBufferInfo[m_LastCmdBufferID].state.idxWidth = - (indexType == VK_INDEX_TYPE_UINT32 ? 4 : 2); + if(indexType == VK_INDEX_TYPE_UINT32) + m_BakedCmdBufferInfo[m_LastCmdBufferID].state.idxWidth = 4; + else if(indexType == VK_INDEX_TYPE_UINT8_EXT) + m_BakedCmdBufferInfo[m_LastCmdBufferID].state.idxWidth = 1; + else + m_BakedCmdBufferInfo[m_LastCmdBufferID].state.idxWidth = 2; // track while reading, as we need to track resource usage m_BakedCmdBufferInfo[m_LastCmdBufferID].state.ibuffer = GetResID(buffer); diff --git a/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp b/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp index bed617209..6aa95ea1d 100644 --- a/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp +++ b/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp @@ -2105,6 +2105,13 @@ bool WrappedVulkan::Serialise_vkCreateDevice(SerialiserType &ser, VkPhysicalDevi CHECK_PHYS_EXT_FEATURE(texelBufferAlignment); } END_PHYS_EXT_CHECK(); + + BEGIN_PHYS_EXT_CHECK(VkPhysicalDeviceIndexTypeUint8FeaturesEXT, + VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_INDEX_TYPE_UINT8_FEATURES_EXT); + { + CHECK_PHYS_EXT_FEATURE(indexTypeUint8); + } + END_PHYS_EXT_CHECK(); } if(availFeatures.depthClamp) diff --git a/util/test/data/VK_Int8_IBuffer/backbuffer.png b/util/test/data/VK_Int8_IBuffer/backbuffer.png new file mode 100644 index 0000000000000000000000000000000000000000..fa04502c4c25ef8fb27d6c528fdbf1c646120a12 GIT binary patch literal 9693 zcmdUVXIPV4*Cs~}#6t-dnh-z%X@Vd~@1jVRUP4EDjet}W5D*a&2`DABfCxwzY0`p7 z#~?)k={*o4A=FR==LzS$?|kp|&5yaRnfWn4;7YRhUe8`-ueI)bKXHcl?lRJG&{0uQ zF=}hwHm0Kb0}1?zo}~qTS+8IIOht9|t@dqo(~ybvY-|6;o}fEqZNrkayES|hXxsI+ z{ca5s1olJfHdRF5NpFSKX=%ZL9M`Hw088u{;wDI=#+{x#+Qj@W-o z`46#KKg5nz_cwkY!orsITSM?g#)oaX;=1@z-TINu!q>jdHS{l=)+|j|?m!W?*=z{` zY&o%aBjlcrc_t#34o@D`%r%GBAObN=cJ*mLH}O--=9Q1iDtHoSqZjMf_Rak5uO9I> zL}={=_fG2$M>n@5kBN)yzG@CPEKb{MG%KE53*oEeOqrnR&emBsEqV7fel5(1@|a)+ zaf6kLu!K@udY+@6-DCXJy_NOF`33~3u_cS!YbAC}*?cUqd!udBI`EgsHw$@05}wEZ zut|^XKOBum+lLH6{=OFQyZrZl^Y zawWGDiMrybC!lzPoUoyt5#zj1Ll@9wvO;Yt`2I4-I^QB@INuC4Nsh}fOo zN!l=4Skd098WY|R)i3T_nGz=yP5jMcSE5`0A4}@xWJ_+4pw{f=Yxpc%sBHCOUi|%IJBinbdhK(i35YgjZ?c}+`szxMO`TOFw_JOD`cUi^#%er&1A5K{J z-%Xlal`MoV2x1cll=-a&YAw_9LSM8LwdZ@-|BkEYiPrg(<`K&jsIuPpFr@4@yo0|_ zv-FyyfNgo%H*!gSzqF|eA+%3&xmt_9RM-t(fFH17}+GKJS<5>en3w`EKh zUSf#V_??ZJ-+)a2(x4eOB;oFfa9=3D#;nA}L%l6$&KoSCY}te;=o)GYqc1h%2|@*q zuc}LrstbH^o2G_`i0P^jB-p&l+NfwHV!fF>yZQrKO(js&e!9p0Q@Dx=6TzNG zyU*>?*0{;;uw;T(B#6DoW-4{FuVajyk`%^neF*j(r#j#;vj$vl$<#Zg!@%Ct z$=Fe^xZf|b|z2-<5c_;HzPuJzI=%;-lDvm$r0!SGJjLf z9*)1;=Y}UpAC%pdEaha1UuEy%J&C>)@npB520zgAIc#h&O%Z<3TpzHq8!(MWejBg1 zqA&e^nWq2k!T?ssnVK&3(j+2LvHImm54vCHb_r~~-_iy~9NfA-)AgJ2i@Nf9*#XlWy$ZD(0}nrj_UOC_(Ioc`9){9q@S0--@Ud} zt6j;vUP`*RyZBo=sJjuSnfZ{tbYtSH7d}ZTuu{udg^-k}B+B6Y2Q%i;YIV_t;LRp^ z#&r!6+eOmII6Bsil1M)%y41TGhHypL%A2rSDZ-D52~WxzT`$K3A2$T@6_cJby(3O0 zgkMW&RYj|=N-kuqZs}038Bnira&&R-4{B6(jG5+aU2kZ>8X-}nra&S<@pwl{*b-i* z`7%K$B)M|&BryYJi9jr|2j8N|mb&z8gXc_H0O^GVZ?MSD;Je$S9Ii8}jF3DBKf!U$ zg}E;|N&$qA4TFG@t)EAOfRoOm^;66jbulLd((J2Q{&wzH6oLfd|mR=YN zTA=Lyo0^_yxP;eemb}cYFE`Jbi+GOY3VH0*J-*o(=9<_~x6R+pRrUP!D_eQ)=P@!48={+{&+Dr;?v%X{#TJHl%d5)r%y$UMnSFkfet4?lx*# z$pCsUhTqLsS_oy#P3UUv>Fhcmg zBdaUwHl9cXplcE27j%s&oo&|pp!A-8I+%=UcQ)So1!X}TZSpto-t0&9i*wVSpRps2 zOiQ2XGPHf~_>okS6&@X?IIWkEq*1bT74j4d$@{t34NcWADFV8L<=Va_(>(ND0rM)OpFe;;Bh4?mc9V&S$-Kjb3fwoXqC)` zQY{&75EFaP1uIL9*oLW>;c1~|SHUw1_M3Wic8?&8F%{fg00$t!25| zK$@E-B(@!)mTbm@wtf(J5&R(XaCdaf{B)8b&G6XmswY==81pyfLV{ELNHXrZnA|9$ z&f;A4qZ7xb(nVK|QxDdk!Dlk@j5B$EU#h9jAjjEgq&EZ-KWC`7jUsq0avh4`{hbd- z0;>0_G8y>_u&$IvPzRd()W5Tey4zIEc%3q?4sf~pHZ*zV=&F-%yt`eY)`mH{Ecy*8 z%Sln|j0kcn&wI7m4cM}+KBPI&CfiKC>5bg}Tez|x4neq*nLu#^Ep-)k-n+*p%nH&h zrh=W~uou%*Z&FoCt#B~Lq(|P=kCqGhq{eC%`C{JRk5soR5fEG#ydASJ=QHm|(mwFt z#>zrK1NRkZB?UXf;R+`Ezm^x~rh9KUK7A|Sjzt8@_S|QA$P^hTYN%d07cdtc&)U&; zciF2f!dn{5P2HweQ5+*3^vuhgO@l)&RxZTaIsJgdI?6yd;nvH_0h&z~ml~MLKqshE zd>ZfW=<-Ax?Q(IR1kAKa4-GqyP(Pi?4pOzke7Dj{Q-@upM9YdK5-|e_U3(CcBN6cJ zs0*naoZhhOy=D743de7434wcVezd^g?k@UDoE|^3%MA%>#r>(9fQso70i1q>`7$6F z>vH8nRNsq#t2OZ=_>mq*Vxf@?Ej(fFWAn|9xI)zQ=t7bGSZ;sudRZ#`lIH!Sj~8hQ zk)AH)r^%o}Xb+!go+D|qtfVT2o!YUI@k(#Vp809|Z_I+lM;V;Mq}i?J z+R@iszGSlTkX_2#P6N{XWjd(QnP%lhG&$KuGsYd0I{M>aN6C*gz~-7W`?>J~&2)F}g-}`=udM_L7Q&G8?y0S8B3~=O-hTb% zN4ht_;!4bnEAuiOoYo#FnA65oMIH(t)Lo#&DOOBW^??wFqJ^C`q?zaDGrOdy0Txii z;SgZU2J&-nA!1BdZb^x!6Qq&fA4!u39ya)qhKn3x&1aFa{MaeC z-9n4p+$KW^To;qoF=I;a<#npZGi6Qci<*oP#g1NmhA2gBXGM&Ff1KR@)=i|hTAOZX z+t`wB2MX;IUun*Kst}77u?t(({%!E7{;Z*F;dnr?T*#Nc=T>o1Y0siJQYv9p>HloQ z^oe!Jjey=Oo50Z;MkjEBO*HI(OO}t}m>;?U9>mNvde&CKW8oL;_hrOwyeLkr=IC)c z8t~Ig-wEt&(}gX4z2HiG16ySjTK&olib$?}$G_cnz<#eDQBfbCFq%hrw(m5uo6g|1r?tVk4G~>l3DE|Xk>0o4g#Acj2f$WUFrT=srSLbC zrMoBvq$m;&Yqf~Q=IMLIMb{fS;)pxqJ~rx{mq4)de59)99rcV?h5wE#pd|19;NE+R z%^Rpy-?dttBZ!qHT$2+3&&Ve5nJ3*Q+HAa#p0%Rvncl;3`R4tMh&q$sFVfgRFcD7j zbFFE0?mMtdM<=?8ZqLV9)Av`=2K68At*Ff4h97&rwaD>$M#*>TVr4fH(szN96qFu1 z?&+8}cSZ4;2B5sF`M-j0ZY!?(EVoA*cALH46omO^rJ;F-GGhGUXG}rGfeoR4Zg? zIPOO(r{n)9{?LKrD1&}RUh*gc@O5sc|XG<0> zdJ!o<rlPGu8&||;9I(w(ClmK$p#4)sWmBnqaa>F zxI=l&G`?9WyJ=0jB1jDWlW&Y%6rh-0pBTHc*GAa$4=c;&u_IgBs&co2x?v-mrF!P(_gfi3M0aocelk0!hiJ=mYE9dns>tdtk-a(guPJ$uJAjti{NVE8EoQN zzdf-I?tT-Zz}jTowD5rV+!%N^a{--29~U~n<-v}BMYtFof`QF85~u9G>WptT``uIw zZ%G?h2`6$ddyB7dyX^gt3r`CGI5k;$vYD&>(i9u&xDtC@f3nzm^2&&n8kHEaIc$9_ z!o#eK3477T^pVInP=J}0#R!_sK=Tgisn|N3q(pInhaFpd)&@*dB1_P{<1Ph_$1hm<-7#CieA-}*w*>r z!>Wv-(OJH#Sl$&xd-?!GAOO8VV6l6t+k0RWTo zK+YC0g$eVdYY`z6WTlm>EEjR(P7jhCuehD9`D1&HI#KZ2&=)4Nt5H$obSkOI4I1dp zX!;koc1!pJ84~i%LQMzQZfM|3bo}4%j0tK-5xUoIDeM>~4hSw+e++%!_&icHlBe&` zdM@7=$Q9|XacBuu6*AIYdF;OJQ{0Wc9B^VG;#hDdoI&I~^G~w}Jk2vY_{FLVo9AB% zah6v+K|QSfaBsHsrOcS&LD`D%m)8TlVr7LkBex6oq=AaH^*J_re%`N$Q~BMhck@GO zheD(j1biHP__Q_P;WiVCmPT8qVsVI8O@$%pA)EobXc;$!+HqLlOX0Y?u_jh#j`-m+ z>XHu{u>_$+(Sb~)`C8F~vUZ!YDq3px9tf?m3L|1(BBU~#$mhk2&`9SYVYc~VWmFya zy)bS^95;?_f88>o2_}!oh%xJlSpJF2%a7A)tZ(OY<(5|FTQ6~SS#TrQrsUt)F#Q=~{FgbEB6juSUj=;<1j#L#Iq-+lMXPjRN6Sj+L;p zI}@cQZ3}iH#IO=%!nKb~6cvOdlA7}4p2XvlSk1k-I_j;V{o9{AYu_C|%lEK63f>D6 zIEY%4w*G&|GL0IgJX{}Kv$CPUJ>NcJ-n<+`cP_xxGiVRH=`r_Bw_hzL*}s5-gs@wG zU66}N4dUTvE^8ef|F#f{Rl0TYU~Yi`3DNq)upC!HFWcJX7a15rea0D4!;HE_}KJQp>zBSK;^@x4l#%<(0t9--17nV3S}qovYi_bS2=O_HSEsYWPVUmdMG@x zi!DCdV}qXaOfJur_aJNDY7R}(FSY1|PH6w?w&s_kBP;c9g;3;p*Kc5PJikCga%Bt4OnEbaQ zO$yN5V7im79E^({h& z3(&+Tj`UQWr!m+Hoce7fb+NQE5wdviv^N!?-aOWL90<bbjAA&i|;=wASkcFU4EH^^Qp-jd~w(l>WA z-idpsyXSbO-OuFOpCQ$ao9tiwo3ECRMNN7vH6f$+&ZYOzu_&2eI{G9k8kA#cli1nj zrE%I7`&^c1Jo_cf4Ip;((Y)(r(oay<#z|7b?&z!a!*cuP%R}BSfzxakRO)w6VUVw5 z!j(X>h7x$D{5p0@K%WLAG2JX5@ZeiPXoc>1N9+x$69YDq`8DQL1ad*=m z18AMCTQ7Sx^f~-u-c(GGf!G@}@qB?aJ=5)u9 zMdK56jD?PnS}q&T(^vrFXY0g9us;Ie$B1hH+u@e7$w1f00yyF))1bnr=6#mlu2Wv< z7QNT1MB3!ghGb$bFuHOrN4w1{F`8U4X=yx?8B;m=fMRl8Pe=XiZ!h8j!iv*XMU~q7%jk{Q;;M$NGlzeZob)AXhZ#bt5>GIWH@QI@6_Vf?TE_2erh~Q~5O~mls&{bU zl(!Z|+_w2;#P&SjQDj0!R*?C&N|M)Esrs+BmQq4L8V#1{n18;mY)G#N^Bq+Qs&;() zdR&jPh`vnCdk($5Be!3qo{IzB3kOy;#~Q1uPj%-$ni`@Re8 z*S~gOi+bf82o}#E#WMpiZ2;*xers{gddmfevfi$~ae4;NiygC`3$(g0~Z01Gt2&A1uRvcqmPL<9?Wv4 zkB>GG<0kiw{c@jTq#)rBHFW&-o07HTk}80TzzRNJk}|st06C*a5nVDc z05_vCGRIdkl>r*K>3xMBMfN#R#8k{kvUt0e=!M`npO}ry%cM_%9b2c$@lc=1)6%S! zq8L=B-4|Czsrql3$8;<=UwQvQVfGG7-T)l%b{lYc^1>2Tp8{XY20K0N2T*iy!{oiR zvbiijl3ntbcMnAUrQS9?f&Wb~7qj#<;J)>Tl3W(%(5l!erW~5 zqjG8K+bgxF+yGy(PqW5M4v<%QU?6%^QpUm@fM<0midhVP=1@fE2!c-zK4Z77EiK;U z9D zj7!g1;HOx7{@yj^X|~~H@s)tF`cu;C1+Ao{oZojHhmQFn+w*8xZ`!LLnVyb5ViY|tRdu^N ze111CF72&t9m9~8nd z@gC2))^+pIbrui|3N^ZBM7rQdYT4QqW4-v#Hl9Dwm3F-@&u@1f*h#ftyQAkS!M<>J zA}fmc!jCjFW^GX}le&R~EwIkwVgNb^sEg**q29mcU;gz9Qy~|EPnd#!#@g;Mw){ae zjRLH;#ggKzBFDq`H=zK19J#|7i9k06Bm77Tx)3<@sF>4)1o0z9FUS9~&O9x&n%=X@ z*yW$%o-@-AzRruR9C!eHmsf+fBW_*jRf6!UKv&zN-=?Ab%iDa*)}fRD0Iq+OixqqS zxJe!eDfc7crI86ja#z8atLP2nKsVbDCoh3)%@19te9cYd#yWWoDPQ9u2Jr17*s1K3 zf`n`Ce2a5p_y1HePuj1`(KN-(t@z0NlMP^-+V=Lcs^sgjgv4lrfxP%{u%}V$ zI!ktICWFjSHKyh5I4&!U2IjQNOS;i)l}E(+_PM<+R|aF`^z)1!m_mN}oo|5#@-LTS zd)VBPH!@C>w-!w@$C-dVwsK0I5gz`!l^is5=OJ@heYJ}2=y?!N%785ENc4DcCH!xL ze5G5`g)!Us`$!d?ug1ykcUQ2|(1|F5F=ml}l4EK{H4c~lCmGeK`tIfh0j}vt%(C3R z?N+{>(fc4}phr6XMisHPdGrLX(sEK&ds_d?)lLjr0#?5xe~i7>{yS7d zjNV(Q$9J@$tt?c+?SYn(^YpH1lx`{F^++e$ztraI7P=EN#+{<&!SW_VT znn8Y5or%kc<^B6QHQ_YfltxZ5@*Mh43|YrT|E{e|V50#Ey8|!LJZgtz6b{v@NSb4S zo2vGbD#~j}dKZD0kM{rM`3s<$sQ;j%V)&DaO5z+9)h9LJhkt+guOt6Cv4hp7JnNBksV literal 0 HcmV?d00001 diff --git a/util/test/demos/CMakeLists.txt b/util/test/demos/CMakeLists.txt index 07f38f728..729e5b026 100644 --- a/util/test/demos/CMakeLists.txt +++ b/util/test/demos/CMakeLists.txt @@ -14,6 +14,7 @@ set(VULKAN_SRC vk/vk_draw_zoo.cpp vk/vk_image_layouts.cpp vk/vk_indirect.cpp + vk/vk_int8_ibuffer.cpp vk/vk_multi_thread_windows.cpp vk/vk_overlay_test.cpp vk/vk_resource_lifetimes.cpp diff --git a/util/test/demos/demos.vcxproj b/util/test/demos/demos.vcxproj index 0b489f948..6ef3aba07 100644 --- a/util/test/demos/demos.vcxproj +++ b/util/test/demos/demos.vcxproj @@ -209,6 +209,7 @@ + diff --git a/util/test/demos/demos.vcxproj.filters b/util/test/demos/demos.vcxproj.filters index 8acd6803b..7cc5edd19 100644 --- a/util/test/demos/demos.vcxproj.filters +++ b/util/test/demos/demos.vcxproj.filters @@ -309,6 +309,9 @@ D3D11\demos + + Vulkan\demos + diff --git a/util/test/demos/vk/vk_int8_ibuffer.cpp b/util/test/demos/vk/vk_int8_ibuffer.cpp new file mode 100644 index 000000000..4a21dd87b --- /dev/null +++ b/util/test/demos/vk/vk_int8_ibuffer.cpp @@ -0,0 +1,195 @@ +/****************************************************************************** + * The MIT License (MIT) + * + * Copyright (c) 2018-2019 Baldur Karlsson + * + * Permission is hereby granted, free of charge, to any person obtaining a copy + * of this software and associated documentation files (the "Software"), to deal + * in the Software without restriction, including without limitation the rights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE + * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN + * THE SOFTWARE. + ******************************************************************************/ + +#include "vk_test.h" + +TEST(VK_Int8_IBuffer, VulkanGraphicsTest) +{ + static constexpr const char *Description = "Draws a triangle strip with 8-bit width indices"; + + std::string common = R"EOSHADER( + +#version 420 core + +struct v2f +{ + vec4 pos; + vec4 col; + vec4 uv; +}; + +)EOSHADER"; + + const std::string vertex = R"EOSHADER( + +layout(location = 0) in vec3 Position; +layout(location = 1) in vec4 Color; +layout(location = 2) in vec2 UV; + +layout(location = 0) out v2f vertOut; + +void main() +{ + vertOut.pos = vec4(Position.xyz*vec3(1,-1,1), 1); + gl_Position = vertOut.pos; + vertOut.col = Color; + vertOut.uv = vec4(UV.xy, 0, 1); +} + +)EOSHADER"; + + const std::string pixel = R"EOSHADER( + +layout(location = 0) in v2f vertIn; + +layout(location = 0, index = 0) out vec4 Color; + +void main() +{ + Color = vertIn.col; +} + +)EOSHADER"; + + void Prepare(int argc, char **argv) + { + devExts.push_back(VK_EXT_INDEX_TYPE_UINT8_EXTENSION_NAME); + + VulkanGraphicsTest::Prepare(argc, argv); + } + + int main() + { + // initialise, create window, create context, etc + if(!Init()) + return 3; + + VkPipelineLayout layout = createPipelineLayout(vkh::PipelineLayoutCreateInfo()); + + vkh::GraphicsPipelineCreateInfo pipeCreateInfo; + + pipeCreateInfo.layout = layout; + pipeCreateInfo.renderPass = mainWindow->rp; + + pipeCreateInfo.vertexInputState.vertexBindingDescriptions = {vkh::vertexBind(0, DefaultA2V)}; + pipeCreateInfo.vertexInputState.vertexAttributeDescriptions = { + vkh::vertexAttr(0, 0, DefaultA2V, pos), vkh::vertexAttr(1, 0, DefaultA2V, col), + vkh::vertexAttr(2, 0, DefaultA2V, uv), + }; + + pipeCreateInfo.stages = { + CompileShaderModule(common + vertex, ShaderLang::glsl, ShaderStage::vert, "main"), + CompileShaderModule(common + pixel, ShaderLang::glsl, ShaderStage::frag, "main"), + }; + + pipeCreateInfo.inputAssemblyState.topology = VK_PRIMITIVE_TOPOLOGY_TRIANGLE_STRIP; + pipeCreateInfo.inputAssemblyState.primitiveRestartEnable = VK_TRUE; + + VkPipeline pipe = createGraphicsPipeline(pipeCreateInfo); + + DefaultA2V strip[] = { + {Vec3f(-0.8f, 0.2f, 0.0f), Vec4f(1.0f, 0.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(-0.8f, 0.7f, 0.0f), Vec4f(0.0f, 1.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(-0.4f, 0.2f, 0.0f), Vec4f(0.0f, 0.0f, 1.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(-0.4f, 0.7f, 0.0f), Vec4f(1.0f, 0.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.0f, 0.2f, 0.0f), Vec4f(0.0f, 1.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.0f, 0.7f, 0.0f), Vec4f(0.0f, 0.0f, 1.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.4f, 0.2f, 0.0f), Vec4f(1.0f, 0.0f, 1.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.4f, 0.7f, 0.0f), Vec4f(1.0f, 1.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + + {Vec3f(-0.8f, -0.7f, 0.0f), Vec4f(1.0f, 0.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(-0.8f, -0.2f, 0.0f), Vec4f(0.0f, 1.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(-0.4f, -0.7f, 0.0f), Vec4f(0.0f, 0.0f, 1.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(-0.4f, -0.2f, 0.0f), Vec4f(1.0f, 0.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.0f, -0.7f, 0.0f), Vec4f(0.0f, 1.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.0f, -0.2f, 0.0f), Vec4f(0.0f, 0.0f, 1.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.4f, -0.7f, 0.0f), Vec4f(1.0f, 0.0f, 1.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + {Vec3f(0.4f, -0.2f, 0.0f), Vec4f(1.0f, 1.0f, 0.0f, 1.0f), Vec2f(0.0f, 0.0f)}, + }; + + AllocatedBuffer vb(allocator, + vkh::BufferCreateInfo(sizeof(strip), VK_BUFFER_USAGE_VERTEX_BUFFER_BIT | + VK_BUFFER_USAGE_TRANSFER_DST_BIT), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_CPU_TO_GPU})); + + vb.upload(strip); + + uint8_t idx[] = { + // strip 0 + 0, 1, 2, 3, 4, 5, 6, 7, + + // restart + 0xff, + + // strip 1 + 8, 9, 10, 11, 12, 13, 14, 15, + }; + + AllocatedBuffer ib(allocator, + vkh::BufferCreateInfo(sizeof(idx), VK_BUFFER_USAGE_INDEX_BUFFER_BIT | + VK_BUFFER_USAGE_TRANSFER_DST_BIT), + VmaAllocationCreateInfo({0, VMA_MEMORY_USAGE_CPU_TO_GPU})); + + ib.upload(idx); + + while(Running()) + { + VkCommandBuffer cmd = GetCommandBuffer(); + + vkBeginCommandBuffer(cmd, vkh::CommandBufferBeginInfo()); + + VkImage swapimg = + StartUsingBackbuffer(cmd, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_GENERAL); + + vkCmdClearColorImage(cmd, swapimg, VK_IMAGE_LAYOUT_GENERAL, + vkh::ClearColorValue(0.4f, 0.5f, 0.6f, 1.0f), 1, + vkh::ImageSubresourceRange()); + + vkCmdBeginRenderPass( + cmd, vkh::RenderPassBeginInfo(mainWindow->rp, mainWindow->GetFB(), mainWindow->scissor), + VK_SUBPASS_CONTENTS_INLINE); + + vkCmdBindPipeline(cmd, VK_PIPELINE_BIND_POINT_GRAPHICS, pipe); + vkCmdSetViewport(cmd, 0, 1, &mainWindow->viewport); + vkCmdSetScissor(cmd, 0, 1, &mainWindow->scissor); + vkh::cmdBindVertexBuffers(cmd, 0, {vb.buffer}, {0}); + vkCmdBindIndexBuffer(cmd, ib.buffer, 0, VK_INDEX_TYPE_UINT8_EXT); + vkCmdDrawIndexed(cmd, 17, 1, 0, 0, 0); + + vkCmdEndRenderPass(cmd); + + FinishUsingBackbuffer(cmd, VK_ACCESS_TRANSFER_WRITE_BIT, VK_IMAGE_LAYOUT_GENERAL); + + vkEndCommandBuffer(cmd); + + Submit(0, 1, {cmd}); + + Present(); + } + + return 0; + } +}; + +REGISTER_TEST(); diff --git a/util/test/tests/Vulkan/VK_Int8_IBuffer.py b/util/test/tests/Vulkan/VK_Int8_IBuffer.py new file mode 100644 index 000000000..818e1973b --- /dev/null +++ b/util/test/tests/Vulkan/VK_Int8_IBuffer.py @@ -0,0 +1,52 @@ +import rdtest +import renderdoc as rd + + +class VK_Int8_IBuffer(rdtest.TestCase): + demos_test_name = 'VK_Int8_IBuffer' + + def check_capture(self): + self.check_final_backbuffer() + + draw = self.find_draw("Draw") + + self.check(draw is not None) + + self.controller.SetFrameEvent(draw.eventId, False) + + pipe: rd.PipeState = self.controller.GetPipelineState() + + postvs_data = self.get_postvs(rd.MeshDataStage.VSOut, 0, draw.numIndices) + + # Calculate the strip restart index for this index width + striprestart_index = pipe.GetStripRestartIndex() & ((1 << (draw.indexByteWidth*8)) - 1) + + # We don't check all of the output, we check a few key vertices to ensure they match up + postvs_ref = { + 0: { + 'vtx': 0, + 'idx': 0, + 'gl_PerVertex.gl_Position': [-0.8, -0.2, 0.0, 1.0], + 'vertOut.col': [1.0, 0.0, 0.0, 1.0], + 'vertOut.uv': [0.0, 0.0, 0.0, 1.0], + }, + 4: { + 'vtx': 4, + 'idx': 4, + 'gl_PerVertex.gl_Position': [0.0, -0.2, 0.0, 1.0], + 'vertOut.col': [0.0, 1.0, 0.0, 1.0], + 'vertOut.uv': [0.0, 0.0, 0.0, 1.0], + }, + 8: { + 'idx': striprestart_index + }, + 9: { + 'vtx': 9, + 'idx': 8, + 'gl_PerVertex.gl_Position': [-0.8, 0.7, 0.0, 1.0], + 'vertOut.col': [1.0, 0.0, 0.0, 1.0], + 'vertOut.uv': [0.0, 0.0, 0.0, 1.0], + }, + } + + self.check_mesh_data(postvs_ref, postvs_data)