From patchwork Fri Sep 18 23:13:00 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arjun Shankar X-Patchwork-Id: 40454 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 516623973054; Fri, 18 Sep 2020 23:13:06 +0000 (GMT) X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from aloka.lostca.se (aloka.lostca.se [IPv6:2a01:4f8:120:624c::2]) by sourceware.org (Postfix) with ESMTPS id 8DE1F397304A for ; Fri, 18 Sep 2020 23:13:03 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 8DE1F397304A Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=lostca.se Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=arjun.is@lostca.se Received: from aloka.lostca.se (aloka [127.0.0.1]) by aloka.lostca.se (Postfix) with ESMTP id 2AD4D17B2D; Fri, 18 Sep 2020 23:13:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=lostca.se; h=date:from:to :cc:subject:message-id:mime-version:content-type; s=howrah; bh=V CMSkzkvR8opbij4v0wo/FLr3hA=; b=1T3CH9YlI2i9aFQ21qh+b/X7MuvaMmMyf cGgcETNxTQXqbT+6jRpsL/UeqP8QIFm7pUanfkSiB8rnal9iuSG4bl8/1EPIsDyT dYuYmTTLiKSh8fu22zNy93p8m89eD8gEju5IIpZiPWfoi4iw2gFVRtN3e0NJBb9J IqM/DXpkqw= Received: from localhost (unknown [IPv6:2a01:4f8:120:624c::25]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aloka.lostca.se (Postfix) with ESMTPSA id E416617B2C; Fri, 18 Sep 2020 23:13:01 +0000 (UTC) Date: Fri, 18 Sep 2020 23:13:00 +0000 From: Arjun Shankar To: libc-alpha@sourceware.org Subject: [PATCH v2] benchtests: Run _Float128 tests only on architectures that support it Message-ID: <20200918231259.GA16472@aloka.lostca.se> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.10.1 (2018-07-13) X-Spam-Status: No, score=-8.9 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Florian Weimer , Joseph Myers Errors-To: libc-alpha-bounces@sourceware.org Sender: "Libc-alpha" From: Arjun Shankar __float128 is a non-standard name and is not available on some architectures (like aarch64 or s390x) even though they may support the standard _Float128 type. Other architectures (like armv7) don't support quad-precision floating-point operations at all. This commit replaces benchtests references to __float128 with _Float128 and runs the corresponding tests only on architectures that support it. --- Thanks for looking at v1, Florian, Joseph and Paul. v2 changes: dropped configure[.ac] changes; used the float128[-alias]-fcts variables instead. Tested on armv7hl, i686, x86_64, aarch64, ppc64le, and s390x. benchtests/Makefile | 6 +++++- benchtests/expf128-inputs | 4 ++-- benchtests/powf128-inputs | 4 ++-- benchtests/sinf128-inputs | 4 ++-- 4 files changed, 11 insertions(+), 7 deletions(-) diff --git a/benchtests/Makefile b/benchtests/Makefile index 3095076055..922e2a94b1 100644 --- a/benchtests/Makefile +++ b/benchtests/Makefile @@ -25,7 +25,11 @@ bench-math := acos acosh asin asinh atan atanh cos cosh exp exp2 log log2 \ modf pow rint sin sincos sinh sqrt tan tanh fmin fmax fminf \ fmaxf powf trunc truncf roundeven roundevenf expf exp2f logf \ log2f sincosf sinf cosf isnan isinf isfinite hypot logb logbf \ - exp10f expf128 powf128 sinf128 + exp10f + +ifneq (,$(filter yes,$(float128-fcts) $(float128-alias-fcts))) +bench-math += expf128 powf128 sinf128 +endif bench-pthread := pthread_once thread_create diff --git a/benchtests/expf128-inputs b/benchtests/expf128-inputs index 5b36f8672a..5dc0f4e49b 100644 --- a/benchtests/expf128-inputs +++ b/benchtests/expf128-inputs @@ -1,6 +1,6 @@ ## includes: math.h -## args: __float128 -## ret: __float128 +## args: _Float128 +## ret: _Float128 # Random inputs in [-10,10] ## name: workload-random.wrf 0x4.e6f9d6da10d9a422942a89cdfa1p+0L diff --git a/benchtests/powf128-inputs b/benchtests/powf128-inputs index 7cbabafff6..6826266ad5 100644 --- a/benchtests/powf128-inputs +++ b/benchtests/powf128-inputs @@ -1,6 +1,6 @@ ## includes: math.h -## args: __float128:__float128 -## ret: __float128 +## args: _Float128:_Float128 +## ret: _Float128 # Random inputs in [-10,10] such that x and y are not both negative ## name: workload-random.wrf 0x8.130b31ed5288656428a29cead83p+0L, -0x6.e7ead09b7877a118813b50cfb3c8p+0L diff --git a/benchtests/sinf128-inputs b/benchtests/sinf128-inputs index 9aaf312413..01b7533680 100644 --- a/benchtests/sinf128-inputs +++ b/benchtests/sinf128-inputs @@ -1,6 +1,6 @@ ## includes: math.h -## args: __float128 -## ret: __float128 +## args: _Float128 +## ret: _Float128 # Random inputs in [-10,10] ## name: workload-random.wrf 0x4.e6f9d6da10d9a422942a89cdfa1p+0