This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 2/4] Check n instead of k1 to decide on sign of sin/cos result
- From: Siddhesh Poyarekar <siddhesh at sourceware dot org>
- To: Carlos O'Donell <carlos at redhat dot com>, libc-alpha at sourceware dot org
- Date: Thu, 6 Oct 2016 00:37:57 +0530
- Subject: Re: [PATCH 2/4] Check n instead of k1 to decide on sign of sin/cos result
- Authentication-results: sourceware.org; auth=none
- References: <1474998553-2366-1-git-send-email-siddhesh@sourceware.org> <1474998553-2366-3-git-send-email-siddhesh@sourceware.org> <dd631de5-d7b3-41b7-434a-3273bcab1f1e@redhat.com> <6e9f05db-6856-f760-c47c-322e5c141246@sourceware.org> <9150bada-c418-2249-c6d4-b26845b1bfef@redhat.com>
- Reply-to: siddhesh at sourceware dot org
On Friday 30 September 2016 06:07 AM, Carlos O'Donell wrote:
> I like where you are going with this.
>
> The simplest solution I think is:
>
> * Make K a bool (C99 bool type), which should resolve the issue.
> The compiler will always only use 0 or 1 and the result works.
Ack, I made that a separate patch and pushed it (see other [committed]
email)
> * Update with the description above.
Done and pushed.
Thanks,
Siddhesh