Nicolas Joly
2011-09-20 07:03:35 UTC
Hi,
While checking some generated i386 assembly, i noticed a small part
which was not properly indented.
[...]
fldl .L382
fxch
fucomip %st(1),%st
fstp %st(0)
jp .L380
jnc .L380
[...]
I tracked it to the fcomp() function which use 2 spaces instead of a
tab to output asm directives.
The attached patch make use of `\t' for printf and friends to ensure
correct indentation. And, while here, i modified the amd64 version for
consistency.
Hope this helps,
Regards.
While checking some generated i386 assembly, i noticed a small part
which was not properly indented.
[...]
fldl .L382
fxch
fucomip %st(1),%st
fstp %st(0)
jp .L380
jnc .L380
[...]
I tracked it to the fcomp() function which use 2 spaces instead of a
tab to output asm directives.
The attached patch make use of `\t' for printf and friends to ensure
correct indentation. And, while here, i modified the amd64 version for
consistency.
Hope this helps,
Regards.
--
Nicolas Joly
Projects and Developments in Bioinformatics
Institut Pasteur, Paris.
Nicolas Joly
Projects and Developments in Bioinformatics
Institut Pasteur, Paris.