Update and Fill Buffers
Implemented vkCmdUpdateBuffer and vkCmdFillBuffer.
Passes all tests in:
dEQP-VK.api.fill_and_update_buffer.*
Bug b/118619338 b/118383648
Change-Id: I3cf317b4cb766618a54b2550019ac2702c24bfed
Reviewed-on: https://swiftshader-review.googlesource.com/c/24529
Tested-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
diff --git a/src/Vulkan/VkCommandBuffer.cpp b/src/Vulkan/VkCommandBuffer.cpp
index 52bde46..b3bca2f 100644
--- a/src/Vulkan/VkCommandBuffer.cpp
+++ b/src/Vulkan/VkCommandBuffer.cpp
@@ -251,6 +251,44 @@
const VkBufferImageCopy region;
};
+struct FillBuffer : public CommandBuffer::Command
+{
+ FillBuffer(VkBuffer dstBuffer, VkDeviceSize dstOffset, VkDeviceSize size, uint32_t data) :
+ dstBuffer(dstBuffer), dstOffset(dstOffset), size(size), data(data)
+ {
+ }
+
+ void play(CommandBuffer::ExecutionState& executionState)
+ {
+ Cast(dstBuffer)->fill(dstOffset, size, data);
+ }
+
+private:
+ VkBuffer dstBuffer;
+ VkDeviceSize dstOffset;
+ VkDeviceSize size;
+ uint32_t data;
+};
+
+struct UpdateBuffer : public CommandBuffer::Command
+{
+ UpdateBuffer(VkBuffer dstBuffer, VkDeviceSize dstOffset, VkDeviceSize dataSize, const void* pData) :
+ dstBuffer(dstBuffer), dstOffset(dstOffset), dataSize(dataSize), pData(pData)
+ {
+ }
+
+ void play(CommandBuffer::ExecutionState& executionState)
+ {
+ Cast(dstBuffer)->update(dstOffset, dataSize, pData);
+ }
+
+private:
+ VkBuffer dstBuffer;
+ VkDeviceSize dstOffset;
+ VkDeviceSize dataSize;
+ const void* pData;
+};
+
struct ClearColorImage : public CommandBuffer::Command
{
ClearColorImage(VkImage image, const VkClearColorValue& color, const VkImageSubresourceRange& range) :
@@ -710,12 +748,16 @@
void CommandBuffer::updateBuffer(VkBuffer dstBuffer, VkDeviceSize dstOffset, VkDeviceSize dataSize, const void* pData)
{
- UNIMPLEMENTED();
+ ASSERT(state == RECORDING);
+
+ addCommand<UpdateBuffer>(dstBuffer, dstOffset, dataSize, pData);
}
void CommandBuffer::fillBuffer(VkBuffer dstBuffer, VkDeviceSize dstOffset, VkDeviceSize size, uint32_t data)
{
- UNIMPLEMENTED();
+ ASSERT(state == RECORDING);
+
+ addCommand<FillBuffer>(dstBuffer, dstOffset, size, data);
}
void CommandBuffer::clearColorImage(VkImage image, VkImageLayout imageLayout, const VkClearColorValue* pColor,
