I only see one remaining namespace glitch:

On Sun, Apr 17, 2011 at 4:11 AM, Martynas Venckus
<[email protected]> wrote:
...
> +/*
> + * fenv_t represents the entire floating-point environment
> + */
> +typedef struct {
> +       struct {
> +               unsigned int __control;         /* Control word register */
> +               unsigned int __status;          /* Status word register */
> +               unsigned int __tag;             /* Tag word register */
> +               unsigned int __others[4];       /* EIP, Pointer Selector,
etc */
> +       } x87;
> +       unsigned int __mxcsr;                   /* Control, status register
*/
> +} fenv_t;

The 'x87' member should be something like '__x87', with the matching
tweak to feclearexcept(), fesetexceptflag(), fegetenv(), fesetenv(),
and feupdateenv().

I haven't worked through all the bit-twiddling and asm, but what I
have looks good and the abundance of regress tests that keep showing
up in "cvs up -d" inspires confidence.  With that x87-->__x87 nit, ok
guenther@


Philip Guenther

Reply via email to