From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg1-x530.google.com (mail-pg1-x530.google.com [IPv6:2607:f8b0:4864:20::530]) by sourceware.org (Postfix) with ESMTPS id F390C3858D28 for ; Wed, 20 Jul 2022 17:01:05 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org F390C3858D28 Received: by mail-pg1-x530.google.com with SMTP id o18so16926792pgu.9 for ; Wed, 20 Jul 2022 10:01:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=g7r982aG2PiEYoG5YnfLEPW5m//xw/0K6kSuUOHqeZ8=; b=0cmEovwEaz2tqkDLX0VvJzqgo43UqX1ebGN1zRcwV1v6GXetld6DcVnlWHz5mplxVD fzOSKPKudzTXylWtDEm85xQ1g/U1KXrSbb10EYY5W2LOedAfaiiDBk0LOUORYMpEVWVI zNrs0/A72dnDKwTahR283oqL4UtLUycl07XAnUmLqFwi2mVEeHd7wHAWSsRZqS7JwB5y ywX9U8WgK1ZFH91j+3wvNAmzOz3Ou4okaBVlGD9tr+LbKmFN0Ja2R2p6Fl6WXLs0hQei oWehEtAmt7Tysv+hELIM9/EJ+zIazP3T6eW0tgicLpQtJe0j4nL4eclXs8SLXaz0NmSG OE7A== X-Gm-Message-State: AJIora82ZK8AaOETlpFZSve1fkk9Bl8uI2CDv0RiWziCZYwPZFHwQ2Cx euJoFaT9Ms7C4MVtxjfq0HN62dyU7lEy9X+mQI5dJXNC X-Google-Smtp-Source: AGRyM1tCLo0qChv74nwf2tJVqAPf3+0j535XzHVfUIQS90/g9Vqv1TLPyaDDPKKm1uxr8xtmgpyHkr7nUEqkUFx44rk= X-Received: by 2002:a05:6a00:793:b0:52a:b261:f8e7 with SMTP id g19-20020a056a00079300b0052ab261f8e7mr39599857pfu.20.1658336464802; Wed, 20 Jul 2022 10:01:04 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: "H.J. Lu" Date: Wed, 20 Jul 2022 10:00:29 -0700 Message-ID: Subject: Re: [PATCH] x86: replace wrong attributes on VCVTDQ2PH{X,Y} To: Jan Beulich Cc: Binutils Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-3018.5 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: binutils@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Binutils mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Jul 2022 17:01:07 -0000 On Wed, Jul 20, 2022 at 9:05 AM Jan Beulich wrote: > > A standalone (without SAE) StaticRounding attribute is meaningless, and > indeed all other similar insns have ATTSyntax there instead. I can only > assume this was some strange copy-and-paste mistake. Correct. These 2 instructions don't support static rounding. > --- > Yet another candidate for 2.39 I think. OK for master and 2.39 branch. Thanks. > --- a/opcodes/i386-opc.tbl > +++ b/opcodes/i386-opc.tbl > @@ -3560,8 +3560,8 @@ vucomish, 0x2e, None, CpuAVX512_FP16, Mo > > vcvtdq2ph, 0x5b, None, CpuAVX512_FP16, Modrm|EVex512|Masking=3|EVexMap5|VexW0|Broadcast|Disp8MemShift=6|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|StaticRounding|SAE, { RegZMM|Dword|Unspecified|BaseIndex, RegYMM } > vcvtdq2ph, 0x5b, None, CpuAVX512_FP16|CpuAVX512VL, Modrm|Masking=3|EVexMap5|VexW0|Broadcast|Disp8ShiftVL|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf, { RegXMM|RegYMM|Dword|BaseIndex, RegXMM } > -vcvtdq2phx, 0x5b, None, CpuAVX512_FP16|CpuAVX512VL, Modrm|EVex128|Masking=3|EVexMap5|VexW0|Disp8MemShift=4|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|StaticRounding, { RegXMM|Unspecified|BaseIndex, RegXMM } > -vcvtdq2phy, 0x5b, None, CpuAVX512_FP16|CpuAVX512VL, Modrm|EVex256|Masking=3|EVexMap5|VexW0|Disp8MemShift=5|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|StaticRounding, { RegYMM|Unspecified|BaseIndex, RegXMM } > +vcvtdq2phx, 0x5b, None, CpuAVX512_FP16|CpuAVX512VL, Modrm|EVex128|Masking=3|EVexMap5|VexW0|Disp8MemShift=4|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|ATTSyntax, { RegXMM|Unspecified|BaseIndex, RegXMM } > +vcvtdq2phy, 0x5b, None, CpuAVX512_FP16|CpuAVX512VL, Modrm|EVex256|Masking=3|EVexMap5|VexW0|Disp8MemShift=5|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|ATTSyntax, { RegYMM|Unspecified|BaseIndex, RegXMM } > > vcvtudq2ph, 0xf27a, None, CpuAVX512_FP16, Modrm|EVex512|Masking=3|EVexMap5|VexW0|Broadcast|Disp8MemShift=6|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf|StaticRounding|SAE, { RegZMM|Dword|Unspecified|BaseIndex, RegYMM } > vcvtudq2ph, 0xf27a, None, CpuAVX512_FP16|CpuAVX512VL, Modrm|Masking=3|EVexMap5|VexW0|Broadcast|Disp8ShiftVL|No_bSuf|No_wSuf|No_lSuf|No_sSuf|No_qSuf|No_ldSuf, { RegXMM|RegYMM|Dword|BaseIndex, RegXMM } -- H.J.