[Mono-list] JIT patch for stabs debugging

Paolo Molaro lupus@ximian.com
Mon, 11 Mar 2002 10:32:46 +0100


On 03/10/02 Serge wrote:
> > but x86_mov_reg_imm() already emits a XOR if the argument is zer
> 
> BTW, I also noticed this optimization some time ago, and maybe it would be
> better to remove it?
> This may lead to subtle bugs because the XOR has effect on flags.
> For example if emitting ADD/ADC sequence for 64-bit values.
> Sure such bugs are easy to spot, but maybe it's better to make it explicit.

I think I put the optimization there only because the ORP sources did
that as well and then forgot about it:-)
I don't think it's likely to cause bugs, though, it's dietmar call if he
prefers one version or the other.

lupus

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