------- Additional Comments From chris at seberino dot org  2009-06-30 20:22 
-------
I couldn't apply tc-arm.c.patch but it looked like you applied it already to 
binutils-2.19.51.tar.bz2.  So these comments pertain to your June 30th 
2.19.51...

I'll mention 2 things in this paragraph that if you don't want to fix I'm ok
with..There were still some inconsistencies with some hex equivalents that were
missing.  Also, some undefineds that were "undefined instuction 0x...." and
others that were "undefined".   (If you just want to cut out hex equivalents
that ok by me.  Also, objdump already shows the instruction and address so if
you want to just make all undefined's just print "undefined" that is ok with me
too.)

More importantly, it looks like you reintroduced DSP instructions and
instructions for later CPU architectures that don't belong in ARM7TDMI like...
mrrc, blx and ldc2.

There was also some unidentifiable instructions I didn't know what to make of
like these...
usat, movw, vdupcs.8 and fldmdbxmi.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=10288

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils

Reply via email to