From: Matt Turner <mattst88@gmail.com>
Date: Mon, 7 Nov 2016 05:20:16 +0000 (-0800)
Subject: i965: Factor out sources_not_null() validation function.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=7abc65dd7c2856b71c1db468af1cc6b6889ad293;p=mesa.git

i965: Factor out sources_not_null() validation function.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
---

diff --git a/src/mesa/drivers/dri/i965/brw_eu_validate.c b/src/mesa/drivers/dri/i965/brw_eu_validate.c
index d3c15da8c55..4874adda277 100644
--- a/src/mesa/drivers/dri/i965/brw_eu_validate.c
+++ b/src/mesa/drivers/dri/i965/brw_eu_validate.c
@@ -132,6 +132,28 @@ num_sources_from_inst(const struct gen_device_info *devinfo,
    }
 }
 
+static struct string
+sources_not_null(const struct gen_device_info *devinfo,
+                 const brw_inst *inst)
+{
+   unsigned num_sources = num_sources_from_inst(devinfo, inst);
+   struct string error_msg = { .str = NULL, .len = 0 };
+
+   /* Nothing to test. 3-src instructions can only have GRF sources, and
+    * there's no bit to control the file.
+    */
+   if (num_sources == 3)
+      return (struct string){};
+
+   if (num_sources >= 1)
+      ERROR_IF(src0_is_null(devinfo, inst), "src0 is null");
+
+   if (num_sources == 2)
+      ERROR_IF(src1_is_null(devinfo, inst), "src1 is null");
+
+   return error_msg;
+}
+
 static bool
 is_unsupported_inst(const struct gen_device_info *devinfo,
                     const brw_inst *inst)
@@ -152,26 +174,10 @@ brw_validate_instructions(const struct brw_codegen *p, int start_offset,
       struct string error_msg = { .str = NULL, .len = 0 };
       const brw_inst *inst = store + src_offset;
 
-      switch (num_sources_from_inst(devinfo, inst)) {
-      case 3:
-         /* Nothing to test. 3-src instructions can only have GRF sources, and
-          * there's no bit to control the file.
-          */
-         break;
-      case 2:
-         ERROR_IF(src1_is_null(devinfo, inst), "src1 is null");
-         /* fallthrough */
-      case 1:
-         ERROR_IF(src0_is_null(devinfo, inst), "src0 is null");
-         break;
-      case 0:
-      default:
-         break;
-      }
-
       if (is_unsupported_inst(devinfo, inst)) {
          ERROR("Instruction not supported on this Gen");
       } else {
+         CHECK(sources_not_null);
       }
 
       if (brw_inst_opcode(devinfo, inst) == BRW_OPCODE_SEND) {