From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05on2046.outbound.protection.outlook.com [40.107.21.46]) by sourceware.org (Postfix) with ESMTPS id B36733858C74 for ; Thu, 11 Aug 2022 12:12:12 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org B36733858C74 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=m9sUH+pjL0QRnMLpXvMDob1us8pWSHsBNgW2bm/GA9whDxeV4P6CS5Wcr+cE06ztaBTlIfJA2rjetjBCrxZZttDBhQTxCqsiwxLFUaAp1BqzTiZUscsIoU8NFUN5drWU3oVI2lXl5y5jYFxg8hzQomLHV5pvI49Th3HI8bBgXaMHXHJ96wJaYKoCaYmcRU5f6U0B1gXBohuMCXhdvHSntPoH3bTc8EbbaMsyCYRwaTEpdbcdM32Wn+CO3ayu5gousb3ge1ziF8zTAWQOzO7MItiOGE6XQVT5NxCq4bn4fqIBLFzrvXISYMiU3RrAAbkh+J5/Bi/YoBAlduwNwxy3MQ== 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=ipqw7bri/Eu6OGw8Ruyzpo+qaq6Q8udP3XkQsBTqTl4=; b=Qu6lHC5vIYJV+0mZ5+y5WF8ih0Q6cryGcD0ZhJd9yiaKLcCKDrZKVvXhNGxlz/i1c986sAi9J8V8vgoNmIjaRf4RtIrhSW7yGQOpNBKnZsuv63tJvEVvhYYFY1lpXvtMSOlszKTPv+1ybFyOtM63cd6OIROENCmDPK47fnPfdrBNwpiL2LnOYhNiEI+9ptzzvJ2hzUJe3StJNZhqVOEYzRKd+5eKahgxMSjN2Bkanr3PS9etrjp9Zt8KXN5PqUAMXF6pedp3Tu7ROA1wGPBJZ/CkH7tRkh+j+7ubDwhgZxAIP6lpZxOphRqYKMaaS42VXXZF7LCsyzyyMmNpTQj/gw== 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 Received: from VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) by AM6PR04MB4855.eurprd04.prod.outlook.com (2603:10a6:20b:6::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5504.23; Thu, 11 Aug 2022 12:12:10 +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.5525.010; Thu, 11 Aug 2022 12:12:09 +0000 Message-ID: Date: Thu, 11 Aug 2022 14:12:07 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.12.0 Subject: Re: [RFC PATCH 3/5] RISC-V: Mapping symbols with ISA string on assembler Content-Language: en-US To: Tsukasa OI Cc: binutils@sourceware.org, Nelson Chu , Kito Cheng , Palmer Dabbelt References: <1659692183-5682-1-git-send-email-nelson.chu@sifive.com> <21270977a91ae08292727829cbee7ef230896d2f.1660201178.git.research_trasio@irq.a4lg.com> <96c38e56-6454-5eb0-c0e6-6652ae14aa71@irq.a4lg.com> From: Jan Beulich In-Reply-To: <96c38e56-6454-5eb0-c0e6-6652ae14aa71@irq.a4lg.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: AM7PR04CA0008.eurprd04.prod.outlook.com (2603:10a6:20b:110::18) 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: d4509627-1bb5-4550-0c16-08da7b92b731 X-MS-TrafficTypeDiagnostic: AM6PR04MB4855:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: t8s/xEnFUdD9lLQa31GLk5AGe78dwLjHhE9UWe4OS3LqIhaWfoSSDjcuA5YU7WjdPoNTbuOeFS+qYiOKY61gzRId9G2ljp+8zCaMUh34oJegatgrtZBgJqVB3BTzhK4NoNy4XbdTfERGm7pLQLpIS57eA/4CW3W1x7Lq7YKFjS8Qjs3a+/kqowpBzdsdYq4pwc1M6SFt4LXVhzl+3dmfCvVnqZSqVQQ1KtPqCfNeOhYNLZ09SFaHPp5NVEx0uKYiFbOYXYZnbBQSnM8gB2urnYqV25jdtjUY/fkyVoY0ETvw2aYKZK1Cqi06o9neR7MjHFrjEFzRwYcmneewqSNEPkoH6/niiDUoVFUhtQDEWLjAShWpis67aKhx8XWBIrbXczXeEUb/cggF9nGapeZzh0eJNMeXs6DjOXqt12JRKuxh4UF1Q5dmQc5rvWGaBLmTgbNBTNluNSPkyqcm09RoDmYds5b9ckk5Yyp6l9xo1fuNQqYvuv/phQt4v1KYx/zu902HEUvmt3lOuNpxNO+yPjEPoqjqA57uOqeDGcVNOyOxBBMmwHfzjiX17zXrExLdw/Ukj3/hoVYywAml7dryf0RUhSycFRqEHQFS6WLjfo433DmTGQ39q+5ePxHIz12yEIgbQ4C2WweYSmeYdEABA7N0LZMPr6CAN2qw/qFn9duDXJ9zdap458T+OxIOVQW8f5rpte3b/dkIVaQ9xT6OuGHnDEn0THfQjC5eGeT5G+ffjXjLMeLHM/5B+eCHvz74dDO3JJHvFaMHlGj1hSMhXU0pJ0rYCXXNvmj1d4cUnX96infsbY9Mhaq2IUwt2Wtz+5yTPdHw1J8A8DvET4OBfg== 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)(136003)(39860400002)(396003)(376002)(366004)(346002)(316002)(83380400001)(54906003)(6916009)(6486002)(41300700001)(8676002)(478600001)(2906002)(4326008)(8936002)(66476007)(66556008)(5660300002)(66946007)(38100700002)(36756003)(186003)(2616005)(53546011)(6506007)(26005)(31696002)(86362001)(6512007)(31686004)(43740500002)(45980500001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cGUzWmZQRGd2c09vK2pkbDBudXJQTXZ1T3E2bHl0MTJHQyt6Z1NwRmdDNnBF?= =?utf-8?B?bVBUTWhsVUthc2FXRHZlWkFlZ3kvOWE0M0hEY1pkWGdycDU0OWxvcy8yMGhQ?= =?utf-8?B?VTg5aEVQSmNiRXRoWjJpMlNDa1RKTFk1WXB0eU10TDJQKzNSc3RFbWRIQVZC?= =?utf-8?B?RGVycWlldjFQazg2VFAvanZMeG42U0dFSy9oTGJqZTltejJvVDdnc2hhcFpq?= =?utf-8?B?VzZMRzltVHJDZU9CaEhlVEZKenhZMWordzJja3JXNVhuUm94OU5MZDJOMEJC?= =?utf-8?B?VGhaMUVVUTdvTG9IeC9JdzZoOXRadHpTUTZFbXg2c3ZHK1c3MWpVeStBQWR4?= =?utf-8?B?c01kbE50YWFYb1ZYOUkxNkJLSjF1R0t2K2VlS1NrT2QwUTZLY21GOCtnKzF4?= =?utf-8?B?YTJCb0xHYnZkc2ZURzUvQ0w4dEdwM0VCSzhHc28wVkprL3c2c0gvVGQ1b0Nt?= =?utf-8?B?UFVZR0dPYlJTZXRaNDJ2SnRjMkxyR3BaRkpQSkVlcldLc05ybE5RK0FTSUpQ?= =?utf-8?B?TFNaMHVNOHJPRXkvWlprSFJGV0RGZVdQZmt3NFd2Qy90TFpLYTViSExqTjZj?= =?utf-8?B?eUU4cXBRL0NUdTA5a1hqelZhK3B2cXVsU2Y4d3hjQ1VMQ1dZSW5wczRhWFEy?= =?utf-8?B?c1o4Vnlydm10TnNjTGJJWnM5T3prK1NZTWNFMFV2cUZvbmxyV0k4OVpacVJM?= =?utf-8?B?QU8rMm5KNGx0bjdqbkZhc3JidUxtZUR0cXJvdzdNRTIxNkE5MW5xekZKUzM0?= =?utf-8?B?OVoveUpJMnRPQkdmcWpZVDIzbmJrZlhWNzJud003eXJwQmtrd3kxNDZCR1RI?= =?utf-8?B?eUlFZFlqMkRBcXdjNmN2aVpKbUV6blJBVllxVVBDRjBBeWZaZ21sR3dQeWFu?= =?utf-8?B?bFhvUDhaaFU2Z3QrVkR4ZnRPWVNhbkE0dTFCcE05eHFWdG1hQzdIWEljaVd0?= =?utf-8?B?a1p1VURlbkNFRFdpNTdISXkyMHJOS2VjL2Qxam5NZE4zakQ0VmRlZFp3ZDNz?= =?utf-8?B?Q3N1RmxmOFlZb25qSFBFOEhtc285TlEzRTljck9qdW1WRi9aTklLeFdjd0Nl?= =?utf-8?B?eGxQTHRGYlUvUFpRWkY1bkxWa2ZSRG93ZXQwOWJ3b1dlZHRYRUxqS3JPWmk3?= =?utf-8?B?cEt6RjVLRHdUeGU3YlQyQWJYQkRZL08vbmZwbjh5cmJOVVUxV3k5SXBFZGVr?= =?utf-8?B?ZkpHYXhERHl0OTYyM1RtU2EvY0hjS3lBR3RYS3plZFNCOExORmZZNWV6cU9h?= =?utf-8?B?KzBPTzM5VmVkbGV4dENsWTFoeVcyak9CenhhYVpUTGNIOUNsYllxNU83NnBt?= =?utf-8?B?S2pUL0hQOVpRSGM2UW1GQXlOTC9lbXFTTlg2MStGYmRvRGw0bUVoUmZtb2dl?= =?utf-8?B?QjgydGVtR0VIc2p3amhMRndMa1UremEvSFBEUmdFT3h2WFFTK2Y4QVJyaHp5?= =?utf-8?B?SXU1elpLRnppTklrcXJkdFN5REs2UVRmeVhWakYzNEtDN3c2OWpBMXNLZWdy?= =?utf-8?B?VUc3aFVkdm1VaUw5VEJpZlFrZEFUOStCSUVJMFJJU1c2UGF4Y0xzMm5iVnBP?= =?utf-8?B?TGMxSk91dFRqZlpHSzBnbWVxZzM1Q3ZJS1BOeTFJQ3ZCclhtUnFuUGRnNGpl?= =?utf-8?B?b3djak43OXB1elUzOGFFbk92azUzYlh4MTNSMVl4cDlyVmJXZW1iNW1RNnRw?= =?utf-8?B?TG1kT29LQzA1dmZrMWdXNzVHRzNvMXoxaHE5dDRLeXhaQ0pBMVh0cUNkK3B6?= =?utf-8?B?S3AzRFE5USt4MnczSitha2RPbFprZkFkREUzeFNPWlVZc2EzYU1yMjR3SGY1?= =?utf-8?B?T3dCQ0VsdUNxQkJsdDhDOVg5bUc3cEtxaVhQNkdxUkw5Z3VFelFmN2xDU0tK?= =?utf-8?B?aEtnODdUMFB1U2JnR2xZNFB4VVQyS0l0bDJJeDg0RmJqSlhnZm5ZbUFCUnhQ?= =?utf-8?B?eERHTTRRTmFqQ2RhTDRmVWsrcVg4MHVxeHVSSndYSnl2RFpVZFVnZFB0ZTJU?= =?utf-8?B?ZmxiQ2VNVC8xUkUzeGNleWR0NGVWMkY5NDhoSDljdmo2a0VaZEZKMkdPeWM0?= =?utf-8?B?MmE0eVlGN1BicWE5VjMvY05LNFJKTWNpWStwZmNuajZ5a1NacDl0SWJUMSt4?= =?utf-8?Q?GIOmHaW1/XISXdn7jzT5yiaYq?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: d4509627-1bb5-4550-0c16-08da7b92b731 X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Aug 2022 12:12:09.7420 (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: UcABAqRHvZWZoajhkR+brgM0ys7sUU7diWZPzR+EQefiLX2OwO/PBjuRC1PQABzcw0VQpkA1Uc0JsoLYbK175A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR04MB4855 X-Spam-Status: No, score=-3030.4 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 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: Thu, 11 Aug 2022 12:12:14 -0000 On 11.08.2022 13:43, Tsukasa OI wrote: > On 2022/08/11 16:31, Jan Beulich wrote: >> On 11.08.2022 09:00, Tsukasa OI via Binutils wrote: >>> @@ -3855,11 +3887,15 @@ s_riscv_option (int x ATTRIBUTE_UNUSED) >>> if (strcmp (name, "rvc") == 0) >>> { >>> riscv_update_subset (&riscv_rps_as, "+c"); >>> + updated_riscv_subsets = true; >>> + riscv_opts.arch_is_default = false; >>> riscv_set_rvc (true); >>> } >>> else if (strcmp (name, "norvc") == 0) >>> { >>> riscv_update_subset (&riscv_rps_as, "-c"); >>> + updated_riscv_subsets = true; >>> + riscv_opts.arch_is_default = false; >>> riscv_set_rvc (false); >>> } >>> else if (strcmp (name, "pic") == 0) >>> @@ -3880,6 +3916,8 @@ s_riscv_option (int x ATTRIBUTE_UNUSED) >>> if (ISSPACE (*name) && *name != '\0') >>> name++; >>> riscv_update_subset (&riscv_rps_as, name); >>> + updated_riscv_subsets = true; >>> + riscv_opts.arch_is_default = false; >> >> Seeing that all three call sites of riscv_update_subset() gain the >> same extra code - wouldn't these assignments better move into that >> function? (The function living in bfd may make this difficult, but >> it being used by gas only suggests it might better be moved over. Or >> otherwise maybe add a local helper function doing all three things?) > > Adding a local helper function would be the best. Just out of curiosity, could you explain to me why the function is to remain in bfd when gas is its only user? Jan