This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 6/7] i386/x86_64: Optimize feholdexcept.


On Monday, March 19, 2012 00:46:31 Richard Henderson wrote:
> 	* sysdeps/i386/fpu/feholdexcpt.c (feholdexcept): Avoid the fldenv.
> 	* sysdeps/x86_64/fpu/feholdexcpt.c (feholdexcept): Likewise.
> ---
>  sysdeps/i386/fpu/feholdexcpt.c   |   17 ++++-------------
>  sysdeps/x86_64/fpu/feholdexcpt.c |   21 +++++++--------------
>  2 files changed, 11 insertions(+), 27 deletions(-)
> 
> diff --git a/sysdeps/i386/fpu/feholdexcpt.c
> b/sysdeps/i386/fpu/feholdexcpt.c index a09d45e..49f7ced 100644
> --- a/sysdeps/i386/fpu/feholdexcpt.c
> +++ b/sysdeps/i386/fpu/feholdexcpt.c
> @@ -1,6 +1,5 @@
>  /* Store current floating-point environment and clear exceptions.
> -   Copyright (C) 1997, 1999, 2003, 2004, 2005, 2007
> -   Free Software Foundation, Inc.
> +   Copyright (C) 1997-2012 Free Software Foundation, Inc.
>     This file is part of the GNU C Library.
>     Contributed by Ulrich Drepper <drepper@cygnus.com>, 1997.
> 
> @@ -28,17 +27,9 @@ feholdexcept (fenv_t *envp)
>  {
>    fenv_t temp;

Please remove temp like you do in the x86-64 version, it's unused now.

Andreas
-- 
 Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi
  SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
   GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)
    GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]