From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 29828 invoked by alias); 9 Mar 2018 15:54:49 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 29815 invoked by uid 89); 9 Mar 2018 15:54:49 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.2 spammy=Hx-languages-length:1084, million X-HELO: homiemail-a82.g.dreamhost.com Subject: Re: [PATCH 1/6] Remove slow paths from sin/cos To: Wilco Dijkstra , "libc-alpha@sourceware.org" Cc: nd References: From: Siddhesh Poyarekar Message-ID: Date: Fri, 09 Mar 2018 15:54:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2018-03/txt/msg00235.txt.bz2 On Friday 09 March 2018 09:14 PM, Wilco Dijkstra wrote: > This series of patches removes the slow patchs from sin, cos and sincos. > Besides greatly simplifying the implementation, the new version is also much > faster for inputs up to PI (41% faster) and for large inputs needing range > reduction (27% faster). > > ULP is ~0.55 with no errors found after testing 1.6 billion inputs across most > of the range with mpsin and mpcos. The number of incorrectly rounded results > (ie. ULP >0.5) is at most ~2750 per million inputs between 0.125 and 0.5, > the average is ~850 per million between 0 and PI. > > Tested on AArch64 and x86_64 with no regressions. > > The first patch removes the slow paths for the cases where the input is small > and doesn't require range reduction. Update ULP tables for sin, cos and sincos > on AArch64 and x86_64. Please drop the mp benchtests as well (the ##name:768bits namespaces). A separate pre-approved patch to simply drop that name line is sufficient once these slow paths are removed. Siddhesh