[Intel-gfx] [PATCH 42/90] assembler: Make struct declared_register use struct brw_reg

Damien Lespiau damien.lespiau at intel.com
Mon Feb 4 16:27:37 CET 2013


It's time to start converting the emission code in gram.y to use libbrw
infrastructure. Let's start with using brw_reg for declared register.

Signed-off-by: Damien Lespiau <damien.lespiau at intel.com>
---
 assembler/gen4asm.h |    4 +++-
 assembler/gram.y    |   46 +++++++++++++++++++++++-----------------------
 2 files changed, 26 insertions(+), 24 deletions(-)

diff --git a/assembler/gen4asm.h b/assembler/gen4asm.h
index 8dd08b7..d81f597 100644
--- a/assembler/gen4asm.h
+++ b/assembler/gen4asm.h
@@ -33,6 +33,8 @@
 #include <stdbool.h>
 #include <assert.h>
 
+#include "brw_reg.h"
+
 typedef unsigned char GLubyte;
 typedef short GLshort;
 typedef unsigned int GLuint;
@@ -226,7 +228,7 @@ extern struct program_defaults program_defaults;
 
 struct declared_register {
     char *name;
-    struct direct_reg base;
+    struct brw_reg reg;
     int element_size;
     struct region src_region;
     int dst_region;
diff --git a/assembler/gram.y b/assembler/gram.y
index 342c66d..7b4cdee 100644
--- a/assembler/gram.y
+++ b/assembler/gram.y
@@ -353,9 +353,9 @@ declare_pragma:	DECLARE_PRAGMA STRING declare_base declare_elementsize declare_s
 			reg = calloc(sizeof(struct declared_register), 1);
 			reg->name = $2;
 		    }
-		    reg->base.reg_file = $3.reg_file;
-		    reg->base.reg_nr = $3.reg_nr;
-		    reg->base.subreg_nr = $3.subreg_nr;
+		    reg->reg.file = $3.reg_file;
+		    reg->reg.nr = $3.reg_nr;
+		    reg->reg.subnr = $3.subreg_nr;
 		    reg->element_size = $4;
 		    reg->src_region = $5;
 		    reg->dst_region = $6;
@@ -1548,9 +1548,9 @@ dst:		dstoperand | dstoperandex
 dstoperand:	symbol_reg dstregion
 		{
 		  memset (&$$, '\0', sizeof ($$));
-		  $$.reg_file = $1.base.reg_file;
-		  $$.reg_nr = $1.base.reg_nr;
-		  $$.subreg_nr = $1.base.subreg_nr;
+		  $$.reg_file = $1.reg.file;
+		  $$.reg_nr = $1.reg.nr;
+		  $$.subreg_nr = $1.reg.subnr;
 		  if ($2 == DEFAULT_DSTREGION) {
 		      $$.horiz_stride = $1.dst_region;
 		  } else {
@@ -1657,7 +1657,7 @@ symbol_reg_p: STRING LPAREN exp RPAREN
 		    }
 
 		    memcpy(&$$, dcl_reg, sizeof(*dcl_reg));
-		    $$.base.reg_nr += $3;
+		    $$.reg.nr += $3;
 		    free($1);
 		}
 		| STRING LPAREN exp COMMA exp RPAREN
@@ -1670,15 +1670,15 @@ symbol_reg_p: STRING LPAREN exp RPAREN
 		    }
 
 		    memcpy(&$$, dcl_reg, sizeof(*dcl_reg));
-		    $$.base.reg_nr += $3;
-		    $$.base.subreg_nr += $5;
+		    $$.reg.nr += $3;
 		    if(advanced_flag) {
-		        $$.base.reg_nr += $$.base.subreg_nr / (32 / get_type_size(dcl_reg->type));
-		        $$.base.subreg_nr = $$.base.subreg_nr % (32 / get_type_size(dcl_reg->type));
+			int size = get_type_size(dcl_reg->type);
+		        $$.reg.nr += ($$.reg.subnr + $5) / (32 / size);
+		        $$.reg.subnr = ($$.reg.subnr + $5) % (32 / size);
 		    } else {
-		        $$.base.reg_nr += $$.base.subreg_nr / 32;
-		        $$.base.subreg_nr = $$.base.subreg_nr % 32;
-			}
+		        $$.reg.nr += ($$.reg.subnr + $5) / 32;
+		        $$.reg.subnr = ($$.reg.subnr + $5) % 32;
+		    }
 		    free($1);
 		}
 ;
@@ -1857,9 +1857,9 @@ srcarchoperandex_typed: flagreg | addrreg | maskreg
 sendleadreg: symbol_reg
              {
 		  memset (&$$, '\0', sizeof ($$));
-		  $$.reg_file = $1.base.reg_file;
-		  $$.reg_nr = $1.base.reg_nr;
-		  $$.subreg_nr = $1.base.subreg_nr;
+		  $$.reg_file = $1.reg.file;
+		  $$.reg_nr = $1.reg.nr;
+		  $$.subreg_nr = $1.reg.subnr;
              }
              | directgenreg | directmsgreg
 ;
@@ -1871,9 +1871,9 @@ directsrcoperand:	negate abs symbol_reg region regtype
 		{
 		  memset (&$$, '\0', sizeof ($$));
 		  $$.address_mode = BRW_ADDRESS_DIRECT;
-		  $$.reg_file = $3.base.reg_file;
-		  $$.reg_nr = $3.base.reg_nr;
-		  $$.subreg_nr = $3.base.subreg_nr;
+		  $$.reg_file = $3.reg.file;
+		  $$.reg_nr = $3.reg.nr;
+		  $$.subreg_nr = $3.reg.subnr;
 		  if ($5.is_default) {
 		    $$.reg_type = $3.type;
 		  } else {
@@ -2303,9 +2303,9 @@ relativelocation2:
 		{
 		  memset (&$$, '\0', sizeof ($$));
 		  $$.address_mode = BRW_ADDRESS_DIRECT;
-		  $$.reg_file = $1.base.reg_file;
-		  $$.reg_nr = $1.base.reg_nr;
-		  $$.subreg_nr = $1.base.subreg_nr;
+		  $$.reg_file = $1.reg.file;
+		  $$.reg_nr = $1.reg.nr;
+		  $$.subreg_nr = $1.reg.subnr;
 		  $$.reg_type = $1.type;
 		  $$.vert_stride = $1.src_region.vert_stride;
 		  $$.width = $1.src_region.width;
-- 
1.7.7.5




More information about the Intel-gfx mailing list