--- Additional Comments From haubi at gentoo dot org 2007-12-07 08:39
---
Created an attachment (id=2126)
-- (http://sourceware.org/bugzilla/attachment.cgi?id=2126action=view)
binary built on ia64-hp-hpux11.23 with gcc-4.1.1
Source file simply contained one line:
int main(void) {
--- Additional Comments From schwab at suse dot de 2007-12-07 23:19 ---
According to the M68000PRM a cache field of 0 is valid for both cpush and cinv.
The change needs to be restricted to the ColdFire.
--
http://sourceware.org/bugzilla/show_bug.cgi?id=5457
--- You are
--- Additional Comments From shap at eros-os dot com 2007-12-07 22:01
---
Created an attachment (id=2127)
-- (http://sourceware.org/bugzilla/attachment.cgi?id=2127action=view)
Fix to mask values for cpush* instructions on m68k target.
--
--- Additional Comments From shap at eros-os dot com 2007-12-08 02:27
---
Hmm. Thank you for pointing that out.
Actually, M68000PRM specifies that if that field is 0 then the operation is a
no-op. While the encoding is hypothetically legal, the definition of the
assembly language
The intouch instruction is incorrectly disassembled as cpushl on platforms
supporting mcfisa_b or mcfisa_c. A patch correcting this is attached.
Repeat by compiling an assembly file containing use of an intouch instruction:
intouch %a0
then attempt to disassemble. The disassembly will report:
