From nobody Wed Nov 12 08:39:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1569342170; cv=none; d=zoho.com; s=zohoarc; b=TaIQiVYKXebP+RTqVzB5wE2SDUmL3Wc5DjYXkf2CYvBjAg6NpJmOlwjVtqv8lQ350HdxIcrWKKGm7P/8xwtwoFtgklVhA0jcH9haa/zHtO0Ct+huCpZy5dQrwJc8R8zVcjbyOsayTacTvTi0wLD8FyTLq4YL2IEsthmhivsE8Nc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1569342170; h=Content-Transfer-Encoding:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=Wxf2Z9wE3XH1g18IZhNDDG/1eENkvZ8JmM1gkxwbS0I=; b=dkD6J5OnLmiD8NMH6w4KEaEqOTMfhYTpq5oB9mm2X6dQ9mc9EHNTiM92leuFYu9ORvSOG9Nca8DmqXDPYHHPejxZIaFApDRwL6QC37qZctVeXW+pPIrgoL5InjqmtvZ1XNtLqlRW+4kZxqhvO9RO+HKsfjPdHE1TCHYKfvGlEbo= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1569342170713577.5356271312971; Tue, 24 Sep 2019 09:22:50 -0700 (PDT) Received: from localhost ([::1]:47952 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iCnaT-0001OH-98 for importer@patchew.org; Tue, 24 Sep 2019 12:22:49 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51214) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iCmuV-0008HL-KT for qemu-devel@nongnu.org; Tue, 24 Sep 2019 11:39:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iCmuT-0006Gg-M9 for qemu-devel@nongnu.org; Tue, 24 Sep 2019 11:39:27 -0400 Received: from mail.ilande.co.uk ([46.43.2.167]:35716 helo=mail.default.ilande.uk0.bigv.io) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iCmuR-0005fn-Ki; Tue, 24 Sep 2019 11:39:23 -0400 Received: from host86-138-245-63.range86-138.btcentralplus.com ([86.138.245.63] helo=kentang.home) by mail.default.ilande.uk0.bigv.io with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.89) (envelope-from ) id 1iCmrc-0007WZ-0s; Tue, 24 Sep 2019 16:36:28 +0100 From: Mark Cave-Ayland To: qemu-devel@nongnu.org, qemu-ppc@nongnu.org, pc@us.ibm.com, david@gibson.dropbear.id.au Date: Tue, 24 Sep 2019 16:35:56 +0100 Message-Id: <20190924153556.27575-8-mark.cave-ayland@ilande.co.uk> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20190924153556.27575-1-mark.cave-ayland@ilande.co.uk> References: <20190924153556.27575-1-mark.cave-ayland@ilande.co.uk> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-SA-Exim-Connect-IP: 86.138.245.63 X-SA-Exim-Mail-From: mark.cave-ayland@ilande.co.uk Subject: [PATCH 7/7] target/ppc: remove unnecessary if() around calls to set_dfp{64, 128}() in DFP macros X-SA-Exim-Version: 4.2.1 (built Tue, 02 Aug 2016 21:08:31 +0000) X-SA-Exim-Scanned: Yes (on mail.default.ilande.uk0.bigv.io) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 46.43.2.167 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Now that the parameters to both set_dfp64() and set_dfp128() are exactly the same, there is no need for an explicit if() statement to determine which function should be called based upon size. Instead we can simply use the preprocessor to generate the call to set_dfp##size() directly. Signed-off-by: Mark Cave-Ayland Reviewed-by: Richard Henderson --- target/ppc/dfp_helper.c | 60 +++++++---------------------------------- 1 file changed, 10 insertions(+), 50 deletions(-) diff --git a/target/ppc/dfp_helper.c b/target/ppc/dfp_helper.c index c2d335e928..8100bfc98d 100644 --- a/target/ppc/dfp_helper.c +++ b/target/ppc/dfp_helper.c @@ -427,11 +427,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, ppc_= fprp_t *a, \ dnop(&dfp.t, &dfp.a, &dfp.b, &dfp.context); = \ dfp_finalize_decimal##size(&dfp); = \ postprocs(&dfp); = \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); = \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); = \ - } = \ + set_dfp##size(t, &dfp.vt); = \ } =20 static void ADD_PPs(struct PPC_DFP *dfp) @@ -747,11 +743,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, ppc_= fprp_t *b, \ dfp_finalize_decimal##size(&dfp); \ QUA_PPs(&dfp); \ \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); \ - } \ + set_dfp##size(t, &dfp.vt); \ } =20 DFP_HELPER_QUAI(dquai, 64) @@ -769,11 +761,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, ppc_= fprp_t *a, \ dfp_finalize_decimal##size(&dfp); \ QUA_PPs(&dfp); \ \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); \ - } \ + set_dfp##size(t, &dfp.vt); \ } =20 DFP_HELPER_QUA(dqua, 64) @@ -851,11 +839,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, ppc_= fprp_t *a, \ dfp_finalize_decimal##size(&dfp); \ QUA_PPs(&dfp); \ \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); \ - } \ + set_dfp##size(t, &dfp.vt); \ } =20 DFP_HELPER_RRND(drrnd, 64) @@ -874,11 +858,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, ppc_= fprp_t *b, \ dfp_finalize_decimal##size(&dfp); = \ postprocs(&dfp); = \ = \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); = \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); = \ - } = \ + set_dfp##size(t, &dfp.vt); = \ } =20 static void RINTX_PPs(struct PPC_DFP *dfp) @@ -978,11 +958,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, ppc_= fprp_t *b) \ dfp_finalize_decimal##size(&dfp); = \ CFFIX_PPs(&dfp); = \ = \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); = \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); = \ - } = \ + set_dfp##size(t, &dfp.vt); = \ } =20 static void CFFIX_PPs(struct PPC_DFP *dfp) @@ -1088,11 +1064,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, pp= c_fprp_t *b, \ dfp_set_sign_##size(&dfp.vt, sgn); \ } \ \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); \ - } \ + set_dfp##size(t, &dfp.vt); \ } =20 DFP_HELPER_DEDPD(ddedpd, 64) @@ -1160,11 +1132,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, pp= c_fprp_t *b, \ } = \ dfp_finalize_decimal##size(&dfp); = \ dfp_set_FPRF_from_FRT(&dfp); = \ - if ((size) =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); = \ - } else if ((size) =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); = \ - } = \ + set_dfp##size(t, &dfp.vt); = \ } =20 DFP_HELPER_ENBCD(denbcd, 64) @@ -1264,11 +1232,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, pp= c_fprp_t *a, \ dfp.t.exponent =3D exp - bias; = \ dfp_finalize_decimal##size(&dfp); \ } \ - if (size =3D=3D 64) { = \ - set_dfp64(t, &dfp.vt); \ - } else if (size =3D=3D 128) { = \ - set_dfp128(t, &dfp.vt); \ - } \ + set_dfp##size(t, &dfp.vt); \ } =20 DFP_HELPER_IEX(diex, 64) @@ -1356,11 +1320,7 @@ void helper_##op(CPUPPCState *env, ppc_fprp_t *t, pp= c_fprp_t *a, \ } \ } \ \ - if ((size) =3D=3D 64) { \ - set_dfp64(t, &dfp.vt); \ - } else { \ - set_dfp128(t, &dfp.vt); \ - } \ + set_dfp##size(t, &dfp.vt); \ } =20 DFP_HELPER_SHIFT(dscli, 64, 1) --=20 2.20.1