PATCH: Use short for fnstsw

H.J. Lu hjl@lucon.org
Mon Jan 14 14:04:00 GMT 2008


I am checking this x86 assembler patch:

http://sourceware.org/ml/binutils/2008-01/msg00148.html

to check operand size. fnstsw stores 16bit into %ax. The upper
16bit of %eax is unchanged.  The new assembler will disallow
"fnstsw %eax".  Here is a patch for glibc.


H.J.
---
2008-01-12  H.J. Lu  <hongjiu.lu@intel.com>

	* sysdeps/i386/fpu/ftestexcept.c (fetestexcept): Use short
	for fnstsw.

--- sysdeps/i386/fpu/ftestexcept.c.foo	2004-03-05 02:14:48.000000000 -0800
+++ sysdeps/i386/fpu/ftestexcept.c	2008-01-12 07:36:22.000000000 -0800
@@ -26,7 +26,7 @@
 int
 fetestexcept (int excepts)
 {
-  int temp;
+  short temp;
   int xtemp = 0;
 
   /* Get current exceptions.  */



More information about the Binutils mailing list