From: Ian Lance Taylor <ian@airs.com>
Date: Mon, 16 Dec 1996 21:47:23 +0000 (+0000)
Subject: 	* gencode.c (MIPS16_DECODE): SWRASP is I8, not RI.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=063443cf01d7cffebaccc2a2adfc9c8774e9a91e;p=binutils-gdb.git

	* gencode.c (MIPS16_DECODE): SWRASP is I8, not RI.
	(build_endian_shift): Don't check proc64.
	(build_instruction): Always set memval to uword64.  Cast op2 to
	uword64 when shifting it left in memory instructions.  Always use
	the same code for stores--don't special case proc64.
---

diff --git a/sim/mips/ChangeLog b/sim/mips/ChangeLog
index 5abef5b7728..00923df829e 100644
--- a/sim/mips/ChangeLog
+++ b/sim/mips/ChangeLog
@@ -1,5 +1,11 @@
 Mon Dec 16 14:59:38 1996  Ian Lance Taylor  <ian@cygnus.com>
 
+	* gencode.c (MIPS16_DECODE): SWRASP is I8, not RI.
+	(build_endian_shift): Don't check proc64.
+	(build_instruction): Always set memval to uword64.  Cast op2 to
+	uword64 when shifting it left in memory instructions.  Always use
+	the same code for stores--don't special case proc64.
+
 	* gencode.c (build_mips16_operands): Fix base PC value for PC
 	relative operands.
 	(build_instruction): Call JALDELAYSLOT rather than DELAYSLOT for a
diff --git a/sim/mips/gencode.c b/sim/mips/gencode.c
index c8adaa2ac4b..bf3d0edae77 100644
--- a/sim/mips/gencode.c
+++ b/sim/mips/gencode.c
@@ -639,7 +639,7 @@ static const struct instruction MIPS16_DECODE[] = {
 {"SUBU",    1, "11100xxxyyyddd11",  RRR,     SUB,     WORD | WORD32 },
 {"SW",      1, "11011xxxyyyWWWWW",  RRI,     STORE,   WORD },
 {"SWSP",    1, "11010yyyVVVVVVVVs", RI,      STORE,   WORD },
-{"SWRASP",  1, "01100010VVVVVVVVQs", RI,     STORE,   WORD },
+{"SWRASP",  1, "01100010VVVVVVVVQs", I8,     STORE,   WORD },
 {"XOR",     1, "11101wwwyyy01110",  RR,      XOR,     NONE }
 };
 
@@ -1098,7 +1098,7 @@ build_endian_shift(proc64,datalen,endbit,direction,shift)
      e_endshift direction;
      int shift;
 {
-  if (proc64 && (datalen == 4)) {
+  if (datalen == 4) {
     printf("    if ((vaddr & (1 << %d)) ^ (BigEndianCPU << %d)) {\n",endbit,endbit);
     printf("     memval %s= %d;\n",direction == s_left ? "<<" : ">>",shift);
     printf("    }\n");
@@ -1984,7 +1984,7 @@ build_instruction (doisa, features, mips16, insn)
        printf("     Prefetch(uncached,paddr,vaddr,isDATA,hint);\n");
       else {
 	printf("    {\n");
-	printf("     %s memval;\n",(proc64 ? "uword64" : "unsigned int"));
+	printf("     uword64 memval;\n");
 
 	if ((insn->flags & COPROC) && ((datalen != 4) && (datalen != 8))) {
 	  fprintf(stderr,"Co-processor transfer operation not WORD or DOUBLEWORD in length \"%s\"\n",insn->name);
@@ -2081,7 +2081,7 @@ build_instruction (doisa, features, mips16, insn)
 		  if (proc64 && (datalen == 4))
 		   printf("     GPR[destreg] = SIGNEXTEND(GPR[destreg],32);\n");
 		} else { /* store */
-		  printf("     memval = (op2 << (byte * 8));\n");
+		  printf("     memval = ((uword64) op2 << (byte * 8));\n");
 		  build_endian_shift(proc64,datalen,2,s_left,32);
 		  printf("     StoreMemory(uncached,(%s - byte),memval,paddr,vaddr,isREAL);\n",accesslength);
 		}
@@ -2189,9 +2189,9 @@ build_instruction (doisa, features, mips16, insn)
 
 	      printf("     paddr = ((paddr & ~mask) | ((paddr & mask) ^ (reverse << shift)));\n");
 	      printf("     byte = ((vaddr & mask) ^ (bigend << shift));\n");
-	      printf("     memval = (op2 << (8 * byte));\n");
+	      printf("     memval = ((uword64) op2 << (8 * byte));\n");
 	    } else
-	     if (proc64 && (datalen == 4)) { /* proc64 SC and SW */
+	     if (datalen == 4) { /* SC and SW */
 #if 1 /* see the comments attached to LOADDRMASK above */
 	       printf("     uword64 mask = 0x7;\n");
 #else
@@ -2207,11 +2207,14 @@ build_instruction (doisa, features, mips16, insn)
 			: "((instruction >> 26) & 0x3)"),
 		       ((insn->flags & FP) ? "fs" : "destreg"));
 	       else
-		printf("     memval = (op2 << (8 * byte));\n");
-	     } else { /* !proc64 SC and SW, plus proc64 SD and SCD */
+		printf("     memval = ((uword64) op2 << (8 * byte));\n");
+	     } else { /* SD and SCD */
+	       if (!(insn->flags & COPROC) && ((datalen == 8) || ((datalen == 4) & (insn->flags & UNSIGNED))) && !proc64) {
+		 fprintf(stderr,"Operation not available with 32bit wide memory access \"%s\"\n",insn->name);
+		 exit(4);
+	       }
 	       if (insn->flags & COPROC)
-		printf("     memval = (uword64)COP_S%c(%s,%s);\n",
-		       ((datalen == 8) ? 'D' : 'W'),
+		printf("     memval = (uword64)COP_SD(%s,%s);\n",
 		       ((insn->flags & REG)
 			? "1"
 			: "((instruction >> 26) & 0x3)"),