Require 16-byte alignment for Vulkan objects
At least for large formats, we assume 16 byte alignment for render
target writes. Requiring 16 bytes in general seems a good idea for
x86_64.
Bug: b/127721996
Change-Id: Ie4d2d9bf71fb8a3d8b58f099785e75a72ff0e181
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/26448
Tested-by: Chris Forbes <chrisforbes@google.com>
Presubmit-Ready: Chris Forbes <chrisforbes@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>
Reviewed-by: Nicolas Capens <nicolascapens@google.com>
Reviewed-by: Alexis Hétu <sugoi@google.com>
Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
diff --git a/src/Vulkan/VkConfig.h b/src/Vulkan/VkConfig.h
index a5f5616..84c9d4b 100644
--- a/src/Vulkan/VkConfig.h
+++ b/src/Vulkan/VkConfig.h
@@ -37,7 +37,7 @@
enum
{
- REQUIRED_MEMORY_ALIGNMENT = 8, // For 64 bit formats on ARM64
+ REQUIRED_MEMORY_ALIGNMENT = 16, // ARM64 will want 8 bytes for 64b formats; x86 wants 16 bytes for 128b formats
MIN_TEXEL_BUFFER_OFFSET_ALIGNMENT = 256,
MIN_UNIFORM_BUFFER_OFFSET_ALIGNMENT = 256,
MIN_STORAGE_BUFFER_OFFSET_ALIGNMENT = 256,