> binutils-2.15 has this in include/opcodes/hppa.h:
> { "fdc",        0x04001280, 0xfc003fdf, "cZx(s,b)", pa10, 0},
> { "fdc",        0x04001280, 0xfc003fdf, "cZx(b)", pa10, 0},
> { "fic",        0x04000280, 0xfc001fdf, "cZx(S,b)", pa10, 0},
> { "fic",        0x04000280, 0xfc001fdf, "cZx(b)", pa10, 0},

The last "fic" opcode entry is wrong.  It's using the wrong
instruction format, the mask is wrong, pa10 is wrong, etc.

> And the binutils_2.16.1cvs20050902 version has only:
> { "fdc",        0x04001280, 0xfc00ffdf, "cZx(b)", pa10, 0},
> { "fdc",        0x04001280, 0xfc003fdf, "cZx(s,b)", pa10, 0},
> { "fic",        0x04000280, 0xfc001fdf, "cZx(S,b)", pa10, 0},

Yes, I deleted the entry because there isn't implicit space
register selection for 'S' instructions.

> The order of the with 's' vs without 's' was switched and
> in the process dropped one "fic" line.

Changing the order of with 's' vs without 's' was intentional.
It generates better disassembly.  Give the new objdump a try
and you will see the difference.

> Can binutils maintainer/folks please just add it back
> in the new location?

We need a new pa20 format 24 entry.

> Is the fix obvious enough or is a patch necessary?

A patch is needed.

Dave
-- 
J. David Anglin                                  [EMAIL PROTECTED]
National Research Council of Canada              (613) 990-0752 (FAX: 952-6602)


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to