This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix for ldm/stm instructions in H8S


Hi Asgari,

> 2003-11-05 Asgari Jinia <asgarij@kpitcummins.com>
> 	* config/tc-h8300.c (md_assemble) : Check operands validity for ldm/stm.
> 	(get_operand) : Check register pair's validity as per tech note TN-H8*-193A/E from Renesas.

What about a simpler test that checks all possibilities of low and
high (see attached)?  Someone reading the code does not have to
decipher it.

By the way, I fixed minor things like formatting.  Next time you
contribute something, please follow the GNU Coding Standards,
available at:

  http://www.gnu.org/prep/standards_toc.html

I assume you actually fed some invalid code to get these error
messages.  The patch looks good.  I leave the approval up to other
people.

Otherwise

Kazu Hirata

2003-11-05  Asgari Jinia  <asgarij@kpitcummins.com>

	* config/tc-h8300.c (md_assemble): Check operands validity for
	ldm/stm.
	(get_operand): Check register pair's validity as per technical
	note TN-H8*-193A/E from Renesas.

Index: tc-h8300.c
===================================================================
RCS file: /cvs/src/src/gas/config/tc-h8300.c,v
retrieving revision 1.37
diff -c -r1.37 tc-h8300.c
*** tc-h8300.c	17 Oct 2003 10:23:33 -0000	1.37
--- tc-h8300.c	5 Nov 2003 22:36:46 -0000
***************
*** 622,635 ****
        low = src[2] - '0';
        high = src[6] - '0';
  
!       if (high == low)
! 	as_bad (_("Invalid register list for ldm/stm\n"));
! 
!       if (high < low)
! 	as_bad (_("Invalid register list for ldm/stm\n"));
! 
!       if (high - low > 3)
! 	as_bad (_("Invalid register list for ldm/stm)\n"));
  
        /* Even sicker.  We encode two registers into op->reg.  One
  	 for the low register to save, the other for the high
--- 622,635 ----
        low = src[2] - '0';
        high = src[6] - '0';
  
!       /* Check register pair's validity as per tech note TN-H8*-193A/E
! 	 from Renesas.  */
!       if (!(low == 0 && (high == 1 || high == 2 || high == 3))
! 	  && !(low == 2 && high == 3)
! 	  && !(low == 4 && (high == 5 || high == 6)))
! 	{
! 	  as_bad (_("Invalid register list for ldm/stm\n"));
! 	}
  
        /* Even sicker.  We encode two registers into op->reg.  One
  	 for the low register to save, the other for the high
***************
*** 1965,1970 ****
--- 1965,1995 ----
    *op_end = c;
    prev_instruction = instruction;
  
+    /* Now we have operands from instuction.  Let's check them out for
+       ldm and stm.  */
+    if (OP_KIND (instruction->opcode->how) == O_LDM)
+      {
+        /* The first operand must be @er7+, and the second operand must
+ 	  be a register pair.  */
+        if ((operand[0].mode != RSINC)
+ 	   || (operand[0].reg != 7)
+ 	   || ((operand[1].reg & 0x80000000) == 0))
+          {
+            as_bad (_("invalid operand in ldm"));
+          }
+      }
+  
+    if (OP_KIND (instruction->opcode->how) == O_STM)
+      {
+        /* The first operand must be a register pair, and the second
+ 	  operand must be @-er7.  */
+        if (((operand[0].reg & 0x80000000) == 0)
+ 	   || (operand[1].mode != RDDEC) || (operand[1].reg != 7))
+          {
+            as_bad (_("invalid operand in stm"));
+          }
+      }
+  
    size = SN;
    if (dot)
      {


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]