From patchwork Tue May 1 13:55:46 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Laurent Vivier X-Patchwork-Id: 907066 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=nongnu.org (client-ip=208.118.235.17; helo=lists.gnu.org; envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org; receiver=) Authentication-Results: ozlabs.org; dmarc=none (p=none dis=none) header.from=vivier.eu Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 40b31y6tsgz9s2k for ; Tue, 1 May 2018 23:59:50 +1000 (AEST) Received: from localhost ([::1]:42648 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fDVoq-0005hX-V2 for incoming@patchwork.ozlabs.org; Tue, 01 May 2018 09:59:48 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44049) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fDVlJ-0003RS-W8 for qemu-devel@nongnu.org; Tue, 01 May 2018 09:56:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fDVlI-00081F-OF for qemu-devel@nongnu.org; Tue, 01 May 2018 09:56:10 -0400 Received: from mout.kundenserver.de ([212.227.126.134]:42769) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1fDVlC-0007ua-K4; Tue, 01 May 2018 09:56:02 -0400 Received: from localhost.localdomain ([78.238.229.36]) by mrelayeu.kundenserver.de (mreue007 [212.227.15.167]) with ESMTPSA (Nemesis) id 0MHehs-1fCQMN2Who-003NH7; Tue, 01 May 2018 15:56:00 +0200 From: Laurent Vivier To: qemu-devel@nongnu.org Date: Tue, 1 May 2018 15:55:46 +0200 Message-Id: <20180501135547.12414-4-laurent@vivier.eu> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180501135547.12414-1-laurent@vivier.eu> References: <20180501135547.12414-1-laurent@vivier.eu> X-Provags-ID: V03:K1:t6NaEkclVVLvTfAPpfQo33Ce9K7pX2kK2KMeMgjVlGlpqtetvJY 0At7TbI6KUx8XyaMd5GUPLtskbVR7DtdS0QLDCx8Nj13SNWNGSskUal83qkaIPAFvWbKvih Mzqh91+MnKjfP49aiElqcltx1twz7MjwfFYVcjsOFPYl6203LdLm+mdm97ajkIdku65RkfD WdPXfuCBAufgnBExTsr7w== X-UI-Out-Filterresults: notjunk:1; V01:K0:1AC23fPdMxA=:nCdhlPeR2G9hWynvwWL5KN ROF8XELWpHHDXQ0EVUY7YdKyFvc0SgPq4KJEtH1V9kU9DvKZqdqtdVIzWKKjV3QAD0Z3ykeXi xp/WS0hlAhjl4SsS+vBFwK5wsewUBVW2dmIQMmKqks/XRaxPVDiw1BGvJr0I8Tm6SB/vJjs9q T/+QD+CHWAxEXVndc9p+TGWvQ1XKQu/OlIEaUx+2T1NdJzF7BMnimKLoZ2hio5x9iW+iXzuxr 5PvhzA49gxEItJDd+6WDZJY2OWYAJMg6VKp/c0uszEDMrSZaaIsR22zFYecTeM4L2wDS+PLEn YH2abq2gnpFSxUnMdrDYl0zpVHg5DvfyK6reOpoJiOo4bjmx1ekjrQWUrB8neCK93xpAVnddu pBchuAvFvKj3YKAIkMCc9FTf7GUqlXUECzQ2/jL+1VW/M+Cl+YcJrHz8cS490oGFhDgfHBax1 Jesh3Zk+KkJeiYy+0bXg5GfT5k4ijb9458RpcgRSPav6dipdp7P9/ty0b3F4Z3iXYCN6AFnJR qnEqOeAR85Sf3I1hv5v12HuAci+cygT9mQ6UjFco8OE2LLWmoyuFdXOkOBP/1TBfq2np2P6f1 uNNgNMRWaVGqmIGXC8JUs7qrGq5WlqrsIryIcoAkeZvP3bdnj6OYuGU7iPzk0pneRpC/MyhKY wwHHZ+J0JI3pCJTbI2oYw9KO2Y7DQ2vbQiVh0cPxPKAS33yBrRnQ4rgYl4Izs/LLM1BI= X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 212.227.126.134 Subject: [Qemu-devel] [PULL 3/4] m68k: remove dead code (Coverity CID1390617) X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: qemu-trivial@nongnu.org, Thomas Huth , Michael Tokarev , Laurent Vivier Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: "Qemu-devel" floatx80_sin() and floatx80_cos() are derived from one sincos() function. They have both unused code coming from their common origin. Remove it. Signed-off-by: Laurent Vivier Reviewed-by: Peter Maydell Message-Id: <20180430170156.1860-2-laurent@vivier.eu> --- target/m68k/softfloat.c | 43 ++++++++++++++----------------------------- 1 file changed, 14 insertions(+), 29 deletions(-) diff --git a/target/m68k/softfloat.c b/target/m68k/softfloat.c index 2c069a5efb..e41b07d042 100644 --- a/target/m68k/softfloat.c +++ b/target/m68k/softfloat.c @@ -1497,7 +1497,7 @@ floatx80 floatx80_sin(floatx80 a, float_status *status) int32_t compact, l, n, j; floatx80 fp0, fp1, fp2, fp3, fp4, fp5, x, invtwopi, twopi1, twopi2; float32 posneg1, twoto63; - flag adjn, endflag; + flag endflag; aSig = extractFloatx80Frac(a); aExp = extractFloatx80Exp(a); @@ -1515,8 +1515,6 @@ floatx80 floatx80_sin(floatx80 a, float_status *status) return packFloatx80(aSign, 0, 0); } - adjn = 0; - user_rnd_mode = status->float_rounding_mode; user_rnd_prec = status->floatx80_rounding_precision; status->float_rounding_mode = float_round_nearest_even; @@ -1591,14 +1589,8 @@ floatx80 floatx80_sin(floatx80 a, float_status *status) status->float_rounding_mode = user_rnd_mode; status->floatx80_rounding_precision = user_rnd_prec; - if (adjn) { - /* COSTINY */ - a = floatx80_sub(fp0, float32_to_floatx80( - make_float32(0x00800000), status), status); - } else { - /* SINTINY */ - a = floatx80_move(a, status); - } + /* SINTINY */ + a = floatx80_move(a, status); float_raise(float_flag_inexact, status); return a; @@ -1616,7 +1608,7 @@ floatx80 floatx80_sin(floatx80 a, float_status *status) status); /* FP0 IS R = (X-Y1)-Y2 */ sincont: - if ((n + adjn) & 1) { + if (n & 1) { /* COSPOLY */ fp0 = floatx80_mul(fp0, fp0, status); /* FP0 IS S */ fp1 = floatx80_mul(fp0, fp0, status); /* FP1 IS T */ @@ -1629,7 +1621,7 @@ floatx80 floatx80_sin(floatx80 a, float_status *status) xExp = extractFloatx80Exp(fp0); xSig = extractFloatx80Frac(fp0); - if (((n + adjn) >> 1) & 1) { + if ((n >> 1) & 1) { xSign ^= 1; posneg1 = make_float32(0xBF800000); /* -1 */ } else { @@ -1681,7 +1673,7 @@ floatx80 floatx80_sin(floatx80 a, float_status *status) xExp = extractFloatx80Exp(fp0); xSig = extractFloatx80Frac(fp0); - xSign ^= ((n + adjn) >> 1) & 1; /* X IS NOW R'= SGN*R */ + xSign ^= (n >> 1) & 1; /* X IS NOW R'= SGN*R */ fp0 = floatx80_mul(fp0, fp0, status); /* FP0 IS S */ fp1 = floatx80_mul(fp0, fp0, status); /* FP1 IS T */ @@ -1744,7 +1736,7 @@ floatx80 floatx80_cos(floatx80 a, float_status *status) int32_t compact, l, n, j; floatx80 fp0, fp1, fp2, fp3, fp4, fp5, x, invtwopi, twopi1, twopi2; float32 posneg1, twoto63; - flag adjn, endflag; + flag endflag; aSig = extractFloatx80Frac(a); aExp = extractFloatx80Exp(a); @@ -1762,8 +1754,6 @@ floatx80 floatx80_cos(floatx80 a, float_status *status) return packFloatx80(0, one_exp, one_sig); } - adjn = 1; - user_rnd_mode = status->float_rounding_mode; user_rnd_prec = status->floatx80_rounding_precision; status->float_rounding_mode = float_round_nearest_even; @@ -1837,15 +1827,10 @@ floatx80 floatx80_cos(floatx80 a, float_status *status) status->float_rounding_mode = user_rnd_mode; status->floatx80_rounding_precision = user_rnd_prec; - if (adjn) { - /* COSTINY */ - a = floatx80_sub(fp0, float32_to_floatx80( - make_float32(0x00800000), status), - status); - } else { - /* SINTINY */ - a = floatx80_move(a, status); - } + /* COSTINY */ + a = floatx80_sub(fp0, float32_to_floatx80( + make_float32(0x00800000), status), + status); float_raise(float_flag_inexact, status); return a; @@ -1863,7 +1848,7 @@ floatx80 floatx80_cos(floatx80 a, float_status *status) status); /* FP0 IS R = (X-Y1)-Y2 */ sincont: - if ((n + adjn) & 1) { + if ((n + 1) & 1) { /* COSPOLY */ fp0 = floatx80_mul(fp0, fp0, status); /* FP0 IS S */ fp1 = floatx80_mul(fp0, fp0, status); /* FP1 IS T */ @@ -1876,7 +1861,7 @@ floatx80 floatx80_cos(floatx80 a, float_status *status) xExp = extractFloatx80Exp(fp0); xSig = extractFloatx80Frac(fp0); - if (((n + adjn) >> 1) & 1) { + if (((n + 1) >> 1) & 1) { xSign ^= 1; posneg1 = make_float32(0xBF800000); /* -1 */ } else { @@ -1927,7 +1912,7 @@ floatx80 floatx80_cos(floatx80 a, float_status *status) xExp = extractFloatx80Exp(fp0); xSig = extractFloatx80Frac(fp0); - xSign ^= ((n + adjn) >> 1) & 1; /* X IS NOW R'= SGN*R */ + xSign ^= ((n + 1) >> 1) & 1; /* X IS NOW R'= SGN*R */ fp0 = floatx80_mul(fp0, fp0, status); /* FP0 IS S */ fp1 = floatx80_mul(fp0, fp0, status); /* FP1 IS T */