From 3a7bb38b70daa67c05be105f97b21f0944223e93 Mon Sep 17 00:00:00 2001 From: Caio Marcelo de Oliveira Filho Date: Wed, 3 Mar 2021 14:50:21 -0800 Subject: [PATCH] spirv: Explicitly break when finished handling SpvDecorationBuiltIn When tyding up this section in 1e5b09f42f6 ("spirv: Tidy some repeated if checks by using a switch statement.") the break got lost. It is not a real problem because the next case just break, but better to have it explicitly here instead of a FALLTHROUGH. Reviewed-by: Jason Ekstrand Part-of: --- src/compiler/spirv/vtn_variables.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index 5144bd43120..bd6eab7ec15 100644 --- a/src/compiler/spirv/vtn_variables.c +++ b/src/compiler/spirv/vtn_variables.c @@ -1200,7 +1200,8 @@ apply_var_decoration(struct vtn_builder *b, default: break; } - FALLTHROUGH; + + break; } case SpvDecorationSpecId: