From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2053.outbound.protection.outlook.com [40.107.20.53]) by sourceware.org (Postfix) with ESMTPS id A27493858D33 for ; Wed, 30 Nov 2022 06:58:36 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org A27493858D33 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=f5+CyqWmQfWPOklReu2xIikwpH988RdENq/qFxEjzpWzzo6ch+VE37Qjy7cmuAp4iGTPSgIdGolPqpJJNs21tFXQhQCAZOn9hNZPPytJG7EVdAuZX4ZSTrW9yKAZOp+nXRodw/L+1c1CKzGTjFtCV5cVd/u5nMmGZIvZ5uXdi3kUxMfrKQPQQ/hyYeYUls/UWU+A+YyI8fGU+YHndi4UcoJf1y10fhiGwqPPnjWA8wCwarSjdkdWAQTxBKV7MYS50kuudwSwHFTcRbGNqegV9ofKiRDKNn1EuQlXNakj43w0wAma7yYwD3MtfQa+53pk6XDa0w4weQxWwdApAFVzIw== 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=DLKE+beq/D1Eo/Mxja3u/Q4K4ECOj09vAWEUnc6o0oE=; b=L2IMccJV4QGcDiDBz4cR2SenW0KhE8KlB/5K3fBIPsDXIj1UY4VdUW7fira6fDGfSRaIxTfRc0eEEjJIoThjGAt8hew4961SehaUQRMRb/qVr6o0PSwzJ8f3Lzew0S1AANKx6D7WT01cnnWNN4Twf3v7KRZIhMKYdvVKrxeaOKh0TEWFCrW9OQxr4MUM6lmzO8p2VpGGfG+juh/TfjO5ZAfkYLwspjXuWGL6U2OfJvQU7iSnfkYh3+6gT+XkEhFwjje6G9LGUMUEuOtfDc0spvws6KF4bEXdADq6udNXDMmiiSE3h1YDr7Z1s3i042fD+iOH6RG4HiPPidLVnhFzVA== 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=DLKE+beq/D1Eo/Mxja3u/Q4K4ECOj09vAWEUnc6o0oE=; b=kKIH8Et0Fy5HIRkDJDhuYuBnd/P9XdVgLRbQe3GcFsOen31tsJp49Krn4QGrRmf7XAgLCs8Lf03rwdsNLqHhMQcu0E3ZXP4+DXpbZDFFHxjC8z/A0mFPa4e8GG5iUyo3j05r+zL9nUIZxwLUcs/JDlEL8jEzseQraAtZco1bAinnkoapmSlHji4eWxnuNxZy4wyvg3LsvWSuKnv+3GiA/XQwF+DNh/T9DwJMAhVaV6UxkB92i4Jwntl93TcKqzR4uN7EJC1StiBGbGucXtxkL5HXx+UUtgrZ5liZKLJL2OgTExQ3rkKKJrO1lFciRz9Ius16DikY0+CJb9+IzGxaPg== 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 AM7PR04MB6775.eurprd04.prod.outlook.com (2603:10a6:20b:102::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5857.19; Wed, 30 Nov 2022 06:58:33 +0000 Received: from VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::4da2:ea8b:e71e:b8d8]) by VE1PR04MB6560.eurprd04.prod.outlook.com ([fe80::4da2:ea8b:e71e:b8d8%4]) with mapi id 15.20.5857.023; Wed, 30 Nov 2022 06:58:33 +0000 Message-ID: <63e51eee-e9f4-5332-d69e-feca3421553b@suse.com> Date: Wed, 30 Nov 2022 07:58:31 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 Subject: Re: [PATCH v2] x86: Remove libopcodes dependency Content-Language: en-US To: "H.J. Lu" Cc: Binutils References: <20221122181927.251937-1-hjl.tools@gmail.com> <6a5d4918-919a-8b6b-822b-17ce38488629@suse.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR3P281CA0024.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:1c::15) To VE1PR04MB6560.eurprd04.prod.outlook.com (2603:10a6:803:122::25) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: VE1PR04MB6560:EE_|AM7PR04MB6775:EE_ X-MS-Office365-Filtering-Correlation-Id: fbfff97b-3d20-4ee7-f3ba-08dad2a04b62 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: AxPazEcw5vQOPMP1D8iju0EOkkOsh1otIFOcUbZz8k106IkanynBD7r++uohMBAiBUV+r0jnIYoRk01NotScfFeb8HoY9nRDyNvZyZ0dFAyIFSw2R6RDrICykPuS67eqRZiULaeGBwzsraXkloNS4k8lr8mxqq2F/rIsG+GxYIh/nyX7AHjunW5gn8PqnrF4kjNGbXYT1LCg82Fr0Qiffa3s8VEk5fkxhSzhfKRLH54M1nPTwNjTGcuRsBZAWkjFVNB0ipr9GvMb3MQy2YU/PR32H/aQ8mv1cwCuF1xmSJ007GjtHDS/5qt5j0/ZPCGVpkc7OnwXxVWBgJj8voAtGafkWvsfdjoUg51BRDNnjj7iDOkqu84NgjNW/J3NnkaHbC3GPX1U+rThtFFbiqphmgQXUeRMrQR1wt/5U9fFTLAvW2bC+Wqz2UILjH2052uIA0Ik1mZKCEzU+nAns7jy7jaqD/FlfLcrxBtxm2PtrHDq78f6RGNiq1tILZW1WiOxKHw3cZnaUbsqjanWrGAg4nEa06zeubU1RerJYHE5mbkd/B1WhqGPeL5DNqREyeEbGRBeqZA9PIpIxvw8Vc4dGpMHGl8E70Rhho5/WuyvAD0g1J+Ks/gIK1GkuGwmx4MiNqAgF6tZM34SvAQ5LJWg8GIZtp+rkeKtH8BpHxj2vwOzrMvXifY1tG8ap8dti6pL960vO5hhZO7OWsIgGmcwcotzflHTZhMnB6DSbOd7WKk= 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)(39860400002)(136003)(376002)(396003)(366004)(346002)(451199015)(2906002)(36756003)(86362001)(6486002)(31696002)(478600001)(26005)(41300700001)(316002)(66556008)(66946007)(6916009)(5660300002)(66476007)(8676002)(4326008)(8936002)(6506007)(6512007)(2616005)(186003)(83380400001)(53546011)(38100700002)(31686004)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?azN2RzJlVjF0U3IwSjRodlBvVW5ObFV0WEJIMjJteWNSdVFsWG5hTVBucnBm?= =?utf-8?B?bVVFekh5QUY3M0Z6ODNOSmJnbnQ0amhrVHVkYjg1RkFjVmlSWUFpbENpalJI?= =?utf-8?B?S0k4WmsrbmY2K3MxOVRTTFNlSkFWQkpla0piN0xqK2hUdlc3UlNHYndQM2x4?= =?utf-8?B?b3Q2Q1RTNDEyVytVcWN0NGx5T3RkVVRaMW1aQndXY1NwdEI2NUkrdWR6Rlky?= =?utf-8?B?R25XRldNSzV3VThaaUhMRjBVc0Z4OHhTQ3N3L05IUU4zT2JDMXoyVWJvM2lH?= =?utf-8?B?WWJIampoRWFnVTNTc2lrcnoyUFZ3by9XcG5KMDFtQ0hTS1huUnJwS2RBY0t2?= =?utf-8?B?WitCTTF2VWVGTmd3aWFBN2ZUMzhFSTVaNU9MMHoxeVVLTjVmZ1cxVUhLdnYv?= =?utf-8?B?UGFIMVBRbmJwSzdJK2tzYk9nbEZzaExmQmxKUnVZVWhrQkRMbkZVaHVWK0t2?= =?utf-8?B?QWpIYSs0T2RZU1IrUjQ2L3JlSnVNcmx4dGVJMUJKUkQ5dEdra3A4eUgvUStO?= =?utf-8?B?TktlZUJ2R2xtQnpVeDRRNFdZZWFuVlJjRnNhSUVIVGFXczgxUXUrMzV3OS9t?= =?utf-8?B?NHllbCtRTUN4REVuempFcXE0WGpQQVVjVkZFbkVDdHpWOGJVK1NvM2FsdjhZ?= =?utf-8?B?SzNEZlVyMjlRMTFoOXMrRGRYM25SU1lFY2wrQ05aN0hZaytYOUhucE1reUs0?= =?utf-8?B?N2pDaVJuTFlEL3dFSm94QUVMSFo1UlRSb0Q5WGZwN0hHUnNjTW9VcXliU0c1?= =?utf-8?B?djArZHpKekFvMGw2bXF5bE1HVE9oZGkwWklBZytIUHI4My9tTDNwM3Zvclo0?= =?utf-8?B?OVAxQXJqbXFBdVFWRU5JdUlWK29sVThneklvRGdCUy9RUXd0dmF2TmRzaVVj?= =?utf-8?B?Y2ZPQXQ4ejM1N1IrZTNScFNhblJrK1FvYW9manJma2pVVTdxYTRReW9xczN5?= =?utf-8?B?L1UwekhDUlNWbGVEKytHSHVydWNGR1ZTRURyQ2Y5bjd6V1JJRTBGU0FDREwz?= =?utf-8?B?Z0JFUTdlMUYzQUloZE1CN21sVjJWRTNYOU5PYkw5UEVwU09GZlZKd2M1U3Bq?= =?utf-8?B?NEptdU8zY1dMcDVaZEZNbFNhbTBBOGc2R3A5akN6RkNGZU4xaXYyVVdTcGdr?= =?utf-8?B?RWljVkQrL29BMzA3a0JNcFViMFZITmJrb3VaRkpmbHEzcmVnc1dwaGYrdnZC?= =?utf-8?B?K0lhSUhPanhEWGEra1pFNDRwVmVkVkFIVHF5MmVrelZyeFY5SGRkRi94RmVR?= =?utf-8?B?M3JpM25vRk1INlZGODFYaTI1WDUxeVAxZWk0Q1EzTUR2c1dsRlB4OFc1UjhH?= =?utf-8?B?WHEzS1VaTHNicHc5Mi9EdkJWWU02OVJBclh2THlGSHFueWhoazdIS0JXSFVr?= =?utf-8?B?OVFjZE1RU3JMQnlLazdvRDlBSGlEUTBYMkJmWVAxNXF1U211S1dnalNNWnBj?= =?utf-8?B?eWlZcWY3VURsNFF4Q1lIVVQ5T3B4bmJULzFJUmlmT2ZDUmxwenZxQWl5bXJX?= =?utf-8?B?OWRXc0I4RXB0SFRWb3FhazZ2UU90MWl1NDJYVGt6YTd4MURrc25KNDhnYlVR?= =?utf-8?B?b0xpY2RjcmVVRzNoNFk2ajl0UzN6K1hKYkRHTWxjd2FsN2grbWRXb2pzbGk5?= =?utf-8?B?MFhFajhDWVF6V1VsbkRFTDdlM3VWMTJ5NWlWeHpaZXFwbWU2YlNBd2dITjJu?= =?utf-8?B?V2NQT2dmUFA0dkVGcnBmeCtuNW0zVzVGb3BKd3pubzkxZzU1cEZyd3lmeXRt?= =?utf-8?B?VDQ2SUFHYWlMdlVza0prRTZhZUxxZU85bDdYbmNaemVTK3o0ejMySFlwTzFU?= =?utf-8?B?UUFXTUh6c25VcW93U0JHMXJtbU83bFZ6cUp1OVNndlJmZTRta21oK3RTcnpP?= =?utf-8?B?SkxzdTRaZzBscUt0WVlXY0Z4OXlsWUtkYUtLS1lnYXp4UU1jaUpBS24zTnM0?= =?utf-8?B?a0VEOGJubnJHWXN3YkdtcU1VMm16dHZVWEk3YlpqNnNrUStWRmVsMG5Kb0Fx?= =?utf-8?B?RllrRlVKTCtoUWxPaXEwWTRlcWFReml3dHBaZ284cHEzcGtWa21zQWJQT0Iv?= =?utf-8?B?S0ZlczJ0YURudDc0Ujd5WG9ZTDFGZFM5VUc3cElCbllxcVd6YXRSUW8wdUxi?= =?utf-8?Q?A37I2p3VJdIk8ZVwnvz7rGNFS?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: fbfff97b-3d20-4ee7-f3ba-08dad2a04b62 X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB6560.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Nov 2022 06:58:33.0379 (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: LN7mhigmmRRSiVk3OJMvg19Y4gVA8cQF36eN4eQD0PBiT+zz8AlU1v+hqSUnhQao9JkaE9k0Qjuv/7hch3mLRA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB6775 X-Spam-Status: No, score=-3029.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 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 30.11.2022 01:06, H.J. Lu wrote: > On Tue, Nov 29, 2022 at 11:38 AM H.J. Lu wrote: >> >> On Tue, Nov 29, 2022 at 1:22 AM Jan Beulich wrote: >>> >>> On 29.11.2022 00:49, H.J. Lu wrote: >>>> On Thu, Nov 24, 2022 at 2:19 AM Jan Beulich wrote: >>>>> On 22.11.2022 19:19, H.J. Lu wrote: >>>>>> --- a/gas/Makefile.am >>>>>> +++ b/gas/Makefile.am >>>>>> @@ -446,6 +446,12 @@ development.exp: $(BFDDIR)/development.sh >>>>>> $(EGREP) "(development|experimental)=" $(BFDDIR)/development.sh \ >>>>>> | $(AWK) -F= '{ print "set " $$1 " " $$2 }' > $@ >>>>>> >>>>>> +$(srcdir)/../opcodes/i386-init.h $(srcdir)/../opcodes/i386-tbl.h: \ >>>>>> + @MAINT@ $(srcdir)/../opcodes/i386-opc.tbl \ >>>>>> + $(srcdir)/../opcodes/i386-reg.tbl \ >>>>>> + $(srcdir)/../opcodes/i386-opc.h >>>>>> + cd ../opcodes; make gen-i386-tbl >>>>> >>>>> I've made a patch to gas/Makefile.am as you have requested in reply to >>>>> my series. I will want to put that through some more testing, so I will >>>>> submit a v3 of that only a little later (and of course only unless you >>>>> submit a v2 of your patch earlier that I would also end up being okay >>>>> with). In the course of doing so I noticed a few more issues with your >>>>> change: >>>>> >>>>> For one I don't think you can put @MAINT@ on a continued line, as the >>>>> line continuation might then be hidden when @MAINT@ expands to #. The >>>>> list of dependencies wants expressing via a variable, which would then >>>>> be used immediately after @MAINT@ without any line continuation >>>>> following. >>>> >>>> Fixed. >>> >>> No, the same problem is still there. You either need to use a very long >>> line, or you need to introduce a variable holding the list of prereqs, >>> like I've done in my series. >> >> I got >> >> $(srcdir)/../opcodes/i386-init.h $(srcdir)/../opcodes/i386-tbl.h: >> $(srcdir)/../opcodes/i386-opc.tbl \ >> $(srcdir)/../opcodes/i386-reg.tbl \ >> $(srcdir)/../opcodes/i386-opc.h \ >> $(srcdir)/../opcodes/i386-gen.c >> $(MAKE) -C ../opcodes gen-i386-tbl >> >> and >> >> $(srcdir)/../opcodes/i386-init.h $(srcdir)/../opcodes/i386-tbl.h: # >> $(srcdir)/../opcodes/i386-opc.tbl \ >> $(srcdir)/../opcodes/i386-reg.tbl \ >> $(srcdir)/../opcodes/i386-opc.h \ >> $(srcdir)/../opcodes/i386-gen.c >> $(MAKE) -C ../opcodes gen-i386-tbl >> >> I didn't run into any problems. >> >>>>> And then your rule / dependency won't be enough on a "maintainer-clean" >>>>> tree, i.e. when the generated headers aren't there at all, and when >>>>> config/.deps/tc-i386.Po is still empty. In that case nothing would >>>>> trigger their generation; an explicit dependency of config/tc-i386.o on >>>>> these headers needs adding here. >>>> >>>> Fixed. >>>> >>>>> Finally you're missing a dependency of the generated headers on >>>>> i386-gen.c. >>>> >>>> They have a dependency on i386-gen which depends on i386-gen.c. >>> >>> In opcodes/, yes, but talk was about the rule in gas/. Yet despite >>> your comment above I see that you've added the missing dependency. >>> >>>> Here is the v2 patch. >>> >>> As said in reply to v1 - my objection to this particular use of make >>> recursion remains. >> >> The gen-i386-tbl rule doesn't touch any files used by libopcodes. > > There are > > bfd/Makefile.in: ($(am__cd) $$subdir && $(MAKE) $(AM_MAKEFLAGS) > $$local_target) \ > binutils/Makefile.in: ($(am__cd) $$subdir && $(MAKE) $(AM_MAKEFLAGS) > $$local_target) \ > gas/Makefile.in: ($(am__cd) $$subdir && $(MAKE) $(AM_MAKEFLAGS) > $$local_target) \ I'm glad you spotted this yourself - I was just about to point out that my concern isn't only a theoretical one, because of the generated makefiles we use. You don't draw any conclusion though: Are you willing to accept my approach, specifically reworked to account for an extra request of yours? Or are you meaning to make another attempt yourself, avoiding the undue make recursion? Jan