Skip to content

Commit

Permalink
Drop 'GetBufferByReplayDeviceAddress' - won't be needed
Browse files Browse the repository at this point in the history
- we'll prob. only require lookup of capture-time addresses
  • Loading branch information
fabian-lunarg committed Sep 10, 2024
1 parent 6a804c0 commit d31af56
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 21 deletions.
12 changes: 0 additions & 12 deletions framework/decode/vulkan_buffer_tracker.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -38,11 +38,6 @@ void decode::VulkanBufferTracker::TrackBuffer(const decode::BufferInfo* buffer_i
{
_capture_addresses[buffer_info->capture_address] = buffer_info->capture_id;
}

if (buffer_info->replay_address != 0)
{
_replay_addresses[buffer_info->replay_address] = buffer_info->capture_id;
}
}
}

Expand All @@ -51,16 +46,9 @@ void VulkanBufferTracker::RemoveBuffer(const BufferInfo* buffer_info)
if (buffer_info != nullptr)
{
_capture_addresses.erase(buffer_info->capture_address);
_replay_addresses.erase(buffer_info->replay_address);
}
}

const decode::BufferInfo*
decode::VulkanBufferTracker::GetBufferByReplayDeviceAddress(VkDeviceAddress replay_address) const
{
return GetBufferInfo(replay_address, _replay_addresses);
}

const decode::BufferInfo*
decode::VulkanBufferTracker::GetBufferByCaptureDeviceAddress(VkDeviceAddress capture_address) const
{
Expand Down
10 changes: 1 addition & 9 deletions framework/decode/vulkan_buffer_tracker.h
Original file line number Diff line number Diff line change
Expand Up @@ -58,14 +58,6 @@ class VulkanBufferTracker
*/
void RemoveBuffer(const BufferInfo* buffer_info);

/**
* @brief Retrieve a buffer by providing a replay-time VkDeviceAddress within its range.
*
* @param replay_address a replay-time VkDeviceAddress pointing inside a buffer.
* @return a const-pointer to a found BufferInfo or nullptr.
*/
[[nodiscard]] const BufferInfo* GetBufferByReplayDeviceAddress(VkDeviceAddress replay_address) const;

/**
* @brief Retrieve a buffer by providing a capture-time VkDeviceAddress within its range.
*
Expand All @@ -82,7 +74,7 @@ class VulkanBufferTracker
const device_address_map_t& address_map) const;

const VulkanObjectInfoTable& _object_info_table;
device_address_map_t _capture_addresses, _replay_addresses;
device_address_map_t _capture_addresses;
};

GFXRECON_END_NAMESPACE(decode)
Expand Down

0 comments on commit d31af56

Please sign in to comment.