diff mbox

powerpc - Fix sinf() IFUNC fallback.

Message ID 1498135262-4411-1-git-send-email-rcardoso@linux.vnet.ibm.com
State Committed
Delegated to: Tulio Magno Quites Machado Filho
Headers

Commit Message

rcardoso@linux.vnet.ibm.com June 22, 2017, 12:41 p.m. UTC 
  sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c should fall back to
sysdeps/powerpc/fpu/s_sinf.c not to sysdeps/ieee754/flt-32/s_sinf.c.

2017-06-22  Rogerio A. Cardoso  <rcardoso@linux.vnet.ibm.com>

	* sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c: Change s_sinf.c
	from sysdeps/ieee754/flt-32/ to sysdeps/powerpc/fpu/.

---
 sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Paul A. Clarke June 22, 2017, 1:03 p.m. UTC | #1 
On 06/22/2017 07:41 AM, rcardoso@linux.vnet.ibm.com wrote:
> sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c should fall back to
> sysdeps/powerpc/fpu/s_sinf.c not to sysdeps/ieee754/flt-32/s_sinf.c.
> 
> 2017-06-22  Rogerio A. Cardoso  <rcardoso@linux.vnet.ibm.com>
> 
> 	* sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c: Change s_sinf.c
> 	from sysdeps/ieee754/flt-32/ to sysdeps/powerpc/fpu/.
> 
> ---
>  sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c b/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
> index 83e37f9..b2ff032 100644
> --- a/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
> +++ b/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
> @@ -23,4 +23,4 @@
> 
>  #define __sinf __sinf_ppc64
> 
> -#include <sysdeps/ieee754/flt-32/s_sinf.c>
> +#include <sysdeps/powerpc/fpu/s_sinf.c>

LGTM.

PC
Tulio Magno Quites Machado Filho June 23, 2017, 12:39 p.m. UTC | #2 
rcardoso@linux.vnet.ibm.com writes:

> [ text/plain ]
> sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c should fall back to
> sysdeps/powerpc/fpu/s_sinf.c not to sysdeps/ieee754/flt-32/s_sinf.c.
>
> 2017-06-22  Rogerio A. Cardoso  <rcardoso@linux.vnet.ibm.com>
>
> 	* sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c: Change s_sinf.c
> 	from sysdeps/ieee754/flt-32/ to sysdeps/powerpc/fpu/.
>
> ---
>  sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c b/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
> index 83e37f9..b2ff032 100644
> --- a/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
> +++ b/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
> @@ -23,4 +23,4 @@
>
>  #define __sinf __sinf_ppc64
>
> -#include <sysdeps/ieee754/flt-32/s_sinf.c>
> +#include <sysdeps/powerpc/fpu/s_sinf.c>
> -- 
> 2.7.4
>

Looks good to me.  Pushed as 76c4c838e746.
diff mbox

Patch

diff --git a/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c b/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
index 83e37f9..b2ff032 100644
--- a/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
+++ b/sysdeps/powerpc/powerpc64/fpu/multiarch/s_sinf-ppc64.c
@@ -23,4 +23,4 @@ 
 
 #define __sinf __sinf_ppc64
 
-#include <sysdeps/ieee754/flt-32/s_sinf.c>
+#include <sysdeps/powerpc/fpu/s_sinf.c>