This is the mail archive of the binutils@sourceware.cygnus.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]

Re: AM33: SP-based offsets are zero-extended


  In message <orn1nej50a.fsf@zecarneiro.lsd.ic.unicamp.br>you write:
  > --=-=-=
  > 
  > Unlike other registers-based offsets, that are sign-expected, SP-based
  > offsets are zero-extended to 32 bits, so they must be unsigned.
  > Here's a patch that fixes this bug.  Ok to install?
  > 
  > 
  > --=-=-=
  > Content-Type: text/x-patch
  > Content-Disposition: inline; filename=am33-fix-opcodes.patch
  > 
  > Index: opcodes/ChangeLog
  > from  Alexandre Oliva  <aoliva@cygnus.com>
  > 	
  > 	* m10300-opc.c: SP-based offsets are always unsigned.
This is fine.  Can you please install a regression test into the assembler
for this bug?

Sorry I never caught this one.  I had it right for the mn103, but not any
of the am33 extensions.  Ugh.

jeff


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