[llvm-commits] Support for fld (%edx) alias for flds

Chris Lattner clattner at apple.com
Mon Feb 21 16:18:35 PST 2011


On Feb 21, 2011, at 1:15 PM, Joerg Sonnenberger wrote:

> Hi all,
> Bug#8687 is about recognising
> 	fld	(%edx)
> as alias for
> 	flds	(%edx)
> like GNU as does. The attached patch does that and also adds the usual
> regression tests.

Hi Joerg,

This is a case where we are intentionally differing from GAS, since GAS's behavior here is nonsensical.

GAS also accepts things like "add $4, (%rax)".  Randomly picking a size for an ambiguous instruction is not a good idea when it affects the semantics of the instruction.

-Chris



More information about the llvm-commits mailing list