From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-eopbgr60054.outbound.protection.outlook.com [40.107.6.54]) by sourceware.org (Postfix) with ESMTPS id 070F0385DC09 for ; Fri, 26 Aug 2022 09:26:48 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 070F0385DC09 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=G98ukTjFf+UkpTQ/FCFcRb3k4C7DUOQv6jLdTjVTQIjVcKOz85NF8SccDRzYCQOkc240GtMxVD2vRI+e9OYfmDVdE9CilmFqqiRqRpIYjlRg3jYayEPMbuPhPwKJBdZVdrFf85dnR0pGtrF5lktGtWeaHnUtE0+lvj0NT3CyfVw+SkdBJB92iyDA6/s5zs+5EUlKo9lZCe13NvZZ491006bNz4zCMLnxxykL0IonRblbsSGJCGpgXUZ+NumsYBHqKmONMHse6jBadWRHDBKl0D98f1shls110TJabqn0BA94tQ64RsMdygwRqGXts6mFw1s5CsgKveZKrefGmhdbfA== 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=SjySqUWxbJnykor3CZdp1NTKzOBoDORhhdB6EDtotPY=; b=Lb9CUmTYVMsTVA7qw3lwmeKMvHn9Rk2dJGnIRS/MQ0mVz5TafpfV+UL5aHQdBo3zo0wBkslVRmlA1HknXVI3r80ScwTiR8XluBbbTOEc4jT5QkUfb1o/s3P/00WqcQGhZjxGzc7Kb3wrVn6i5nlIeRJowiPLusrPpIykpPBxlRzyBbA/w6gXvAh1WwIl1CTfufq/zARy07xBpJiSs8mDHWxUvLfcPrHfKWPGJSxgnZp34gz8Ut35r2nMZe8tXs3HUS3j5mOHZGCZfjUtlVl9DHUD4ooIDedDXcMD4KghSQVEipGjO95bhBZlnCnIkSrQuA06XuyXEs8Tafm8XSvGCA== 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=SjySqUWxbJnykor3CZdp1NTKzOBoDORhhdB6EDtotPY=; b=2gBXownnj2Gl5dydk4q1Lcmh/pOe1BiWAIvSZ5P60awaMLekFY2QW/CU4Hgd5j2ndC0sKRS2IOe/fVrxc7tRT4XYENnAWJ+AsQWrNpv3Opz7YEIxrGuO5M0V1zYhfFLfofzvyoD2+Y0DshCLhJ/rPtGv4YwcfUPTmAdChIhD/BasFgeBrt+NMB3ZOXntkKZGZI89eZh2igBmB+3LAKKdOofIRKIs03VGmyk7w+aZcpMmV/I/lq4pdItFuvTbCNWKQdVDVrtgqnQ7FnjVFw0t+OGEaQqsE6n5Z05NZCnZQqRCalBjKT4Wt7sAcHSm2ooM1bOUaeAGN5ee+sxEKua/gQ== 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 DB8PR04MB6972.eurprd04.prod.outlook.com (2603:10a6:10:11c::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5566.16; Fri, 26 Aug 2022 09:26:45 +0000 Received: from VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::2d5d:bae0:430f:70ad]) by VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::2d5d:bae0:430f:70ad%4]) with mapi id 15.20.5566.015; Fri, 26 Aug 2022 09:26:45 +0000 Message-ID: <162c053b-7c68-67b8-1443-926f2ebb321d@suse.com> Date: Fri, 26 Aug 2022 11:26:43 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0 Subject: Re: [PATCH 5/7] x86: re-work insn/suffix recognition Content-Language: en-US To: "H.J. Lu" Cc: Binutils References: <32216291-fd1f-4579-87de-d24cb7190894@suse.com> <995353db-27f3-ea60-8e69-32bcc44dfd49@suse.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR3P281CA0010.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:1d::9) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 130df31c-982b-460f-fd20-08da874517fb X-MS-TrafficTypeDiagnostic: DB8PR04MB6972:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: YOz8obAuWX6jCukNuZo9iE0SvhbqhcLVuOtEVnuRqg3K5jOXz7S63MN0VnNCDYcDqsoImlTfAOmw9SETq6k2sIDbS9Vj4HWEVKyow6/gknkcpB8epEuzsAog1F3lkb+ZzTDKpsM7wuy1ZNSEsBK7WYdk5TED2neF5Moh33mZSnVWCElJMOmlQ63evhuY/OlHFZnlvR5XdmqZXCrM2z5HiLzBHqJ7io++eS0xuv1UE+JJCM0aZbSXqcAMTZrX4eZRN8MhHl6YLwslQkmpy9mmNjNWkDozptJfMyUVV9WFShwoGPEGHD1D+kg6HAXBEFMDhnqQWHbpT2E4Hry4RSGuiqOcddNIiP/VeXFWddr7pq1wC70Ry8WFFmzvY8rqSjA7fjo0B33VAI3ZY+vx35XNtKFDlXHZe+xDoX69FCTSQD68ZkU4AXRBmZcs8/ZEtk3vDbRKvW8p0Et3m9iPRDWX/qizahBACVKQwFfo96cUY3aYB4b0JXd3pb7vjq0OOFyZLDWFnjnc7Sp1yqBlD+VrtBgDQKbKUlfgn45SNZedWeUh0VM/jPc4SKYO2SG4vtFMaFtiLuljIFcdc2ttUHbl2b5SPG0n5DsVp7bXjX7WeLpSp47dSUSIa1HqrccSr8JC2UZ9+3fF3G+vxB7ARdagliL5NreMvEQ5bHl32NFyjI4EFgbxNpm6TGc4k4qvL4QLjHtfzW8zLVpFskmqfwQZng9x9WORpGGXt6FJGKRpyIKc2tl98/IXj4bWqLpGhYbhrERtg/KLdZ91zmm18nUmhcZr+uemohrhvKrftja/pDc= 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:(13230016)(346002)(39860400002)(396003)(136003)(376002)(366004)(478600001)(5660300002)(316002)(38100700002)(6486002)(66556008)(31686004)(66476007)(6506007)(6916009)(8936002)(4326008)(8676002)(66946007)(41300700001)(36756003)(26005)(6512007)(2906002)(53546011)(2616005)(186003)(83380400001)(31696002)(86362001)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?c2pGSmgrc1l0VitsRTQxYnJaa1lyUWRhYVF2RVJvQU9qRjVTcG1ZdS84STFa?= =?utf-8?B?QkRPS3l6eW1uNDd5Y21XNWdKaGFPbGFVNlE5aENhWkFDSE9kT2tpalFKYWFZ?= =?utf-8?B?cEtlWUxSYzhidkVPQk5lcmlyM01ycWJTVThSY09hMUMxdW5CTlVCNiszbmdl?= =?utf-8?B?ckVQNXRiUUJSZWJ4SnZyNHdjUFREQXZ1eUxhakFhK1JUVlk4czF6MmdtNnB0?= =?utf-8?B?cDFvaDB0c212aTY2dkd5T1pDcnFEd0tOa25pNWRvNnNVY0c2Z3FkanB1WGtG?= =?utf-8?B?THRVV0NTR01maEZDTXJTNzFYRlozM0tQVWQ4bVhFTmJURjhPK3dzVEpOaTFp?= =?utf-8?B?cUJnV3RjTkdNaXROa2wvbGllLzlaZGtwVnJIb0hNRzdXVWF0R0pRRXViNjdZ?= =?utf-8?B?VS9rek12bXpiQVlWTEIrQWUyZlAyV25rZ1lEOVFNT0NWSHFGa1ZsY0VlUzZY?= =?utf-8?B?a3VXc3ZuV2VPNmlmcDR5OURsRnkzTWg0Y0Z5N3NMbThNamg0akd4Qmd1VFdJ?= =?utf-8?B?NGpseFZtZjM4TitKNDVieTZ2NG9PMTl5VFllZEJ5TWNXNEtBbG5XVm5QLzlm?= =?utf-8?B?ZVl6a1RzckVWV1hucVpSTCtLZ2FjTWFOSEROdE5pelFFeVZoUFdncjlvd2Fy?= =?utf-8?B?emhoZ1JVR3cveFVmZjFJZnEyVUErS3FhV1c3ZUtZL2J5R2R3UllmL3lxVFJR?= =?utf-8?B?N1FHcTZJUUlGYXBhVUJudHZnRkpLR0pWU2VRS0d2dlQ3NEQvZnN2VWFFQU1k?= =?utf-8?B?TWMxYWZwKzBGZm8wQU1LVE5XM3pPV1dwS2JwRlgvalRqWXQ3d3QzYWtReVVX?= =?utf-8?B?V1gyYWwyYWE4TDY3cy9Rc3RYc0pXTlpSOFd3Wi8rU1VWZjU4WG0vQVpuSmF5?= =?utf-8?B?bXVwbEM0c0FWNUhsYkFpZlNZN2RuZkZtb1hQOFltbnJENUltZ1Q4U3MyVU9I?= =?utf-8?B?R0ZHdWsxbGQyK1JIcTlRckdZM1pUN1dPZVB0cFZxM2V3R29DWHdIVHFXSlFL?= =?utf-8?B?YWdiakZEWklhNk9zWHpld0FLOEpZSFJzK0labUR1RENUQU9CclFFeUlHZVhn?= =?utf-8?B?c2N3OFVlTVU3VGJtellrekRhOGZhYnZnMTJZT2E1dEJoR2tYZHptUGN0RFZB?= =?utf-8?B?Wk5OcE9aWFpZTDRhUWE3YUwxdjBwSXBSSFRNbXMrS3R4NStzYWQzYUgxdHZz?= =?utf-8?B?dkV4amZFeEZwVGRYNkIrTEVwZEVsN0NKRjRFdXZudEhIOTBjVHU5Sm1USy8y?= =?utf-8?B?U3NzeSszRDhGSDAzU1RYWnlBdkh1N3JLTFlhY2ZJMlNSbytSOXFQRisxakh1?= =?utf-8?B?M3F3SmFzTTB4WUZiNUFDYmN5YVRDQ3N4dmlLUkxpZWdXZ3BtQjJNRklhczE0?= =?utf-8?B?bnhWbkk1YzNzTU9IZmg0OHdTRHdzRmxjRzR3SEpXR1VXekJmaDc3VytXQ3dI?= =?utf-8?B?NmJZc3lWcytIVEZVdzJQN1lReU5qbDNMY2FRTzN3T0o5aUNxdy96akF3cmFL?= =?utf-8?B?aitKRHAxNUNjU2kxc0l1bmVHVzk4TzlDSU5oMWlkWG13SHVOQkN5UFIrY3Bq?= =?utf-8?B?QTljQVUyZVJ0TlNnSjVMT0JVL3MwNTdSS3JLMjBiKzFIQWNaSlROR0dQNHh6?= =?utf-8?B?UTYrcnVwcjB6SUEzT1pBZGIzMGZWVzVYcUtXRUhEdm1jbC9PTDlacWxsVkFL?= =?utf-8?B?c1UxWjQ1cC9YVTk0SWM0LzdlVUk2YWhMRkxoQXpNWE1wUVJ1VU1yUmhBWURE?= =?utf-8?B?WVNiWFRzNGFEU050Z0gyalZzbW9HR2ltVFJIMyswK0hwZ3F1M2pyV1hhU0xv?= =?utf-8?B?anVWdVB0eTZjclVjSnZpckJwc3FoTDkxdjdTbUhnSmEvdDdpdDBCU3hKS05r?= =?utf-8?B?RktiSEgvMEZQbUV4MklETlRwZ2hhWlZacmRicTQ5c3JMZEtpbkdBQW81WDVD?= =?utf-8?B?WVkxT1JNRXZLRlNJNC9HbVI5U05xdS9lR296dEV1aEVrMDFPMWVuYVhlNWVC?= =?utf-8?B?b3ZYK2JURFlyWWJxUVZDcWp0VHdIY1RHTmE4U2VQU0k1Rk5HbTJIOWoveVFz?= =?utf-8?B?VUxHdmtmMEVIMEVzYWlLVDhhVFQvTXp0Q1YwWHZCZmRTZml1V1Z4OHY2NWdh?= =?utf-8?Q?ZeritXnXBwfJ5pOjHbJ6zikWs?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: 130df31c-982b-460f-fd20-08da874517fb X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Aug 2022 09:26:45.2792 (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: drEVU728tu1FiL/7yJHAZ9roCPARGXX0eooanEF36lBQu7ycS/YAtruSU8hUq4EPEiCmZEAT5l0BtH7V8IQThA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB6972 X-Spam-Status: No, score=-3030.2 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,T_SCC_BODY_TEXT_LINE 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 23.08.2022 04:00, H.J. Lu wrote: > On Fri, Aug 19, 2022 at 1:28 AM Jan Beulich wrote: >> >> On 18.08.2022 17:14, H.J. Lu wrote: >>> On Wed, Aug 17, 2022 at 11:24 PM Jan Beulich wrote: >>>> >>>> On 17.08.2022 22:29, H.J. Lu wrote: >>>>> On Tue, Aug 16, 2022 at 12:32 AM Jan Beulich wrote: >>>>>> >>>>>> x86: re-work insn/suffix recognition >>>>>> >>>>>> Having templates with a suffix explicitly present has always been >>>>>> quirky. Introduce a 2nd matching pass in case the 1st one couldn't find >>>>> >>>>> I don't like the second pass. What problem does it solve? >>>> >>>> It addresses the reasons we have various pretty odd (and confusing by >>>> their mere presence) insn templates which better would never have been >>>> there. If you have a better suggestion to eliminate those, I'm all ears. >>>> >>>> You can also easily see the issues this solves by looking at the >>>> testsuite changes. Among other things this once again is a matter of >>>> providing consistent and hence predictable behavior. >>> >>> Did you mean the error reporting behavior? I don't think we should add >>> a second pass just for it. >> >> No. Certain insns simply were not accepted previously (this is actually >> what finally made me think of a solution here; prior observations >> weren't severe enough to try to get past your possible opposition which >> was to be expected based on past discussions). And certain other ones >> were wrongly accepted. > > Please open bug reports for these cases. PR gas/29524 PR gas/29525 PR gas/29526 But really - what's the point of making me waste time on creating bug reports when fixes are already available? In any event, I'll be shortly submitting v2 of the series addressing these. >>>> Further this sets the stage for the subsequent two changes, which I >>>> don't think are easily possible without this 2nd pass. >>> >>> Does it indicate that the second pass is used quite often? >> >> No, what I did say ... >> >>>> And finally you've likely spotted that this is actually a reduction in >>>> code size, first and foremost because the odd maybe_adjust_templates() >>>> can now go away. Plus I think you realize that the 2nd pass wouldn't >>>> be engaged in many cases - it requires a template match failure in the >>>> 1st pass, after all, which isn't going to happen very often. > > There is a fixed cost to prepare for the second pass. A very limited one (I suppose there are enough other things which are more of an overhead). Plus part of the preparation happens only when the first pass didn't find any match. Jan