[Mono-devel-list] Mini: cpu-pentium.md

[Paolo Molaro]

On 03/23/04 Willibald Krenn wrote:
> - It says that 'len' is required.
>  Then why e.g. Conv.U4 does not have a length assigned (would be the 
> same as Conv.I4 - namely 2 - AFAIK?)

Not all the opcodes in the .md file are actually used as low-level
instructions, so they may not need a complete description.

> - CEE_JMP's length is - according to the file - 32 Bytes.
[...]
> - CEE_DIV hast a length of 15, but only does a
>            x86_cdq (code);
>            x86_div_reg (code, ins->sreg2, TRUE);
>    which would give a length of  1 + 2 == 3

If the length is bigger than required it's just a possible waste, but
it's no bug. Smaller values than needed is definitely a bug, though it
may not trigger easily, it should be fixed.
Patches welcome.

lupus

-- 
-----------------------------------------------------------------
lupus at debian.org                                     debian/rules
lupus at ximian.com                             Monkeys do it better