Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Commit 0c25fbe

Browse files
ShabbyXCommit Bot
authored and
Commit Bot
committed
Vulkan: Remove accidental padding in push constants
Bug: chromium:1124379 Change-Id: I45f4511c836858c70d4dae23c7b3954e0bc65ab4 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2391616 Reviewed-by: Shahbaz Youssefi <[email protected]> Commit-Queue: Shahbaz Youssefi <[email protected]>
1 parent 84bd9da commit 0c25fbe

14 files changed

+1684
-1684
lines changed

scripts/code_generation_hashes/Vulkan_internal_shader_programs.json

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -92,25 +92,25 @@
9292
"src/libANGLE/renderer/vulkan/shaders/gen/FullScreenQuad.vert.00000000.inc":
9393
"3a4ab796f02d3f1c306c92f7da2c68ee",
9494
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000000.inc":
95-
"f06dfe4a68246020bc3d2c1132e6bd11",
95+
"4e2e1e59d32aed39be0f0ae9d6c71353",
9696
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000001.inc":
97-
"86b35ef5a4f4058bc59c57157bc2a9e8",
97+
"874ada3b975adcd8cdf2482df035c5c4",
9898
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000002.inc":
99-
"bbf41ecbd4b7065cdc02dd91bed2b1c1",
99+
"b1212f80774bbd58dd18adae991a1ca5",
100100
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000003.inc":
101-
"197553fd8abfce67e5aa5abaf4fac069",
101+
"357a59324a3fd8540e47b0dcfd33cfb3",
102102
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000004.inc":
103-
"6173c8b287c379473195c7f2c184d682",
103+
"7c7f23963a3e5b4928e05616d80fa5d9",
104104
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000005.inc":
105-
"77da7b66604d76872d56d4a7e87d2e92",
105+
"611fe224e3fe61293051ebd65f8faafc",
106106
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000006.inc":
107-
"e151f06ebf603206191017aa902db11c",
107+
"4b6575f8503ff7e81e86b347fd2c96bf",
108108
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000007.inc":
109-
"10565b3877038c05d6dcfeaf31d79a78",
109+
"e5ef164ca3f955ef1f66e00f3072711f",
110110
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000008.inc":
111-
"4503b38c206416b27b2c5efee1229d97",
111+
"98d6d99ad0084ac0e6573fb7ff2ea890",
112112
"src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000009.inc":
113-
"b6eae7691b963374795cacf6659f9a2a",
113+
"a1b48c4da4ba2db858c0af03b0a9666e",
114114
"src/libANGLE/renderer/vulkan/shaders/gen/ImageClear.frag.00000000.inc":
115115
"d5a8184fad9c4a315bc3146ad07d3991",
116116
"src/libANGLE/renderer/vulkan/shaders/gen/ImageClear.frag.00000001.inc":
@@ -256,7 +256,7 @@
256256
"src/libANGLE/renderer/vulkan/shaders/src/FullScreenQuad.vert":
257257
"805ec8b2f87d4bd4242dc5b1c58ba3b4",
258258
"src/libANGLE/renderer/vulkan/shaders/src/GenerateMipmap.comp":
259-
"46ac9e6bbb1b6f2af92dcc85ba52faff",
259+
"ec72e3ce787ca499c684eaa7c96bcc74",
260260
"src/libANGLE/renderer/vulkan/shaders/src/GenerateMipmap.comp.json":
261261
"99543d519f0afe66713498e5d7873871",
262262
"src/libANGLE/renderer/vulkan/shaders/src/ImageClear.frag":

src/libANGLE/renderer/vulkan/UtilsVk.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1889,10 +1889,9 @@ angle::Result UtilsVk::generateMipmap(ContextVk *contextVk,
18891889
const uint32_t workGroupY = UnsignedCeilDivide(srcExtents.height, kPixelWorkgroupRatio);
18901890

18911891
GenerateMipmapShaderParams shaderParams;
1892-
shaderParams.levelCount = params.destLevelCount;
1893-
shaderParams.numWorkGroups = workGroupX * workGroupY;
18941892
shaderParams.invSrcExtent[0] = 1.0f / srcExtents.width;
18951893
shaderParams.invSrcExtent[1] = 1.0f / srcExtents.height;
1894+
shaderParams.levelCount = params.destLevelCount;
18961895

18971896
uint32_t flags = GetGenerateMipmapFlags(contextVk, src->getFormat());
18981897

src/libANGLE/renderer/vulkan/UtilsVk.h

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -384,9 +384,8 @@ class UtilsVk : angle::NonCopyable
384384
struct GenerateMipmapShaderParams
385385
{
386386
// Structure matching PushConstants in GenerateMipmap.comp
387-
uint32_t levelCount = 0;
388-
uint32_t numWorkGroups = 0;
389-
float invSrcExtent[2] = {};
387+
float invSrcExtent[2] = {};
388+
uint32_t levelCount = 0;
390389
};
391390

392391
ANGLE_DISABLE_STRUCT_PADDING_WARNINGS

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000000.inc

Lines changed: 165 additions & 164 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000001.inc

Lines changed: 164 additions & 164 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000002.inc

Lines changed: 168 additions & 168 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000003.inc

Lines changed: 171 additions & 171 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000004.inc

Lines changed: 165 additions & 165 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000005.inc

Lines changed: 165 additions & 165 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000006.inc

Lines changed: 170 additions & 170 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000007.inc

Lines changed: 170 additions & 170 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000008.inc

Lines changed: 165 additions & 164 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/gen/GenerateMipmap.comp.00000009.inc

Lines changed: 165 additions & 165 deletions
Large diffs are not rendered by default.

src/libANGLE/renderer/vulkan/shaders/src/GenerateMipmap.comp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ layout(set = 0, binding = 0, DST_FORMAT) uniform coherent image2D dst[DST_COUNT]
5252
layout(set = 0, binding = 1) uniform sampler2D src;
5353

5454
layout(push_constant) uniform PushConstants {
55-
// Number of levels to generate mipmaps for.
56-
uint levelCount;
5755
// Inverse extents of src image for uv calculation.
5856
vec2 invSrcExtent;
57+
// Number of levels to generate mipmaps for.
58+
uint levelCount;
5959
} params;
6060

6161
#define A_GPU

0 commit comments

Comments
 (0)