Skip to content

Commit 96a64b1

Browse files
committed
Update on "[EK-VT] Replacing the use of uvec3 with WorkgroupSize class to reduce memory usage and improve processing speed"
This diff replaces the use of `uvec3` with `WorkgroupSize` class to reduce memory usage and improve processing speed in the Vulkan backend of Executorch. Differential Revision: [D70021032](https://our.internmc.facebook.com/intern/diff/D70021032/) [ghstack-poisoned]
2 parents a0aa663 + 60f2269 commit 96a64b1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

backends/vulkan/runtime/utils/VecUtils.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -499,7 +499,7 @@ class WorkgroupSize final {
499499
// shift numbers by multiple of 11 bits, since each local workgroup axis can
500500
// be 1024 at most and which is 0x400. only z axis can't store 1024, because
501501
// it would overflow uint32_t storage.
502-
if (z == 1024) {
502+
if (vec[2u] == 1024) {
503503
throw std::runtime_error(
504504
"Workgroup size in z axis cannot be 1024 because it would overflow uint32_t storage");
505505
}

0 commit comments

Comments
 (0)