From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 131012 invoked by alias); 19 Jul 2018 09:15:41 -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 129940 invoked by uid 89); 19 Jul 2018 09:15:39 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: EUR01-DB5-obe.outbound.protection.outlook.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector1-arm-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=CEwUPVuuGPNI6Ol/90MmDiyRYlqz7TnQ3opa8M65ArM=; b=eBobnK3XVIRLf14mcXxczhCKjYMhbHnGH0ZN560GjDmytaLGAqCyDyVk1uBISBZcSIESBCgjOGDUORaU9rBEvc3yTkw7NVB09mUs4Qj5AsnfCVj54tzORa1P+5I9xDHmOSCOny5WdgnaGE4qtBlK0VV/EbpfeCKm+o/2gyEHkm0= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Szabolcs.Nagy@arm.com; Cc: nd@arm.com, GNU C Library , Wilco Dijkstra Subject: Re: [PATCH] update libm-test-ulps for new exp To: Joseph Myers References: <4c570272-db39-aa9d-664b-b6e32402f448@arm.com> From: Szabolcs Nagy Message-ID: <62ddbf0a-15be-bafa-6643-70402c872ac2@arm.com> Date: Thu, 19 Jul 2018 09:15:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-Path: szabolcs.nagy@arm.com Received-SPF: None (protection.outlook.com: arm.com does not designate permitted sender hosts) X-SW-Source: 2018-07/txt/msg00601.txt.bz2 On 18/07/18 18:25, Joseph Myers wrote: > On Thu, 12 Jul 2018, Szabolcs Nagy wrote: > >> Update error bounds for the new exp implementation on alpha, hppa, >> m68k/coldfire, microblaze, mips/mips32, mips/mips64, nios2, powerpc, >> riscv/nofpu, riscv/rv64, s390, sh and sparc. Only non-nearest >> rounding ulp bounds increased. Did not run the math tests on real >> hardware, but used the following ULP bounds to generate the new files: > > This is adding ulps for non-to-nearest tests on some architectures that > only support round-to-nearest and so have no such ulps in their existing > files. > > (I am generically dubious of such an update not based on actually running > tests on the architectures involved - there can be architecture variations > because of differences in e.g. whether multiply and add get fused - but > adding ulps for tests that would not run on those architectures is > definitely wrong.) > i created this patch in case the changes get accepted late so there is no time for maintainers to update the ulp file. i agree that this is not good practice. the numbers are upper bounds so max(fma-ulp,nofma-ulp) was used for all targets. in the long term i'd prefer if there was a generic ulp error file and a target could just say it uses the generic bounds (probably per function), so if a generic implementation is changed, less ulp modifications are needed (targets may still need to test in case of variation in target behaviour but most of the time i'd expect the same bound to work for all, if some target wants its own implementation then the burden of maintaining separate ulp bounds is on that target)