Mesa (master): clover/spirv: Add missing break for SpvOpExecutionMode case.
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Tue Nov 17 00:22:24 UTC 2020
Module: Mesa
Branch: master
Commit: 71ee4e2853bc4af270969e6d91533cba7b9c0cb4
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=71ee4e2853bc4af270969e6d91533cba7b9c0cb4
Author: Vinson Lee <vlee at freedesktop.org>
Date: Mon Nov 9 17:31:36 2020 -0800
clover/spirv: Add missing break for SpvOpExecutionMode case.
Fix defect reported by Coverity Scan.
Missing break in switch (MISSING_BREAK)
unterminated_case: The case for value SpvOpExecutionMode is not
terminated by a 'break' statement.
Fixes: ee5b46fcfdb ("clover/spirv: support CL_KERNEL_COMPILE_WORK_GROUP_SIZE")
Signed-off-by: Vinson Lee <vlee at freedesktop.org>
Reviewed-by: Karol Herbst <kherbst at redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7519>
---
src/gallium/frontends/clover/spirv/invocation.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/frontends/clover/spirv/invocation.cpp b/src/gallium/frontends/clover/spirv/invocation.cpp
index 27fa06db46c..a755df18055 100644
--- a/src/gallium/frontends/clover/spirv/invocation.cpp
+++ b/src/gallium/frontends/clover/spirv/invocation.cpp
@@ -199,6 +199,7 @@ namespace {
default:
break;
}
+ break;
case SpvOpDecorate: {
const auto id = get<SpvId>(inst, 1);
More information about the mesa-commit
mailing list