From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-ve1eur01on2081.outbound.protection.outlook.com [40.107.14.81]) by sourceware.org (Postfix) with ESMTPS id 5D366385773A for ; Mon, 16 Oct 2023 09:18:41 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 5D366385773A 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-Filter: OpenARC Filter v1.0.0 sourceware.org 5D366385773A Authentication-Results: server2.sourceware.org; arc=pass smtp.remote-ip=40.107.14.81 ARC-Seal: i=2; a=rsa-sha256; d=sourceware.org; s=key; t=1697447923; cv=pass; b=CvhS6uacieL20B5RZ0Tgv0JzxIW05/AB73rObbPQJv2/jS6jCVcEQQT5zUU+jPN0nCWUM7czFrXf5bKDY9QhCwMGxuZsLVV9BbYCTuDcr9uNFg7lo/Dqdnaxc0gUB/NuZvSsa0NFT/4I/IMcvO9Rv9f88gO50PR4uk/bw5POxGQ= ARC-Message-Signature: i=2; a=rsa-sha256; d=sourceware.org; s=key; t=1697447923; c=relaxed/simple; bh=qgyHQj0SnnUNAUh30sbGgUaigrPZOIPH2OWNkC7sNHc=; h=DKIM-Signature:Message-ID:Date:Subject:To:From:MIME-Version; b=s3yY1C/S+xNnbkG5VqpcJVLWO2q6HeSKBIlqR/CXKZ2YTgalNSXwRiz+7Z6kZNLFMiRPHqrAwEHl/NykeeY2FND2wR05PbdQIF89mIuF7Q80TjaAl7Yonwd3P1BpWRdxWYXqC1tf6p6HCamRizNcvhccUrMH8tD56JAh4acVfmE= ARC-Authentication-Results: i=2; server2.sourceware.org ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=R1vjN/mewWKOY1a/NDCecCmwkOpLasj74ihxmQEvEHqGha/ax+JeRNKSXoSaoZHR/bWkr8V94zcaZFIhGw1QjUfbXY5zKwA1FHHvAyCj0BKYqFfSkTYQaMnnhg499KI7AKbl1EQb/wvJviNYWtkjW3mud8349aGcQ/5Q6V/4XtsaCpn3bV+izmjbRWpDbtJsktABqhmgg+qJm3tVzoPM+jYgxtcncB/zqornz/iTFHyQNFKDXpT9dywbYq64hwOjDe6YuG3R9NB3mKi9BDD9Hiz9OK7X6n4Af3G0aCAvwUu7AansJcWXp8PE+U6IgX/yfLpiMNyIMNTIknIjR7632Q== 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=hyv/PmRQUH+ZCKEsocqfkeL7tzBmtz9CVCcMrW6cYOY=; b=eaGdHPXtY1RhlyRJrY4wfDpQABKmXgY0n7zrub/phn7OYLLd8dyX27Q8bPV/ifFbNSHYm/u/FSKihKb5a5JxivwapmRSyi07RusScX9KphpgwlxZhpjHUw8tdvLCTMitmsIS/3rY/b+yUdKcMqR5HJK1QIW/PaMDu3KNsC8IbHJ1Bf09Ah/U+1eB9oF2NHf9NxWhidXca57tzyYcwBVquYTvf3z8gOvH8Xad+vFrvCxzhc0y57B151T9uM6djhPeqE1ZOD11rg1hJgiOE4yIuxb8i7j7mJIlNHIQDvqFV6nFRhRH5Pd3ZEOpnidFAHCA3eA3C68JtO1NrLJlDE/lnw== 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=hyv/PmRQUH+ZCKEsocqfkeL7tzBmtz9CVCcMrW6cYOY=; b=kKsJOwgr2M4B7w8gd3AiJQrW36lY2705casj7BIIfcc+vXPBNhSMEbkuRSuvXRVmPyfscU6V8LQf4KWiD4AKTqzNslM3157QzWuS3ACd6HSyXchh1cCv700iQGxCQML6dueCFWdVRLf/ZLw02ZEIC40WKXoWHDOnBgHQdKjjVNBIxRqvAnoE21tVq5cXD6YjP1SnQ8UitL0CkOLAnmtf4SkodfZuW/sizDzNEaLgvNmYRCUjgMFGlhWLTfT1X8vYIREn9G8BUwwaLzo5Eu17Hfie46jx4piMO4ebBfnZszEBFr9pz4FbibjyrTIXIOrz2TYJFhJlztjjTBleBUteiw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Received: from DU2PR04MB8790.eurprd04.prod.outlook.com (2603:10a6:10:2e1::23) by DB9PR04MB9476.eurprd04.prod.outlook.com (2603:10a6:10:366::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6886.35; Mon, 16 Oct 2023 09:18:39 +0000 Received: from DU2PR04MB8790.eurprd04.prod.outlook.com ([fe80::d9c0:d907:4d2d:15b3]) by DU2PR04MB8790.eurprd04.prod.outlook.com ([fe80::d9c0:d907:4d2d:15b3%6]) with mapi id 15.20.6886.034; Mon, 16 Oct 2023 09:18:39 +0000 Message-ID: <2ba0ab1a-9190-b65c-679a-853dfb03d1d3@suse.com> Date: Mon, 16 Oct 2023 11:18:36 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Subject: Re: [PATCH v1 1/1] gas: expr: fix support .long 0U and .long 0u Content-Language: en-US To: "Frager, Neal" Cc: "Erkiaga Elorza, Ibai" , "Mekala, Nagaraju" , "Hatle, Mark" , "Mutyala, Sadanand" , "Nali, Appa Rao" , "Hunsigida, Vidhumouli" , "luca.ceresoli@bootlin.com" , "binutils@sourceware.org" , Michael Eager References: <20230927132130.1604555-1-neal.frager@amd.com> <424aef37-8ee6-5309-8342-502e0f41b73c@suse.com> <1eb5429b-94dc-e394-14f3-9063f19125eb@eagercon.com> From: Jan Beulich In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR0P281CA0225.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:ac::13) To DU2PR04MB8790.eurprd04.prod.outlook.com (2603:10a6:10:2e1::23) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DU2PR04MB8790:EE_|DB9PR04MB9476:EE_ X-MS-Office365-Filtering-Correlation-Id: f6ad72a6-8f58-4717-85e7-08dbce28e1d6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: IfjmxpL9ryYShIn4Y46iCcxcqDvp5vO55gPuzcFCpnudYXumbWmPzE0QKmfrbX7q8JzJql62KU2aQ50hTJUvIiKvO0BC4KPl8fkShEeYB5Bs8ADbDd+bYfaNZ4/izXMHnIjTzlkwkOpi2BLcM+y3JIqxfNy7Q0wp+XOaeMUgFEItQdn9ENwbnQfPi+RXpVmRG/KzwG1E7mVCNoVrpJXKOPmtxaf6bIddFDlKAwNY8sAdBs3pnVg9hetuJMFLSjWjQLbliEFw3q3C/NUf2WgYfl7jai6te9vIqKwL/tdYJdsowEP8hxH9sXJ+BSDVypneiQQWTuBuK3qXdhezohZzA1okhA04eY2Qv8Mog6RekKIDl8PU2iOXcjbUxW99ihdR4SY6z9JIuVlD/6wWnbJ+t4xKjqtWWhKHLUuzIKZmL+vf9Ox8Vp9gKfWOOUMeN3EHeuh131D3ImZ4ju0/9wKTqjlqSJadcVACFNenvxnM3SELzfL0ujXe/hPuI5JOPO/vkaQCJ0LNM2RDWcP2BFrO8Y26dVIZceawpJhX+F9O9h1JInwv5SLBFtQ/wX4sa0kshC+fEV5sHzbUWCImzVUrs60xCmCZ930p+fdzkQQ7qmHtJYiGuCTyegYdxqFXBHjOGALPH4bkE0BjOzQwW0WPtrVduzFC65/4ZN72ogHjjXabIsZ1EZnrUIt2BvaHlU08XodDpT5sA+bxUIe5NiKo9htfGWDkpOXXMVIX9Z0Ma3w= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU2PR04MB8790.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(366004)(39860400002)(346002)(376002)(136003)(396003)(230922051799003)(451199024)(64100799003)(1800799009)(186009)(31686004)(6486002)(966005)(36756003)(38100700002)(83380400001)(6666004)(53546011)(6512007)(26005)(6506007)(66946007)(66556008)(54906003)(66476007)(6916009)(316002)(478600001)(2616005)(41300700001)(2906002)(7416002)(86362001)(31696002)(5660300002)(8936002)(8676002)(4326008)(223183001)(142923001)(43740500002)(45980500001);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?T2FUcExrYmIzeHM4MEIwOXFtV1FtYnZWY2FJRVNFTTk2OVgzaXNvenpGTTNs?= =?utf-8?B?QXV3aEFueHpBZ1hXcjNsQktONnFpRWpTR1l4OHBJYk5OeVNQWkJ0RTByQ25t?= =?utf-8?B?MVpHTVlqa1ZkUnZ5VnhoUEt1cFhYQUlIODBOb2draFpZZUVKYkNKT29BZURp?= =?utf-8?B?SGtsUlRDV2toYVY2VHV3cnE4VXJCMHpLNURzTkJId242WVc3bVJOVmtQdmg4?= =?utf-8?B?enhMb1l1MklTUWlJNDZ0K3hqVnBTN21lLzdrWHl0VFRqSytmdVRhTm9UNWVX?= =?utf-8?B?MVN0OCtVMEdVUXpFQloybjlQcitBQTZrOFhKTWNERWI5NWl1WDBmTUdEbzdS?= =?utf-8?B?ckxHQjVydzd0ZW9IK2lEWk9lbmFmaVByT1VXMEU2ZWxWU3JqbGduQWV0dnJF?= =?utf-8?B?ZUNHTFVSdHZCWk9OczRUdHZyRXd4b000bWN0dFNJWUZtSjJpeEkzY21oNUJL?= =?utf-8?B?dTRHa21mZzJqR0R0a3RUOTJQc0MyR2dCdDFsTWxqYUc3OTdScWpNdEhTSWw4?= =?utf-8?B?MnJIb1lOTmNuOEFFMFY3WVk0QVFGSWdzUjhEQmZWaWYxUUNZakNPcTVWeHRl?= =?utf-8?B?aVZnVERIK1cwenVwTmJFY2dEZXdobmF0dm92MHkwbjlPZ2lqdW1YV2FRMHRK?= =?utf-8?B?aUxJRFQzVXArVCtKajZMLzEwbjYraGhNYVFRR3BOUGl0VEhacHgwckpMeVkw?= =?utf-8?B?YVE1ay84dGJxY0JaWlZUK2x6aXRtcXFFazdUUGJuNHFxdi9DUmp0MVd5SGhK?= =?utf-8?B?S01wNHAxVGdlSG5nWDlzZVJ5N2NHNmF6NFRVN3k0cVYwWnVOenZwRHVRUHlr?= =?utf-8?B?U0FTWDUxajNuMDJIVkRFNUw5MEVNK3h0b0E3Rm1iUkgrWDQrM3V6QUZxRTIr?= =?utf-8?B?dGd5by8wS3Bjajk2L2dIWHlwRlBTa3RMQ2Z1ZzFSTytmZVRRbWlnTVRzd2Z5?= =?utf-8?B?UXl0RGUyL1VDNVNJendpY0NpZVFZNG9zTHdvSU9iSVgvSjNJdEM0SUNsOFh3?= =?utf-8?B?NkxhYi80TUFxTEJjaWhBMXhrWlRNeVFrRnBtRS9aZlRCMkRHY1cxOGp6cU95?= =?utf-8?B?VHhpZFhzMHVHZk51T1RtRFBkaHE0ckZBTUxkSi9jZkc0QTFlbkh5OXByaEpK?= =?utf-8?B?eWdUTHRzRkJJUEE5eHBNTXEyRkZOMUVyazRVQTI2bXNBd2pHaGN3TTdmVURM?= =?utf-8?B?akpJYzFzcnNXRUMvYm8xSXU0bys3T3VJSk1hM2gxOTd5VnFGcXdxZXBMQU9Z?= =?utf-8?B?YzZpcitnQk10eWpBTEZ2d0xHV0tJd2UwbmN5THBnbXpHZ2dTR0hBdUNBQ0Mv?= =?utf-8?B?dU12cm9hZEE4azFvOFNPVHJtV2RIQ1RJeGdQYTlCNTMrQ2J1aG0xUm81N3VG?= =?utf-8?B?SjduNVFlZSsvMmJPN0crQzJWVXhNeER1M1doMjZrTjFDRGowQVp0WmtqMGJG?= =?utf-8?B?NTQzc1ovZkVFL01IOEhOaEYzbXd4bFBUSkI0bW05UDFjMDljOFBwZ25NSlU1?= =?utf-8?B?eU5FNUpicytjam5zOVdVemhhU2pOTHlFMzJSaS9WOU4ySTA0T2FSWlJ0R01r?= =?utf-8?B?YzhIL1RROTlRNGJydnlDM0RaSVN4UkJSSWJSYnVWczJCN3U2SGhBeWJoSSs1?= =?utf-8?B?aFBBSXFmSDIrVU4yKzZKcU00THByaXdWK2srL3FUOC8zdjFsYmZsV053MG13?= =?utf-8?B?SVdRQmQxa3dkTFJoNUxEenl2d0drY2dRQW5XK3E2d2hkbzF4WGoxSVQ2aDEr?= =?utf-8?B?RTFWZVBwdG0wR09ldm9GcVZJa2lGeVpkVmRBYWZvT1hIVGp6WmJ2a1FBeU93?= =?utf-8?B?OWJqMEZMK1hFTUNFWGVuZ01JcmNpWDNBRVpwSnRhY0JGWFNucUk2MDRJVi9F?= =?utf-8?B?ZW1aVms1ZE5ORktmUzJzdFo1cGZRZ2FMOXVsdGQwUS8xVDlJTDZ6UXJsbEdK?= =?utf-8?B?dkVKMkNQMHc5dkliZXgrOFRGTUJPR3RYaGJKakRtVkpxRHpEdk1Lc0hHbEdE?= =?utf-8?B?bkoxamN5Unk5NzFKMHB2dXFTQ2RUNjczRk1YU3dtTkVZQkpGN1Y5SmxwZ25B?= =?utf-8?B?eVR6RDNXR3FRSVdyK09VQ21neGprSnFxbFlXQmMvOEdxTmg1bEJnRTRWZVJO?= =?utf-8?Q?gMHoj6UNxATx80kzTpfGv5UAk?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: f6ad72a6-8f58-4717-85e7-08dbce28e1d6 X-MS-Exchange-CrossTenant-AuthSource: DU2PR04MB8790.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Oct 2023 09:18:38.9322 (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: u4B9DWE4jw7NCZEyVi4/gtxJEWsbD/W+OljzRGzdfkBtNNPXGVA2M3RhXXWGH7gPSdWl8wyj0m38pC0CV2P1Ug== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB9476 X-Spam-Status: No, score=-3028.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 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 01.10.2023 18:22, Frager, Neal wrote: >>> Fix support for .long 0U and .long 0u in GCC. >>> >>> This patch has been tested for years of AMD Xilinx Yocto releases as >>> part of the following patch set: >>> >>> https://github.com/Xilinx/meta-xilinx/tree/master/meta-microblaze/rec >>> ipes-devtools/binutils/binutils >>> >>> Signed-off-by: Neal Frager >>> --- >>> gas/expr.c | 9 +++++++++ >>> 1 file changed, 9 insertions(+) >> >> Without a testcase demonstrating what's wrong, I'd almost be inclined >> to say that ... >> >>> --- a/gas/expr.c >>> +++ b/gas/expr.c >>> @@ -824,6 +824,15 @@ operand (expressionS *expressionP, enum expr_mode mode) >>> break; >>> } >>> } >>> + if ((*input_line_pointer == 'U') || (*input_line_pointer == 'u')) >>> + { >>> + input_line_pointer--; >>> + >>> + integer_constant ((NUMBERS_WITH_SUFFIX || flag_m68k_mri) >>> + ? 0 : 10, >>> + expressionP); >>> + break; >>> + } >>> c = *input_line_pointer; >>> switch (c) >>> { >> >> ... this ought to be covered by logic in integer_constant() already. >> But I think I see what the issue is. Nevertheless, go look for >> tc_allow_U_suffix, which wants using here as well. Further I think the >> same issue then exists for L/l suffixes? >> >> Plus I think you want to add the new code to the switch() statement >> rather than immediately ahead of it. >> >> Jan >> > > Hi Michael, Jan, > >> Neal -- > >> Can you provide a test case for this patch? I'm not able to reproduce any error. > >> Please take a look at gas/expr.c:541-543 (PR 19910: Look for, and ignore, a U suffix to the number). Is this a fix for the same issue? >> https://sourceware.org/bugzilla/show_bug.cgi?id=19910 > >> As Jan says, it looks like this code, if needed, should be in the switch >> statement. > > Thank you for your review. > > I would just like to clarify where this patch is coming from. > I am not actually the author of the patches I am submitting. > When we provide a Microblaze toolchain with PetaLinux, Yocto or Vitis, we are actually applying all of the following patches to the binutils portion of the toolchain: > https://github.com/Xilinx/meta-xilinx/tree/master/meta-microblaze/recipes-devtools/binutils/binutils > > Instead of continuing to carry the maintenance of this patch set with all of our releases, I would like to get as many of these patches upstreamed as possible. Entirely fair. Yet then original authorship wants clarifying; in the worst case to indicate that the original author cannot be determined anymore. Alternatively you could enter a bug (without reference to the original work), which ought to trigger creation of a separate, newly written patch (I've already added this to me todo list, just in case no v2 patch appears as per below). At which point you could then still drop that patch at your end. > This particular patch is coming from here: > https://github.com/Xilinx/meta-xilinx/blob/master/meta-microblaze/recipes-devtools/binutils/binutils/0009-Patch-Microblaze-fixed-bug-in-GCC-so-that-It-will-su.patch > > It is possible that this patch is not required anymore. I will check internally at AMD Xilinx to see if I can identify a way to reproduce the error that this patch is solving. I've double-checked that the change is still necessary. Please submit a v2 (preferably, as Michael asked, with a testcase). Jan