lee/renderdoc
1
0
Fork 0
mirror of https://github.com/baldurk/renderdoc.git synced 2026-08-21 14:06:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix overly aggressive check for UAV clears not accounting for depth

Browse Source
This commit is contained in:
baldurk
2025-09-16 11:19:14 +01:00
parent 2fd7cd94e3
commit 0a6a8e763a
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
renderdoc/driver/d3d12/d3d12_pixelhistory.cpp
+13 -5
View File
@@ -2917,14 +2917,22 @@ rdcarray<PixelModification> D3D12Replay::PixelHistory(rdcarray<EventUsage> event
else if(IsCopyWrite(usage))
resourceState = D3D12_RESOURCE_STATE_COPY_DEST;
// if we are assuming render target state but that's not allowed, this is a UAV clear
// if we are assuming render target state but that's not allowed, check for other states
if(resourceState == D3D12_RESOURCE_STATE_RENDER_TARGET &&
(resDesc.Flags & D3D12_RESOURCE_FLAG_ALLOW_RENDER_TARGET) == 0)
{
RDCASSERT(usage == ResourceUsage::Clear &&
(resDesc.Flags & D3D12_RESOURCE_FLAG_ALLOW_UNORDERED_ACCESS),
usage, resDesc.Flags);
resourceState = D3D12_RESOURCE_STATE_UNORDERED_ACCESS;
// if D/S is allowed, assume the state is depth write
if(resDesc.Flags & D3D12_RESOURCE_FLAG_ALLOW_DEPTH_STENCIL)
{
resourceState = D3D12_RESOURCE_STATE_DEPTH_WRITE;
}
// if it's not D/S either this must be a UAV clear
else
{
RDCASSERT(usage == ResourceUsage::Clear &&
(resDesc.Flags & D3D12_RESOURCE_FLAG_ALLOW_UNORDERED_ACCESS),
usage, resDesc.Flags);
}
}
eventInfo.resourceState = resourceState;