From nobody Mon Feb 9 12:09:36 2026 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; dkim=fail; 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; dmarc=fail(p=none dis=none) header.from=gmail.com ARC-Seal: i=1; a=rsa-sha256; t=1564596348; cv=none; d=zoho.com; s=zohoarc; b=L4SmX99KjlUjQ0m7KE2Lbm7YNqcASab0baAo+iiMWhf/jPiu6hVeIbq/pv4CUzWs98OqTYh5qOVBGWppV9FqRcS1YugRq3UiNXYAqsk6S2ZpD5GpmcJLcFFWcJImGr+wVwSSbRwHNOEp6JCcHHG4J5PIrAa/ZzoCuIlIQoEIHps= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1564596348; h=Content-Transfer-Encoding:Cc: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=qOqDTxFtH/3YrZnFMnrtU1fae7f/i4Xpw9RfJj8CEIA=; b=ZZrEb1ua3RPUZey21AcjpNK20X6Swya3D+aqFQOjDKBSbZKc92lG3ynAkXR/tN7HhV9tDb01SNAVgKHdTCORu9Alh4OjOlpXI5wPXqXtOYheyCxQwOs7qVIPRHDXy7wxA50PQRDC3/PF3FxliZJ9B1ZdnLW4gsaVTMBrMhpV8IY= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=fail; 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; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1564596348481162.5073872981485; Wed, 31 Jul 2019 11:05:48 -0700 (PDT) Received: from localhost ([::1]:43450 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hssyx-0000TV-95 for importer@patchew.org; Wed, 31 Jul 2019 14:05:47 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42832) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hssrH-0002Lr-A0 for qemu-devel@nongnu.org; Wed, 31 Jul 2019 13:57:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hssrG-0002qd-0K for qemu-devel@nongnu.org; Wed, 31 Jul 2019 13:57:51 -0400 Received: from mail-yb1-xb44.google.com ([2607:f8b0:4864:20::b44]:46504) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hssrF-0002pw-SG for qemu-devel@nongnu.org; Wed, 31 Jul 2019 13:57:49 -0400 Received: by mail-yb1-xb44.google.com with SMTP id a5so24731488ybo.13 for ; Wed, 31 Jul 2019 10:57:49 -0700 (PDT) Received: from dionysus.attlocal.net (69-222-133-165.lightspeed.tukrga.sbcglobal.net. [69.222.133.165]) by smtp.gmail.com with ESMTPSA id 206sm16077696ywk.44.2019.07.31.10.57.48 (version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256); Wed, 31 Jul 2019 10:57:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=qOqDTxFtH/3YrZnFMnrtU1fae7f/i4Xpw9RfJj8CEIA=; b=WjGwy4kbutEvop+EAKk57CutipLQ838mde8k+PywgdSwCT57rX8Kbf5oT9Ju4tbhfq RFfQhKMWVnJmb2vQ4Xs+FAX/0RS7G/2ey0rvU72ryEqUvJPOl1tkSp0fJ55RN1xZ/c/a uKKKtmzp+xFDx6XgtzUSLH5nqUbeekPCutN3I8h3oN77NQ0hn+2HnCIFbVBp/LiXUiZ/ EG0u69Z3ca6TYfBHEQaB00tPJmRtrFildamzVkjCZ14okvj6xil4wrQLY9Zrv+xEpBS3 ASmJPiqNhS41rPfuWqkge078TaYapBhpm9cDR8y+u8Yi5BEpfdagOxs6oc3nEjodqNmT S+kQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=qOqDTxFtH/3YrZnFMnrtU1fae7f/i4Xpw9RfJj8CEIA=; b=O9p4h1pc0hXz41JJ2OG5kWzS/8yQ4wrhVhaJC+eNU58N+g/BT4QOSj2N0j5nfnPow2 MF5rzRKHNtWk8CadjYovrHyKRVnf6LK+Y49vZ2yZQQGf3Ceo5y+BlZWvIvru0xp7GZZg bCtct0Nj1DTYMrd2StxKf8cWaF3mXlVtzKf1VxVhIpsEp5OqsX5c9IhdzTG24oYdI1Cf taZKWb4Y81CV5ciFBh2m6PY9CATa2CkPEsn9y76Y099BXVbDNf1tI2NEah9FC4XwMqCR xrKCO1UXA0F+FnLyUK+pYKpzKOSet6nzg3IIzABTBonEshWfzJ+q3PIz0Qm8kJyXoXwb qQgw== X-Gm-Message-State: APjAAAUR+YCNqokOY3PCcyXMDcOQPv3id/j9/3/5a9BW9xc2XWFzwnUc cn0m697Av8Nf2rh12glF91DKX0il X-Google-Smtp-Source: APXvYqzxokb+SycQz1t1yMYEg3fsCqPydDjNuNyxYCSDA5ElksCFT0MHJq50oqap7i6+FKWhuSHfag== X-Received: by 2002:a25:ba07:: with SMTP id t7mr16920323ybg.261.1564595869012; Wed, 31 Jul 2019 10:57:49 -0700 (PDT) From: Jan Bobek To: qemu-devel@nongnu.org Date: Wed, 31 Jul 2019 13:56:53 -0400 Message-Id: <20190731175702.4916-14-jan.bobek@gmail.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <20190731175702.4916-1-jan.bobek@gmail.com> References: <20190731175702.4916-1-jan.bobek@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:4864:20::b44 Subject: [Qemu-devel] [RFC PATCH v1 13/22] target/i386: reimplement (V)PSUB(B, W, D, Q) 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: , Cc: =?UTF-8?q?Alex=20Benn=C3=A9e?= , Richard Henderson , Jan Bobek Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Use the gvec infrastructure to achieve the desired functionality. Signed-off-by: Jan Bobek --- target/i386/ops_sse.h | 6 ------ target/i386/ops_sse_header.h | 5 ----- target/i386/translate.c | 33 +++++++++++++++++++++++++++++---- 3 files changed, 29 insertions(+), 15 deletions(-) diff --git a/target/i386/ops_sse.h b/target/i386/ops_sse.h index 31a761a89a..59935a65be 100644 --- a/target/i386/ops_sse.h +++ b/target/i386/ops_sse.h @@ -342,7 +342,6 @@ static inline int satsw(int x) #define FADDSB(a, b) satsb((int8_t)(a) + (int8_t)(b)) #define FADDSW(a, b) satsw((int16_t)(a) + (int16_t)(b)) =20 -#define FSUB(a, b) ((a) - (b)) #define FSUBUB(a, b) satub((a) - (b)) #define FSUBUW(a, b) satuw((a) - (b)) #define FSUBSB(a, b) satsb((int8_t)(a) - (int8_t)(b)) @@ -365,11 +364,6 @@ static inline int satsw(int x) #define FAVG(a, b) (((a) + (b) + 1) >> 1) #endif =20 -SSE_HELPER_B(helper_psubb, FSUB) -SSE_HELPER_W(helper_psubw, FSUB) -SSE_HELPER_L(helper_psubl, FSUB) -SSE_HELPER_Q(helper_psubq, FSUB) - SSE_HELPER_B(helper_paddusb, FADDUB) SSE_HELPER_B(helper_paddsb, FADDSB) SSE_HELPER_B(helper_psubusb, FSUBUB) diff --git a/target/i386/ops_sse_header.h b/target/i386/ops_sse_header.h index 5c69ab91d4..bcdbac99a0 100644 --- a/target/i386/ops_sse_header.h +++ b/target/i386/ops_sse_header.h @@ -60,11 +60,6 @@ DEF_HELPER_3(glue(pslldq, SUFFIX), void, env, Reg, Reg) #define SSE_HELPER_Q(name, F)\ DEF_HELPER_3(glue(name, SUFFIX), void, env, Reg, Reg) =20 -SSE_HELPER_B(psubb, FSUB) -SSE_HELPER_W(psubw, FSUB) -SSE_HELPER_L(psubl, FSUB) -SSE_HELPER_Q(psubq, FSUB) - SSE_HELPER_B(paddusb, FADDUB) SSE_HELPER_B(paddsb, FADDSB) SSE_HELPER_B(psubusb, FSUBUB) diff --git a/target/i386/translate.c b/target/i386/translate.c index 1dbeb49066..6f4dfd06a1 100644 --- a/target/i386/translate.c +++ b/target/i386/translate.c @@ -2855,10 +2855,10 @@ static const SSEFunc_0_epp sse_op_table1[256][4] = =3D { [0xf6] =3D MMX_OP2(psadbw), [0xf7] =3D { (SSEFunc_0_epp)gen_helper_maskmov_mmx, (SSEFunc_0_epp)gen_helper_maskmov_xmm }, /* XXX: casts */ - [0xf8] =3D MMX_OP2(psubb), - [0xf9] =3D MMX_OP2(psubw), - [0xfa] =3D MMX_OP2(psubl), - [0xfb] =3D MMX_OP2(psubq), + [0xf8] =3D { SSE_TOMBSTONE, SSE_TOMBSTONE }, + [0xf9] =3D { SSE_TOMBSTONE, SSE_TOMBSTONE }, + [0xfa] =3D { SSE_TOMBSTONE, SSE_TOMBSTONE }, + [0xfb] =3D { SSE_TOMBSTONE, SSE_TOMBSTONE }, [0xfc] =3D { SSE_TOMBSTONE, SSE_TOMBSTONE }, [0xfd] =3D { SSE_TOMBSTONE, SSE_TOMBSTONE }, [0xfe] =3D { SSE_TOMBSTONE, SSE_TOMBSTONE }, @@ -3171,6 +3171,11 @@ static inline void gen_gvec_ld_modrm_3(CPUX86State *= env, DisasContext *s, #define gen_vpadd_xmm(env, s, modrm, vece) gen_gvec_ld_modrm_vxmm((env), (= s), (modrm), (vece), tcg_gen_gvec_add, 0123) #define gen_vpadd_ymm(env, s, modrm, vece) gen_gvec_ld_modrm_vymm((env), (= s), (modrm), (vece), tcg_gen_gvec_add, 0123) =20 +#define gen_psub_mm(env, s, modrm, vece) gen_gvec_ld_modrm_mm ((env), (= s), (modrm), (vece), tcg_gen_gvec_sub, 0112) +#define gen_psub_xmm(env, s, modrm, vece) gen_gvec_ld_modrm_xmm ((env), (= s), (modrm), (vece), tcg_gen_gvec_sub, 0112) +#define gen_vpsub_xmm(env, s, modrm, vece) gen_gvec_ld_modrm_vxmm((env), (= s), (modrm), (vece), tcg_gen_gvec_sub, 0123) +#define gen_vpsub_ymm(env, s, modrm, vece) gen_gvec_ld_modrm_vymm((env), (= s), (modrm), (vece), tcg_gen_gvec_sub, 0123) + #define gen_pand_mm(env, s, modrm) gen_gvec_ld_modrm_mm ((env), (s), (m= odrm), MO_64, tcg_gen_gvec_and, 0112) #define gen_pand_xmm(env, s, modrm) gen_gvec_ld_modrm_xmm ((env), (s), (m= odrm), MO_64, tcg_gen_gvec_and, 0112) #define gen_vpand_xmm(env, s, modrm) gen_gvec_ld_modrm_vxmm((env), (s), (m= odrm), MO_64, tcg_gen_gvec_and, 0123) @@ -3326,6 +3331,26 @@ static void gen_sse(CPUX86State *env, DisasContext *= s, int b) case 0xd4 | M_0F | P_66 | VEX_128: gen_vpadd_xmm(env, s, modrm, MO_64)= ; return; case 0xd4 | M_0F | P_66 | VEX_256: gen_vpadd_ymm(env, s, modrm, MO_64)= ; return; =20 + case 0xf8 | M_0F: gen_psub_mm(env, s, modrm, MO_8); r= eturn; + case 0xf8 | M_0F | P_66: gen_psub_xmm(env, s, modrm, MO_8); = return; + case 0xf8 | M_0F | P_66 | VEX_128: gen_vpsub_xmm(env, s, modrm, MO_8);= return; + case 0xf8 | M_0F | P_66 | VEX_256: gen_vpsub_ymm(env, s, modrm, MO_8);= return; + + case 0xf9 | M_0F: gen_psub_mm(env, s, modrm, MO_16); = return; + case 0xf9 | M_0F | P_66: gen_psub_xmm(env, s, modrm, MO_16);= return; + case 0xf9 | M_0F | P_66 | VEX_128: gen_vpsub_xmm(env, s, modrm, MO_16)= ; return; + case 0xf9 | M_0F | P_66 | VEX_256: gen_vpsub_ymm(env, s, modrm, MO_16)= ; return; + + case 0xfa | M_0F: gen_psub_mm(env, s, modrm, MO_32); = return; + case 0xfa | M_0F | P_66: gen_psub_xmm(env, s, modrm, MO_32);= return; + case 0xfa | M_0F | P_66 | VEX_128: gen_vpsub_xmm(env, s, modrm, MO_32)= ; return; + case 0xfa | M_0F | P_66 | VEX_256: gen_vpsub_ymm(env, s, modrm, MO_32)= ; return; + + case 0xfb | M_0F: gen_psub_mm(env, s, modrm, MO_64); = return; + case 0xfb | M_0F | P_66: gen_psub_xmm(env, s, modrm, MO_64);= return; + case 0xfb | M_0F | P_66 | VEX_128: gen_vpsub_xmm(env, s, modrm, MO_64)= ; return; + case 0xfb | M_0F | P_66 | VEX_256: gen_vpsub_ymm(env, s, modrm, MO_64)= ; return; + case 0xdb | M_0F: gen_pand_mm(env, s, modrm); return; case 0xdb | M_0F | P_66: gen_pand_xmm(env, s, modrm); return; case 0xdb | M_0F | P_66 | VEX_128: gen_vpand_xmm(env, s, modrm); retur= n; --=20 2.20.1