[PATCH] Fix i386 fenv support

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] Fix i386 fenv support

Joel Sherrill
In removing the symbolic links, I accidentally ended up with the
i386 not having an fenv.h implementation. All the files linked to
the stub.

This patch addresses this.

--joel

Joel Sherrill (1):
  i386/fenv.c: Include fenv.c implementation shared with x86_64, not
    stub

 newlib/libm/machine/i386/fenv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
1.8.3.1

Reply | Threaded
Open this post in threaded view
|

[PATCH] i386/fenv.c: Include fenv.c implementation shared with x86_64, not stub

Joel Sherrill
---
 newlib/libm/machine/i386/fenv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/newlib/libm/machine/i386/fenv.c b/newlib/libm/machine/i386/fenv.c
index 8cbee77..1a97f56 100644
--- a/newlib/libm/machine/i386/fenv.c
+++ b/newlib/libm/machine/i386/fenv.c
@@ -4,4 +4,4 @@
  * (c) Copyright 2019 Joel Sherrill <[hidden email]>
  */
 
-#include "../../fenv/fenv_stub.c"
+#include "../x86_64/fenv.c"
--
1.8.3.1

Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] i386/fenv.c: Include fenv.c implementation shared with x86_64, not stub

Corinna Vinschen
On Mar 10 10:03, Joel Sherrill wrote:

> ---
>  newlib/libm/machine/i386/fenv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/newlib/libm/machine/i386/fenv.c b/newlib/libm/machine/i386/fenv.c
> index 8cbee77..1a97f56 100644
> --- a/newlib/libm/machine/i386/fenv.c
> +++ b/newlib/libm/machine/i386/fenv.c
> @@ -4,4 +4,4 @@
>   * (c) Copyright 2019 Joel Sherrill <[hidden email]>
>   */
>  
> -#include "../../fenv/fenv_stub.c"
> +#include "../x86_64/fenv.c"
> --
> 1.8.3.1
Pushed.


Thanks,
Corinna

--
Corinna Vinschen
Cygwin Maintainer
Red Hat

signature.asc (849 bytes) Download Attachment