From cb026c5d599dddc38f34ee93438d52bbffe2f6ad Mon Sep 17 00:00:00 2001 From: Robin Voetter Date: Tue, 21 Nov 2023 19:58:29 +0100 Subject: [PATCH] spirv: always emit mask constants even if no bits are set A parameter like this is not always optional, even if that is usually implied. SPIR-V tools fail to parse a module with an OpLoopMerge instruction where the loop control parameter is left out. --- src/codegen/spirv/Section.zig | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/codegen/spirv/Section.zig b/src/codegen/spirv/Section.zig index ae88dc7c8a..f569512fdd 100644 --- a/src/codegen/spirv/Section.zig +++ b/src/codegen/spirv/Section.zig @@ -198,10 +198,6 @@ fn writeExtendedMask(section: *Section, comptime Operand: type, operand: Operand } } - if (mask == 0) { - return; - } - section.writeWord(mask); inline for (@typeInfo(Operand).Struct.fields) |field| { @@ -304,9 +300,6 @@ fn extendedMaskSize(comptime Operand: type, operand: Operand) usize { else => unreachable, } } - if (!any_set) { - return 0; - } return total + 1; // Add one for the mask itself. }