Mesa (master): i965: Add missing break in SHADER_OPCODE_GEN7_SCRATCH_READ case.

Vinson Lee vlee at kemper.freedesktop.org
Sat Nov 16 02:43:03 UTC 2013


Module: Mesa
Branch: master
Commit: b570c4229fe9c621b56bb9475d77a344039444d4
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=b570c4229fe9c621b56bb9475d77a344039444d4

Author: Vinson Lee <vlee at freedesktop.org>
Date:   Thu Nov 14 22:47:33 2013 -0800

i965: Add missing break in SHADER_OPCODE_GEN7_SCRATCH_READ case.

Fixes "Missing break in switch" defect reported by Coverity.

Signed-off-by: Vinson Lee <vlee at freedesktop.org>
Reviewed-by: Matt Turner <mattst88 at gmail.com>
Reviewed-by: Eric Anholt <eric at anholt.net>
Cc: "10.0" <mesa-stable at lists.freedesktop.org>

---

 .../drivers/dri/i965/brw_schedule_instructions.cpp |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
index 39a5266..a4fae0d 100644
--- a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
+++ b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
@@ -335,6 +335,8 @@ schedule_node::set_latency_gen7(bool is_haswell)
        * then around 140.  Presumably this is cache hit vs miss.
        */
       latency = 50;
+      break;
+
    case SHADER_OPCODE_UNTYPED_ATOMIC:
       /* Test code:
        *   mov(8)    g112<1>ud       0x00000000ud       { align1 WE_all 1Q };




More information about the mesa-commit mailing list