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