From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70087.outbound.protection.outlook.com [40.107.7.87]) by sourceware.org (Postfix) with ESMTPS id 59C503858C2C for ; Sun, 16 Oct 2022 06:34:39 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 59C503858C2C Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=suse.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=OIXZKQbgb5YTWRgPKHVxP5uWRWl5VerB25ugDQeMR9mNkFnDWk4X5kl3NzJzWAbXSVlQmOBxgYFPNJRgKeZjySQF6+SdbWpJM5lHT8VF4ublsfZzN9tRRIes5Se+Iti6YMAUz2ZQkZXynmqHjAOyOZaAASuGBdSusOBinIRbvvJAlmt2Pazyaybfll0apxOqATlu6sgXBfhD6xvqL5359i4lb0eYU+XuLrFEx9FNfdA3tx5OmgFfvyWW+H6pWjOYf9xqZf4F19KcQfU8Z6VMgzsZgpjGH7sjLZ/YW8rLwyrZO3H5WOYmZaunYOm0sm6T5xi5AXvPtnmcn2Z9CPuT3A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=QkhSlFnT+5KLS+N9MLPGa6LUBcxIj+zoETeMmP9GzTc=; b=fSxu8TS8gvSzqkweIeaVFqeApTfIJ2SrsTkE5WtkXrNIRsK3acvP58wvXCHJhLvMeJLLsMP53QjApwvNPudqdj1k0kfhhkCsQD2b5Dc0/oEfsIM01lKRtHbPNpf6jYrjE52v6nU/77k4Fzf0MbU2Ysl/OGWFhOa2g8NriE2Rn6+4XO4RzyUskfIuumdOHrL6EMx6vGCuVanwk7a/UW29+9iz9nMz8QF/oCjenWmIYg50IpJZEDSTXjdJW0L8a8497v30ByTyQTDoBdbWD7NmpKZj+lXf7OQ7o7G1D1mN0/BvWZojAYB9qLNY23QtnASfbCO8kAlDwuB0s/sJBBfVKA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=QkhSlFnT+5KLS+N9MLPGa6LUBcxIj+zoETeMmP9GzTc=; b=q0Ttt7NuyxXYr4phV0+8KeuF87zGxhyzq9uf52X6Sly0lbrHF++PcIhFmacTRvRXuY5LU2dN5wksjMjpL8BWrZ9SLCDzzPsdWeGCLnqMYnPjB3UDaeEAq0CQV+QvnTPQlE4gvQwu8KIB4i/EjP6P2CRsnLXNzyHHsNTJO/ViEV3fr0Dsw72LXikwlHxfBFpRNWLdqkJT4bhd2ZfZDgZos77r2C6koSaGAv1hcObUwqGkIH7b6LDffqvuXi1LqMc/uAljiclSVCn66SSw5oiXaEPimDxIPygRf7wNENtAbuPXHo6my3m36gg7361QXXr0ms6mLrM/CnRLnXc+yAwCTg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Received: from VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) by AS8PR04MB9048.eurprd04.prod.outlook.com (2603:10a6:20b:443::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5676.34; Sun, 16 Oct 2022 06:34:36 +0000 Received: from VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::2459:15ae:e6cb:218a]) by VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::2459:15ae:e6cb:218a%7]) with mapi id 15.20.5723.032; Sun, 16 Oct 2022 06:34:36 +0000 Message-ID: Date: Sun, 16 Oct 2022 08:34:33 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.3.2 Subject: Re: [PATCH 04/10] Support Intel CMPccXADD Content-Language: en-US To: "H.J. Lu" Cc: Haochen Jiang , binutils@sourceware.org References: <20221014091248.4920-1-haochen.jiang@intel.com> <20221014091248.4920-5-haochen.jiang@intel.com> <1d847a52-b1ff-b816-1507-7077724901bb@suse.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: AM5PR0602CA0019.eurprd06.prod.outlook.com (2603:10a6:203:a3::29) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|AS8PR04MB9048:EE_ X-MS-Office365-Filtering-Correlation-Id: 7b2e9bc9-60c4-489d-d866-08daaf407e61 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: BmRx6/OdNbUUaCd2gg+rXkBQKnMcm1VsdPB0S87EA2RNTqsLeDKN+3xQhTcPNwv/RZPiY2v2+rYsMB4ANiyjl4IzEjKzLeizbsU2Yq8ciCuSEbz0DexQVSECFSou5ynyMdYIXkZBX2/U8kll6dVTRe13SNPdX5vQovEO2U9P62sgxNP8SyYlBFkCGuVEkeHfYF1Ht2MvIw//jf3/ySvKQ6i5pmnRenmogKZHm2wb4OOhE8TMk9HPbKYqBqZUQYT7dgzreCCrmY8FQ3UDy0yXZ0233+VjNiN+Nh+sQ2EIuTiSNFHg1M8Rs9bzvVMMcAg6R5hycstzOimQwcMOl9J87AeNLTFPzCinJcDT9yONg1nfVzMe9I8BIRXBPBv5FNJvvU6DfToB3W8rkN5sl7xKJnqquKsRvr4NiW4WpbVHIrz2ixakdcWJ+h1tyfmG9puZw8krgCvid8tosgMSj5EVmXDE1TWNq6RSlfi8q9yy24h0ZA8KC3LvvyJrFC8IlfFnr63lI1PdD5S+gQ8+tWCRSA4UpWsXb/QtvqVxpuDp0mvJDIox20ExtJLE7Xxc/enYVCQGVExde1BmP6duZn3hoDxpv8EOtXu61Qf7AEsEZh1UEl9HTWHow8rtxpm0lKomPPl4ZYRS69zL/EwKvSF0rLSTyLf4J9sAFiSMrQcUJJQzTSDxTW9HK3UzB3FQen2MhvBffTbPwRkfzTPCP5zRXpauK31TGAiYWgj6ib1XZGJ6GGZwUmu3yj9zpDBizf5KJInzob9EXrNSK41lgXuo8vuDvQXAuEtjt2dEQVJI60M= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VE1PR04MB6560.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(376002)(136003)(366004)(39850400004)(396003)(346002)(451199015)(6916009)(31686004)(5660300002)(66556008)(66946007)(66476007)(316002)(478600001)(4326008)(6506007)(41300700001)(53546011)(8676002)(6512007)(186003)(6486002)(2906002)(8936002)(2616005)(83380400001)(36756003)(6666004)(38100700002)(31696002)(86362001)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?VzhFUmljbXdZcXJTdGk0Wmtpd3BpTEpibFd6bFMwejhJOEE0RFdjdHB0ZWlk?= =?utf-8?B?VWsxRVorTFlxUWJBTTM2RFErejZiT1g2QUhqNkRLaWt4a25lellQQUJscG56?= =?utf-8?B?bWVKeklmNG8rTWYyZm1VRGVFeSs5MjMyZXZyTHNrbVV0N3VKY2VjN3l5UFJj?= =?utf-8?B?YjVTUUhaU2xhSHdEeHlMREVzYURjemVSdlFlSkJIeU5BYjlqUUhTaFNPdFpO?= =?utf-8?B?c1YrTkR0WWp2SUJzd1NPV1FFWlM5MSsxb0h1VWg5bk1ZeVRWd0ZTRkVWOVZW?= =?utf-8?B?UUlCdkpPRjZFY2sxYTY2V05wQ2pTLy9DUFIyTE5lU3pGU29FR2RBbnlBbHBx?= =?utf-8?B?dkR6TlVXYmM5MmUxYTJNMUN5Y2NkeUMzR1A0KysrcGJsVE1KaFh6eEt4RmJT?= =?utf-8?B?T0JucGNhcjRMQW9oS05RWmFNMmU0eGVSRWpNV3ZsZmpXSHVMNkhYbjRXYW5B?= =?utf-8?B?K2lCMndXbWdBSERlSDVGWk1aWHJhZitGQlZiazFsOUFpNVdSRElIQVpJbUtU?= =?utf-8?B?Z0pjR3l6eVVxVDAxVkFDOWtYSnRRemdCbjBFNWFxV2l3OTl1OUJ1N1BibFl0?= =?utf-8?B?MnpYQzBDU2ppTWdtdVByNGxsZDFRUDQvVXpLbXIyOXZPbDJZSzBDeXhURGY3?= =?utf-8?B?YnovQ0ZvU1JTcHZzMFBseU9PN3ZoQlZodU4vRWR6T2ZCUGVVRmJuempydkJx?= =?utf-8?B?a2hpUG9sSzZieUtBTjFSVTVhZFg1UDg4dlA3aUZkRUNRYWp3bHZnOHh3QXhn?= =?utf-8?B?TVlzSTJIVEhTS2l0Tk9SYkZTYU5zMzZtckplaFZsVlVDWFpENVRNL3dSVkdL?= =?utf-8?B?RmY3UXo3a3ZhVkZJVWhkL2tiNmNGWDkyNlRCaFdHWExoZUFpL2d1MXZmMzRD?= =?utf-8?B?M3RDUDdCdGhNU1gxVlBwd2dZd0RGOXUydCs1Vk9wNWRQakYzWmJyNnV4alVh?= =?utf-8?B?cGdNRnNFeE42cVRjTXlqejN0cFRudkhjVForVHVmYTVsY3c0WlBrNFpsanNY?= =?utf-8?B?V0lMZHRiLzZneS9FcFRydzhiMzlFM3F5ak1lOG9nQzNzMmdsYjd4QkI2Z1Bi?= =?utf-8?B?ei96ZnN5MjhoSUo4RnM2azBRRTJoWkFnWlEyNjR2WDZEekYxYm5xZDlkWG1H?= =?utf-8?B?dHIxUUVhTUo2UW5YTDVQa1pDanZVQlptcnRqUTBBUWVOV1RPbVFna1lScU5G?= =?utf-8?B?d29jQUZrSk5TMHFjTXFOMmVmaXhvWUcrTythTkl4VWQ5UDJOd1FlMEtmbWlV?= =?utf-8?B?ajdwY1BTU2RPSlFrcm9MREV0Qy9HaWNDOWJjenpRNm5sL1A0ZE9ETU9CQU84?= =?utf-8?B?SUcvM2NXVkExZVpsbTNJRjQ5ZnEyR3ZWY3FwZWgreUNuTnRTdXRwT1phckdE?= =?utf-8?B?SzNuRFRuVnRISmtyYUtDblpqckc0dGJHM0MwcXREeVBqM2NWYm9SS1ljRVNY?= =?utf-8?B?UE9pY1h2enJsckVkMmNVU1VaeXpFTEp0K21pTlQrdEZ1N0xaaDhVTVd5d0Fy?= =?utf-8?B?RjVITGVNVDBoMHZlb2k1VEtzNkw2dnlYdTI0OWxQYlh1UTZpOTJLYXNWWVpE?= =?utf-8?B?T1pxazZ6N2dSUXptVWNnZ1pLTU1PN1RmVEFZaHZIb3lLci9sU1l4ZFh2cjRZ?= =?utf-8?B?elRoZnRGbUpVMVJlK2FvQVpOQzBnTTcwUVdCOG5KaGtSMCsvQWViaGFlcVVi?= =?utf-8?B?b2Rkek9FMUc1WFJzSHFBZ2c3TW1JT05GTDY3MjZVeVVCREdtTHVtcFFOdTk1?= =?utf-8?B?M1FnY0pCb3ExME9POU1ZMW43WDRVSUlOVk5oTytEREN4eW0yclo1d3R5L2Ns?= =?utf-8?B?bnVXSmFsZURqVGprSTFIYjVhQjAwclFKSVFHWW4veHpiQUFkV1ZNM2JCOUNK?= =?utf-8?B?UWZjRlNjc2tOVll0dm1SMVFLVWp6QzFCZ3RLSExRNTdIUlpuemszc0JXVDBi?= =?utf-8?B?bE5QMS9sUVdWOWs0UmEyS05UNjZQaENoQWZpSy92ZmRVemlvZWF6UzUzeFVL?= =?utf-8?B?QnJNUWFaT1JHQ0N5UVNqRHUvK0tRU0tHd2ZvMGo0RW5XM21xZ1oxVktRaHM3?= =?utf-8?B?eFhDSnVHbkQvZHVjKzFIdGxBZndvaHRPdDdUQnZCNjdpODRIaTdjRldkVytS?= =?utf-8?B?aEtWS1J1dm9SWTdDTDZSME5ubUlhbDM5Ti9PK05YRmdrOVpLM0NpbWlsU0lL?= =?utf-8?Q?qy8PYfaz/1SYaCnmFTsMRbT+w9FV+01pswpT8f+Uw1d5?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7b2e9bc9-60c4-489d-d866-08daaf407e61 X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Oct 2022 06:34:36.0549 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 5u89X/woFV6Fc5l3Mj8dxVTuHhYyUBd9ASZ6DxHk167wY84X0AnyYh5lIB8kYNRWwHJNWjU2EvfWuMje2JUnMQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR04MB9048 X-Spam-Status: No, score=-3029.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,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 List-Id: On 14.10.2022 23:51, H.J. Lu wrote: > On Fri, Oct 14, 2022 at 11:27 AM H.J. Lu wrote: >> On Fri, Oct 14, 2022 at 6:46 AM Jan Beulich wrote: >>> On 14.10.2022 11:12, Haochen Jiang wrote: >>>> @@ -8480,6 +8609,70 @@ static const struct dis386 mod_table[][2] = { >>>> /* MOD_VEX_0F38B1*/ >>>> { VEX_W_TABLE (VEX_W_0F38B1) }, >>>> }, >>>> + { >>>> + /* MOD_VEX_0F38E0_X86_64 */ >>>> + { "cmpoxadd", { Mdq, Gdq, VexGdq }, PREFIX_DATA }, >>>> + }, >>>> + { >>>> + /* MOD_VEX_0F38E1_X86_64 */ >>>> + { "cmpnoxadd", { Mdq, Gdq, VexGdq }, PREFIX_DATA }, >>>> + }, >>>> + { >>>> + /* MOD_VEX_0F38E2_X86_64 */ >>>> + { "cmpbxadd", { Mdq, Gdq, VexGdq }, PREFIX_DATA }, >>>> + }, >>>> + { >>>> + /* MOD_VEX_0F38E3_X86_64 */ >>>> + { "cmpnbxadd", { Mdq, Gdq, VexGdq }, PREFIX_DATA }, >>> >>> I understand the ISA extensions document names the insn this way and doesn't >>> list cmpaexadd (same for other aliases), but I think this is a mistake in >> >> Lack of aliases is a bad thing. In any case, assembler should follow > > Oops. I meant "Lack of aliases isn't a bad thing." Aliases make me > wonder if 2 different jcc are really different. Have you taken the time to read through my question raised in the forum? There is a reason for there being multiple names for a single condition code: What is meant depends on context. As said there, JZ makes sense to use with e.g. TEST or SUB, but it doesn't make sense to use with CMP (where JE is the appropriate mnemonic). Here we're talking of only CMP (an advanced form of it), so not being able to use CMPEXADD (for there only being CMPZXADD) is an issue. >> the spec. First of all the (early) spec should be sanitized. I'd be surprised if there were actually rational reasons for the choice of mnemonics which the doc currently lists; the set looks rather arbitrary to me. If the spec cannot be relied on, I think it's better to defer implementation of something that's questionable. Plus: What harm would there be if gas supported the full set of mnemonics even without the spec listing them all? It's not like there's any non- negligible risk of these mnemonics later gaining some different meaning. Jan >>> the doc. I've raised a respective question in the ISA extensions forum: I >>> think representation of conditions to check for should be uniform among >>> insns, and hence it should be "ae" here. (That would also be the effect if >>> you used %C here.)