From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR03-AM7-obe.outbound.protection.outlook.com (mail-am7eur03on2083.outbound.protection.outlook.com [40.107.105.83]) by sourceware.org (Postfix) with ESMTPS id C4A263858D32 for ; Tue, 18 Oct 2022 06:31:57 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org C4A263858D32 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=kZA+yIznnJTlzpeXUab31kGI5jNr8MN3C58k+2xQkp0FVLwKy88/x5r/N1wv+SNFb3lHRadbCwxl3EFWMaAKbmpo3UaO3yvw5Nhbsb+qoi8cUE992tT7cZgjHVIyAGFQ9SAFvL7i/4B1NFUrGAUy5oQ1SfWGDyOgDoaxyqWKKL8oy2FiutAq7qHl8QW79Kl+F1AxVObNTap6ueWMjSj6VZJHKh16wdcfGK8FCuTCcPtwzATIxj/31epZUz49kU8TwBdJYmwTGXnhqQzv94Uwrj9myOILg3fEdTkIbWiX40cemDS89QZnaorE8sR9SRTwYGQqfiERnnxBJdQjhn8p/w== 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=IEmIeuyguv5iN3QM5UfZ0S8YrlbR62SC5aL8eHIDD0Q=; b=U7si/wqKJv908xRtBBCwFA7cHLPOKvnmA8UkgLdzVBc2AYdlOBgtRNjAVWHa5V3WwvRoOnGkUw9GAwc6YnBGksG2I+fqPE22Yphwb8nK9a7wdHdzUgohTOWb68VdElkszBXdR/7AK2HsAuT/0DYOPGj904o4grODYcisyasGTQoNVZP5b7mNKogToP1ZiB3hFQYbQUApdXUaooo3+QLk5aRTVYp9x+dAdw5KVr6A8aPvYKfyYpWg7ncpu/EWK1plcecMgJZRGWg8rvrs1FgKkc+CWWmWw1xmWSTqqPECTHbHpW9OsKPyw6/+bfN9tS1IsfjBmVUH3Hbw9huQ6AegKA== 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=IEmIeuyguv5iN3QM5UfZ0S8YrlbR62SC5aL8eHIDD0Q=; b=0c6bv37LcsQCI8MJ/HdaCRlgDqNjSP6FvMsExSqUj+6/4TJO2RXgbXmGIbpd4vdOV5Jf9T5loAGEZp+koYUcB65+SufzThnw4BeQQ2StvC1fhtfvmhNIRD/jLuJBwfcYGFhElCr9A1QxuFaWZB2gFW+/VkAO6pwNTJb/IfBs/VSHjtccF9xIPcKC9+N04LRvrxAaBeF0pEr6R9n1PKsM4B6qOPGN05f0knLxEOPGKH0McLk17QHXFc+sg2x35vab2GW7dpsZ95K+FKLB900P5viM5xFPKhM+pDNez9jvcU33hBZVoD+uylxZdHHriWEJfZAYkKAwtFG47is0nkyzNw== 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 PA4PR04MB7952.eurprd04.prod.outlook.com (2603:10a6:102:b8::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5723.26; Tue, 18 Oct 2022 06:31:52 +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; Tue, 18 Oct 2022 06:31:52 +0000 Message-ID: Date: Tue, 18 Oct 2022 08:31:50 +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> <8a1fe0ad-d5a3-cdb6-780b-6ac69123a51d@suse.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR0P281CA0082.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:1e::21) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|PA4PR04MB7952:EE_ X-MS-Office365-Filtering-Correlation-Id: 2caf15b4-f48e-4c2d-38e2-08dab0d271be X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: M86RY1fq66kWbW0lJOzysCUOXw1p9U/ChhRIQOdjLefEZVBTqB1RZBxk3KjOhzIOp5u0IHbXovE4AVEsRADXpEGLYCkOB03cCvtIkMpDvVK3w/OZbm7nmKAAHC1zPFLG2GiWm8uBd2jP4f8gRXGdiUukPP7rYJN/zrt1yXC/3G0j9nmmCTZZyy5/BfEWy6uy0Fwla49bqSabk49S7gZbtHY0kZfmw+bNbjRMOj5rWNjVwh+6YXwxUZFcwTsZNpT25MWdEpN5QoqP2mczkz9n6AKBFNOxbN6CJMNmp6xCQKd+wh2sFbX4vCQjqrVrl3XaChcmJG+IhC4O/iget8WD40i1utSzhriFN1lNYzJYx+5DyykJp6EY+GWGVOplCzfVMXPvd9YaN69B8B6rRlG8EVRz1a6JLg6aNHgEg8msddsxyztrKjq+yCuNy3o4f4i3/cgwoDDOmKCKe8NUF2r7JSWHMkgzjAHUEksWj7qtdJC39yYLkCmfGBCtbSKy5ot2RXMAkGRByNDcnFdrBcF5uKCLXk+0DYi1S7FxX96czaCGs2lqlirHvv60XR9azQjWWnC6jG8hBG6plcMHhJh07QaUJTRpg7bShqKpF1jEzm5i3Lp23A9Z6W0hUazymUs4UkH4Ins25xj4HQposWe6i8GV5E8tU7Lzx8bwni8dqRs0vu5vYG1gET+mkoAEnPK0TCaOTfQ12YzIzCqVU3teu3zCmw++EChRz4h/x0eW3Xl53H0cDA/3hHI7xVs7IVkihHJRy8uYxVASj4hUS3TebYuTtmZmBsR6ZMQxOYywV5I= 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)(366004)(396003)(39860400002)(376002)(136003)(451199015)(186003)(316002)(6916009)(36756003)(2616005)(6506007)(4326008)(26005)(2906002)(5660300002)(8936002)(83380400001)(6512007)(53546011)(41300700001)(31696002)(86362001)(66476007)(66946007)(66556008)(8676002)(66899015)(31686004)(6486002)(38100700002)(478600001)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cjdSUjJOeVJYQmZINTdPZHNiWGwzcktKUk1scjZyeVExUlg1MWo1OGU5bzlz?= =?utf-8?B?NmEzYlNRaXphTUhpbE9FRFBrbHFFaFEwTUtiN1lFdFV6K2htZU91bkFmRDlW?= =?utf-8?B?N1p5VGRzQmlydFJJcFpXd3JpWnhyZUJJN1BhUEJOakpGejZoMGprUUJzMEs0?= =?utf-8?B?SXI5bm84Z3gwNnZ0WWR1R0xXejUrTVpUN2xIQ2tqanNYbjNKVlRkd2NLdXdq?= =?utf-8?B?Nnh4eW95M0FhSm53bDFTZjFsUzNxRFZjT2wrSGx3S3YrWTJoL2U2VkgwejlM?= =?utf-8?B?YlE1VTNZWTR1NDdqWkNPTUZhcFVyazZmYlZ0R0ZQZXNyOXIyYjBOd0dScDky?= =?utf-8?B?V3A3a2xweHBDYXBtT1ArL1pndEc0cjlReGhyYnlTdHZ4U2puTVh0OFhoMFgy?= =?utf-8?B?eHAwOHk2U0kwOERyVDRDZ0hXc0dSQkplM0lzT2ZzRUhXUys1TEV2bkkzL3BI?= =?utf-8?B?UzVXL2ZMQml1ajdXTk9wekUrODZ0WmE5dTBlN3E3M0tlL1pBNW9MdjFIZlg4?= =?utf-8?B?QmdOSzFUeWx6VElIZ3l2RzJNVmwxeElhSEdlVlM2KzlvSjNrQ3I3bUZ2RnFX?= =?utf-8?B?cy84RElvTDUxTmdLbmpvWkVxSVlQaDBRSit1RFJBcU9ZYXc0YmdsdkxsZjFX?= =?utf-8?B?TlMyQ3FnS0dZemtucHVmdy8vZ2R6ZDdHQlFpZG5aaUwxL1BpTzdDRjlpSy9P?= =?utf-8?B?RUxmaHRCNXRoTDBaKytYYTNtd0dUQVBSZk05ZVRlRFlzS0hXTzduSlIxT3Qr?= =?utf-8?B?ZVVkZ1ZIVVlpWlpvOTA4M0NzWjhyN2lZbUhFN0ZvNDJHVldkZHcxNjBPUXRN?= =?utf-8?B?K0x0K3RIZ0RYSFh0UVBKa2Q5YWl6dHJFTzA0cEpaUE95ZHJvdUxuc294bkUv?= =?utf-8?B?WDh2UENGbGxjZ2JQcS9uWlVldXhNV3IzMVpsTnYwWWFHK1Zyc0FSV01xSkRU?= =?utf-8?B?Y1ZMeEZ3OEtLNlllU2tFUGtmZ3UvMERZdzZka1Q5dGdQVUdRaWg1NGRFbUVm?= =?utf-8?B?VzJ5NWNIWEhxR2ZHMml6bUhoM3llTFpJM2VsWkJ2T0hRVmNqdzc1T0tGVjBK?= =?utf-8?B?WmVIQ0IxeDVLOU15NVRNTFp1bGlzY2p3TzRmelhjRmF6elB0dHhNOFc2TG12?= =?utf-8?B?MWduQXNFUzUyR0Eyb0IrdC9COE0xYnJBRjJkUS9lYjNGa3NhemphU0EvSDdw?= =?utf-8?B?NFYvZlNVMGNUNkFmQUthWEFzSnVsSy9jMFI2eUd4U3ZIMmtKbjI3SWtWYXlU?= =?utf-8?B?cS9MT1RLcWhsajZHTHg1QWExZmZXc3NQcVdESS95MFRBdS96QVhDZXhId0pu?= =?utf-8?B?VnJ2SElsSFU4ZWM1OUtTRVJsczBhSUUzOWdYRVR0OCtxeVNmTit4d3VIN2sz?= =?utf-8?B?T0gwOWV2MlhGL3NvRjBPcmZycmdWLzZJc2h3NjZ5MmhTT0xXTGx5OWxUelZB?= =?utf-8?B?STFCVGc2eWwydDUzRWQxaWZ3WVp3QStGVWFtMS81LzhaY0hudG44REVwU0hY?= =?utf-8?B?UXpjNlVmOEk4eWpicDE4eXorU1JWbVlHYVZkZTJHOTdVY3JPTlVKTStOR2Z1?= =?utf-8?B?eTNDYWs0NWNmKzFSTm50NWZUTXYxalZUY2NzWnNka1MyaTNrY2dqNGVDVENn?= =?utf-8?B?NXUxM1ROZ3N2VXpTbUJLNUNPeTdBUnBpK1kzTkkrSWlBQWlqQ3JJb2IzSWta?= =?utf-8?B?MWNDR282dEJ3emJNK0ZBdEd6ZUIxQ1VuOEFiUDVxOHAwZ01NaHozM0g5Z3M0?= =?utf-8?B?OXNRWldDVlZVWlRrYURJaXV0akZ6MmtKcUxGc1Fpc01aVWlVVGl3dWttNHpp?= =?utf-8?B?Q3U2cDBraU1qcWo5ak1BbzJRSk9rVTJ0WCtabjZHR1FETUFVQlpTNm5zMUNo?= =?utf-8?B?c3NneVozVm1QdEhkVkZ0dUU3UEJBMFYxYkF6SHF4elBRMzZSYmhraE5iSDU2?= =?utf-8?B?VUYrT3RLeDk2c3ZiVFFNYm1EcCtLOWk4cmpyTk9DSnd1NGJBb3pvcStGMlhh?= =?utf-8?B?U1h2VERpZ2RyZmw4NVdCcmVtakdTSmY0YW5jNlBWNzJzL0JBSjlnL3Y3azhq?= =?utf-8?B?Y3BCcTM3d3cwV0V1c05qbTF1cVJQWHVPekp5ZFdwaWpYRnUzWWlwODZZU3pK?= =?utf-8?Q?EfaS6gzTZeOX91nQ7whJx5bjt?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2caf15b4-f48e-4c2d-38e2-08dab0d271be X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2022 06:31:52.5508 (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: fmZVXyY3YXi4pvNJnvaRY2vgITarTElUhxj4cuUPy1o0nfbCVQUQSn4hUWx22uwOnwg68qRpfEpb0+t2NEVH8Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB7952 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 18.10.2022 00:36, H.J. Lu wrote: > On Mon, Oct 17, 2022 at 12:02 AM Jan Beulich wrote: >> >> 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? > > GNU assembler has been this way for a long time and the current GNU > assembler will still be in use for the next few years. Assembler writers > should know about all these. Hmm, so after all not any ISA to follow? Plus do you suggest there's only people having written x86 assembler code for many years? And there's no people who would prefer to get their code into more consistent shape, but who are limited by assembler shortcomings? In any event, ... >> 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? ... this request continues to stand. Jan