From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70045.outbound.protection.outlook.com [40.107.7.45]) by sourceware.org (Postfix) with ESMTPS id 6B0B13858D39 for ; Mon, 17 Oct 2022 07:02:17 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 6B0B13858D39 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=OuU0nzjtIHmxfZLf6pzQHeW5paoWeTnTYeR1yDI1MzbX1GVa6q7xtMaie+Cl6UJ9/88V9v8V+nbBd3HtIVun5SAwV4Mi4MV6evkwsOpAYkYNEjsuhkIQQyZ6/aK3eM5bP7nO6ED+jjNzFinzQdQwFNUQcOs1ZFs1Hu9Hk/4FdQbwRc9FgPwcyTOkg0quUvvufRjA0WVl/NSMXOK1GN0HPeFeGdtrx6eOI2pPUO33w9YgDLHbUIhNJeDM0wty8YTLaGULNVvjLbXcrAUZ/Mmq8K5R0X7vdVeOFJFMH0TTZ8rjZHPAiP1wec9HWH8GiPIyW5WK9c58sOLF0RFcmGZvDQ== 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=E3FHI8p27r1jAH26Y0y6zXinQap1brq+/pb1/SeGpQg=; b=AOjCwvPkJ9zYJ/cZ5rIVcCKMQhZMg/VZeHb6/PCvWo9wMEC4WM6CzWMJViWR7hRB9XyiUNNBfee42Wqp7UHT49yQGv8kdUGCuJZEFSOPCy0SA92f0K80uKukQA+JX72kKizwzDvcXPphOi2EK3JI2rZko2B/MZnEcvv/m/gVYD5zfz6hxWRSLRQCakVK4XK4NTOdBScTJTLTXSoukGxJe3p66IDvthnGzEXlyb9iLF0bgm66rTrGXFUSMHRN+gVuQnEvGAW84n/LaDu8Jms9z1+ZITlQ00OeqzP/3IoYrEi7YMAHPC61o2oIbMcSR47z7FMAJKttBwPBD8QAcKa/9A== 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=E3FHI8p27r1jAH26Y0y6zXinQap1brq+/pb1/SeGpQg=; b=RiOqOgnwNO79YAV8CGqyihaTZ2nvCHShdYBVcJR7KSjz5trERs2NdE0wIeaArBaHF/hPXnrevGt3ZSUPkWF9jIIcld5pImWrURiWoBdaRaodIeNPnLbDAbp3AMpQIsjGa59JMbnFaFmLY8xgGCWglehD9lz02NYEboPMWQM6YYRibnYlA8Zl9FnciMXq0oQN9S4bwepw+PnDojbPmGcFHMG8/k8G6V58qm78zqyugLf8eiTCo+CNfeTcJLkVfU2o4cR83eQJXwcTYF3F+cttpmI/HTKzlm3WE+8YgekHFmks1HlMrxgVoT6GnqClUpvGKK56RHr6P3TxnZfBA37D7g== 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 PR3PR04MB7226.eurprd04.prod.outlook.com (2603:10a6:102:8e::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5723.29; Mon, 17 Oct 2022 07:02:14 +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.033; Mon, 17 Oct 2022 07:02:14 +0000 Message-ID: <8a1fe0ad-d5a3-cdb6-780b-6ac69123a51d@suse.com> Date: Mon, 17 Oct 2022 09:02:15 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.3.3 Subject: Re: [PATCH v3 4/7] x86-64: further re-work insn/suffix recognition to also cover MOVSL Content-Language: en-US To: "H.J. Lu" Cc: Binutils References: <20e2773a-2e47-869b-1900-709f8ad4cd6b@suse.com> <2981100a-17bf-623c-27fc-0da08279c3ff@suse.com> <32052f49-9789-36c6-fc26-a7e24f248435@suse.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR3P281CA0081.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:1f::11) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|PR3PR04MB7226:EE_ X-MS-Office365-Filtering-Correlation-Id: de77dd34-54fa-40bc-bb65-08dab00d857d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 9WjJ/Qczlj/ZU5woyORTGzl2Za7mZ6Uy/D8fmx/IZyWlvSVBikJMJVN8cs8APrVVxMdUKEffWE7PmtE7MSUUyGrGMV3049yUzR5H2kMhNrwMw9yvYyEpkOWiuFwowaK/ol/3oc3zeKJjP7NcxcPs5y7leWnI7FiR8ZYgEaF8zeOnynCmH9ncvx2zM/lShrH1UgVxLj0rGtBnghtYD3G2p/V+7olRPRUAPNCyLV7vuCJaQY57DciOIUureYa2Fmo8IkLQOAyHyvv9cQ8HRu8s69g0Zl9ewhzHdOULgcG0UWUa64wQPM6wfsX/1zO89h4bLaYxjxeWD21bVfGvim0tRzn0cD2tGB+5pwc8iRA957ZR3GDj+kFZE2fY1TDV7cKAhW2YYZ+8mnPFyGTYBuuHxMWCMykJTbPdvzxkGmGuGh2vc+dHBP9RhVCSmsDDg2K+YBcF5/D/w7BJmUDNsif4SJykQQbZL65PLhKGmDp6obOz8znB0Am5KQvxt8X/6VL24ksY40BGT23Fo1eb7Jd4gOEt32SsSZQ0FAUZbgjjmxa71qE+FBRgomePkRbvcqmRvMi7eZ6TKcyYMaDFdp/yP0S1dKK1G15Xn22FhhGLDavYhYFCupW85n9QI9N1SCpnvw1DL4goDGGJlae6uHXx9UvVvqJFXzm4epwwvCrcwWFxWi6HE5EcSWw+6OaDyd99Xaf3BgU9wXlV+z/FzOAG2K5nmoe25EjZ81hbyHlbsuZwKJy8e6ylCIwSM3hU+RReZMg4cQh/gYxnh59wh9GuEApF1DMsdi2o98R/I5GfoOg= 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)(346002)(136003)(39860400002)(366004)(396003)(376002)(451199015)(66899015)(41300700001)(2906002)(5660300002)(8936002)(31686004)(316002)(36756003)(6916009)(6506007)(478600001)(53546011)(66556008)(66946007)(6486002)(66476007)(4326008)(8676002)(83380400001)(6512007)(31696002)(86362001)(26005)(38100700002)(2616005)(186003)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TDNrMUMvU0pUZnA3TlY0SWFRdGx5VWFGQTV2STg2eU4rTlhkVHYwUXAzeXNK?= =?utf-8?B?RlQyeTBxMWxPUkpqN2hoN252S2VjTmh1cHVHTW1nRkRpbWY2cnhsT0dqZERX?= =?utf-8?B?c0RYYUxaL2w5NFhGcnBJYjQ1YXJuNTEvTEhwaEdOQm13ei9zNHpSTWIwZUNj?= =?utf-8?B?QlFxcGUvVm9oWk5mWXB2MXR6aUFTanZIOTI0RjdHaHVZVTIyUlRjV0doakho?= =?utf-8?B?WStxUnc1VklwTHNkeFA2eUM4TU5GS1ZmRnB0dEdaMGdNWE5mT0g3K2M5MXBV?= =?utf-8?B?VXNUS1pEUlNIcjN0WEpZbm5pVXNuMHN5cjROTEgyalIzSUp0ZnpySFJvVzZ1?= =?utf-8?B?VllkbGswdTRYc09vYkV2S09pTzVCa3ZQdW85bExKVDFoNm9wSTBOT05nQXB5?= =?utf-8?B?V2JEZkdzZlg4R2lCd1JmakRtdzk2NmY2dDlrWGhySVdMUk5BMTZidG10Zk8x?= =?utf-8?B?TThWSEdqQXVyaTFuTk1sa1hrYnZMZWVsQnMzdVZQN0hNUm1TdTJScUZwNjNW?= =?utf-8?B?R29xL2Q2R2JucUtBMGxXQm56NUZWZzVteUt5cHRzcVhuTnUyL2hoMUtaTFNG?= =?utf-8?B?YlJ2emRLZWg2eDBhKzFxVHRxYnA0dm9ESmZncGx1cGc0Y3NKVjRKanNsaEk1?= =?utf-8?B?Y3pmU1IwSnJJdVEvdFRuUjBhajdkVEhPb3hOeW5JV1FGOXdSY0xMTk5UQ1NP?= =?utf-8?B?d1ZKS0hVVEYvem1ZMGZxQ3poVitnNHJHTXZWVGM3eVBZL0ZUclJrcHA2cDUw?= =?utf-8?B?L3N5SVd5RGQ4RGxmUEFGZjIxejlLa283WHQvU0hGcUdwM296ZEdPWEo3SWZZ?= =?utf-8?B?Z01naTlncUxabDAwTWRQcWhVTkFreWFUVXZ6cWIrakd5NkU2Y2ViZHVjbUFt?= =?utf-8?B?eFJSaDdNVDQ1OUl0UVBQSmxYK3JpYjc4K2RYeVpIY0RJaHpXQmMrZUJjSEll?= =?utf-8?B?NWw2UTZFcFZyTmYxTlZFcmxtb2loYjNSeXQyV01MaEx1dGp6UzZCSFoyc1ZO?= =?utf-8?B?RHQyOGh4K3JBci9FNTFuWnR4ZmI2MktjWDJFYk5XUjc0VG5LTG1mQTlyRXNp?= =?utf-8?B?R2NHY3RaUDhNRzRBcE5OWGxjRTRRc0RNMFoxS0Z5MEpMTzE5MHIrZkhwQ1ov?= =?utf-8?B?NEJueVBORDhFcWNqMnM0cS93N1pLRVZWL24zNUEzUFAzRk5lNGZYYUZnYTEx?= =?utf-8?B?NDZMTTdCeDltcWhRVFFENEtjdVRGNnRYTUUrTlJtT3M4WUdYZUk1VS9ob3Ni?= =?utf-8?B?bXAvSDI4eURtN1FYZDQvZkNZOTRtdEQwdEc0dTR2MGtWVVM4cnhRbTVXL1N0?= =?utf-8?B?YTN1V1EvSTk4eGJUYnhrMmhXK0lvaDRESkhvU3duR0NLcGRHWE9sRXFMczcy?= =?utf-8?B?S053M0hPSmpObExTZkwyUVYxbnZ2bUpQaHorNkMra3ZjbkdDek9NdlY0VFVV?= =?utf-8?B?S05qK1N2b05pM0p2aG1jaHQwU3B0N0tvZU5UUDZYajE0SEhNdzAxZ0dYeGJz?= =?utf-8?B?MnFSVGIxZEtySHNoTCt1ZFF1OXlHTitDcjFDZFl4MXZpelpqcHhrT1Arcjk4?= =?utf-8?B?cktPUFBBWWZrZi8vMmJuejdOK3c5YnFLbGFFbmJQODByZ2krTm4wZjQrUmU5?= =?utf-8?B?ZVN6U2IwWTY5empPdXh4aEx2dGZFYnFodFRRK2VBS2g2R01PMFp6dHF3NS9u?= =?utf-8?B?RllhVktzZXJuT2NwY1p6OWd6L0FMcnErTFV3OW1JU3U2cXJZeDlkNTBPbVRQ?= =?utf-8?B?RHU0ZlFBVXFvTVVIVmc2QzBGaDlPd1ZYbkU2TS9BNmxlQi96NGoyNHJjQTlw?= =?utf-8?B?NlR0dUNHaWtrL0k4MFFDUkRrV2NrUmlQZU1ZWlNkVWt1YUh5aGF1WFViYTBU?= =?utf-8?B?QyswYytud0Jhd1Rkd1hYMGhOYmxWakRIQ1BWakYxdmVUYTNXVWd0a1FIbHhH?= =?utf-8?B?WFMrblkzcnM5cGhHdVN4YmFvdWt6bndDOGFZQWFOY0VUNS9MTTBUQWxZQlVC?= =?utf-8?B?Qk1rM3pBZ0ZRbVFSK3ExSDR5Nzc2anVFRGhIeklkckNjUUlHT2lxendCZWVn?= =?utf-8?B?VGt2dG1tWlkwU1k2eldZbS9hT3BLUitOeEgwSWpDSUo1aDM1M2tla1VPNXpY?= =?utf-8?Q?NQb0Sqzh1vlUdaDYhC4qA8Crn?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: de77dd34-54fa-40bc-bb65-08dab00d857d X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Oct 2022 07:02:14.7957 (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: /cqE7bPmNys8jTB1LkD1cqvZvUQpV13XZPZXa7H6ozEkeKU8HSkCqFiXxtdFsDMDkrWOtRZLYTWuBR0prlVmmQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR04MB7226 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 19:07, H.J. Lu wrote: > On Fri, Oct 14, 2022 at 12:03 AM Jan Beulich wrote: >> >> On 13.10.2022 19:00, H.J. Lu wrote: >>> On Wed, Oct 12, 2022 at 11:08 PM Jan Beulich wrote: >>>> >>>> On 12.10.2022 19:10, H.J. Lu wrote: >>>>> On Wed, Oct 12, 2022 at 12:08 AM Jan Beulich wrote: >>>>>> >>>>>> On 11.10.2022 19:44, H.J. Lu wrote: >>>>>>> On Wed, Oct 5, 2022 at 12:24 AM Jan Beulich wrote: >>>>>>>> >>>>>>>> PR gas/29524 >>>>>>>> In order to make MOVSL{,Q} behave similarly to MOVSB{W,L,Q} and >>>>>>>> MOVSW{L,Q} we need to defer parse_insn()'s emitting of errors unrelated >>>>>>>> to prefix parsing. Utilize i.error just like match_template() does. >>>>>>> >>>>>>> Since movs{b,w,l,q} are string instructions, integer sign extensions >>>>>>> require a suffix to specify the destination size. This is different from other >>>>>>> integer instructions. Since only the new assembler allows the implicit suffix, >>>>>>> it won't be easy to use. We should improve error messages, but allowing >>>>>>> new syntax doesn't help much. >>>>>> >>>>>> It is an earlier change making most of this consistent with MOVZ*; it is >>>>> >>>>> MOVZ is different. There are no MOVZ string instructions. MOVS has >>>>> different meanings in ISA. MOVS difference from MOVZ in assembly >>>>> syntax should be expected. >>>> >>>> You've said so before, yes, but I continue to disagree. And as we can see >>>> from the series things can be made work consistently (and imo nothing else >>>> should have been done right from the beginning). >>>> >>> >>> There are inconsistencies in ISA. >> >> Sure. But we shouldn't add further ones in the assembler. > > Assembler just follows ISA. Programmers should learn to > deal with it or use a compiler. This is entirely non-constructive. Assembler writers should get things into usable (read: consistent) shape. Plus what ISA are you talking about here? We're talking of mnemonics which aren't spelled out in any ISA document anyway. The only halfway official AT&T doc I'm aware of doesn't provide room for omitting size suffixes [1]. Yet that's a fundamental feature of gas, and elsewhere (recently: CMPccXADD) you're even suggesting to force people to omit suffixes (plus you've previously objected to the disassembler to consistently emit them in suffix-always mode). That same document was also only updated to cover 64-bit code in a half- hearted way, so can't necessarily be used for 64-bit only insns (it doesn't list any form of MOVSXD at all afaics, for example). Where not explicitly mentioned, their intended handling can only be inferred by using analogies. Nor do we support some of the odd (quirky I would say) mnemonics that are listed there, like xchglA or movabsbA (which is even wrongly described as moving an immediate value into the register). Bottom line: May I please ask that you take another (constructive) look at the v4 submission? Jan [1] Really it does, by saying "long" is then implied, which has never been the behavior of gas when a register saying otherwise is also in use.