This is the mail archive of the glibc-bugs@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]

[Bug math/15384] One constant fewer in ieee754/dbl-64/wordsize-64/s_finite.c


http://sourceware.org/bugzilla/show_bug.cgi?id=15384

--- Comment #1 from Ondrej Bilka <neleai at seznam dot cz> 2013-04-21 10:54:14 UTC ---
On Sun, Apr 21, 2013 at 10:06:00AM +0000, marc.glisse at normalesup dot org
wrote:
> http://sourceware.org/bugzilla/show_bug.cgi?id=15384
> 
>              Bug #: 15384
>            Summary: One constant fewer in
>                     ieee754/dbl-64/wordsize-64/s_finite.c
>            Product: glibc
>            Version: unspecified
>             Status: NEW
>           Severity: enhancement
>           Priority: P2
>          Component: math
>         AssignedTo: unassigned@sourceware.org
>         ReportedBy: marc.glisse@normalesup.org
>     Classification: Unclassified
> 
> 
> Hello,
> 
> this is very minor, and I don't even know what platforms use this file, but in
> ieee754/dbl-64/wordsize-64/s_finite.c:
> 
> int
> __finite(double x)
> {
>   int64_t lx;
>   EXTRACT_WORDS64(lx,x);
>   return
> (int)((uint64_t)((lx&INT64_C(0x7fffffffffffffff))-INT64_C(0x7ff0000000000000))>>63);
> }
>
It could be simplified more. Constants are  not necessary here. I will
post patch to libc-alpha. 
> The first constant 0x7fffffffffffffff is unnecessary, it would work just as
> well with 0x7ff0000000000000, so there is only one constant loaded. The same
> applies to the 32 bit version, but for platforms like x86 that embed 32 bit
> constants in the instruction it probably wouldn't matter.
> 
> -- 
> Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.


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