bugZ80 assembler - Bugs: bug #51332, Wrong opcode generated for...

 
 

bug #51332: Wrong opcode generated for "add a,ixh"

Submitter:  Tomaž Šolc <avian>
Submitted:  Wed 28 Jun 2017 11:36:24 AM UTC
Votes: 1
 
Category:  None Severity:  3 - Normal
Item Group:  None Status:  Fixed
Privacy:  Public Assigned to:  None
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 17 Mar 2021 07:17:24 PM UTC, comment #5: 

Thanks Tomaž.

Paul Flo Williams <hisdeedsaredust>
Group Member
Wed 17 Mar 2021 06:32:48 PM UTC, comment #4: 

Hi Paul,

I can confirm that as of git commit 0df4a465 "add a,ixh" generates the correct opcode. Thanks!

Best regards
Tomaž

Tomaž Šolc <avian>
Tue 16 Mar 2021 04:42:49 PM UTC, comment #3: 

Fix available for testing.

Paul Flo Williams <hisdeedsaredust>
Group Member
Tue 16 Mar 2021 04:16:30 PM UTC, comment #2: 

Confirmed that ixh/ixl opcodes are swapped for most ops.

Fixed coming today, for this bug and bug #55596.

Paul Flo Williams <hisdeedsaredust>
Group Member
Sat 09 Jan 2021 11:04:05 PM UTC, comment #1: 

It also fails with ld a,ixl and ld a,ixh , but with other registers it works.

Sergio Costas <rastersoft>
Wed 28 Jun 2017 11:36:24 AM UTC, original submission:  

Hi,

z80asm generates a wrong opcode for "add a,ixh" (and possibly other undocumented instructions involving IXH and IXL).

How to reproduce (using latest code from git, commit f49176a):

$ cat test.asm
add a,ixh
$ src/z80asm -o test.bin test.asm
$ hd test.bin
00000000  dd 85

Generated opcode should be "dd 84" (opcode for "add a,h" with "dd" prefix). See:

http://www.z80.info/z80undoc.htm

If this project is still maintained, I can make a patch for this bug, like I recently did for "inc ixh".

Best regards
Tomaž

Tomaž Šolc <avian>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by hisdeedsaredust (Posted a comment)
  • -email is unavailable- added by asgalon (Updated the item)
  • -email is unavailable- added by rastersoft (Posted a comment)
  • -email is unavailable- added by rastersoft (Voted in favor of this item)
  • -email is unavailable- added by avian (Submitted the item)
  •  

    There is 1 vote so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-03-17 hisdeedsaredust StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2021-03-16 hisdeedsaredust StatusConfirmed Ready For Test
        Assigned tohisdeedsaredust None
    2021-03-16 hisdeedsaredust Assigned toNone hisdeedsaredust
    2021-03-13 asgalon StatusNone Confirmed
    2021-01-09 rastersoft Carbon-Copy- Added rastersoft

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code