From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2107.outbound.protection.outlook.com [40.107.22.107]) by sourceware.org (Postfix) with ESMTPS id F1EBB3858403; Wed, 13 Oct 2021 21:37:19 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org F1EBB3858403 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=mXRtsu7b1XmypdIxbj2I8twpBu13v4SjGF5OvLkdnekDDNQlAbZe2m/eh6pIrCMnt9hEgRplzjYkYLtQMWbpbIKyJ03HG4XSuaEnCeuff4Ks1ccP7KKpjIPvY36Rs0G4svSOjHXVMwbMhakMBeQTznflZMmkfbo8U6JVb4NiLXa7ZPfeWVCkJxW8orA5NSJpG63b5A2O8InJw2f4j26QJau4ct/Mh3TwNQ3SgoCOgx+kIiIT6g5LdDRRMuPH1jVS1uxIGGug3YTuZxh5gOKIU0njlWZF87r7tOE0vkAFau4dwHBiWaEJGBeeHDd6fLbHI6WGvE2xNqfpa63lNu9hTw== 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=yBmqwPuKg5Sbhay1ToIYYipGyOXONPcUVxcWp0gHW6I=; b=WLtYeuyoAN7WVtVUVWrQVhQJkGQgwmD+pmeSn4XHWLIoZSU3Bp1JWG/zFwsjfw2FrQAP9lzT/DT2Pa2m8T6leh4WS2iffNhdX90SPgrXtkL4uTXw9graJpo75SXBNPzlZlUk852ZUEGDhMQQOGzHO9AUaj4OTml0vwgt2VbxQOvBAK+HR3L3u2bFc8AtmXLrV/TUARDOANH2KtqguHvCPnexM9wt7YEf+W/UmPdtR3T06REcc9k7frG+Zf96xJbWA7EfJjkMdbVPgWkvA1RoJJEN4cUTARlTM607KBDVpbzFVof9Mu8o5l7I7taudp9aI1UIa8RbRaozXbEL6OItUA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=uws.ac.uk; dmarc=pass action=none header.from=uws.ac.uk; dkim=pass header.d=uws.ac.uk; arc=none Received: from VI1PR0102MB3197.eurprd01.prod.exchangelabs.com (2603:10a6:803:2::28) by VE1PR01MB5918.eurprd01.prod.exchangelabs.com (2603:10a6:803:114::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4587.20; Wed, 13 Oct 2021 21:37:17 +0000 Received: from VI1PR0102MB3197.eurprd01.prod.exchangelabs.com ([fe80::31b0:1b9:1b31:87e]) by VI1PR0102MB3197.eurprd01.prod.exchangelabs.com ([fe80::31b0:1b9:1b31:87e%6]) with mapi id 15.20.4587.026; Wed, 13 Oct 2021 21:37:17 +0000 From: Paul Keir CC: "gcc-patches@gcc.gnu.org" , "libstdc++@gcc.gnu.org" Subject: Re: [PATCH] libstdc++: Fix compare_three_way for constexpr and Clang Thread-Topic: [PATCH] libstdc++: Fix compare_three_way for constexpr and Clang Thread-Index: AQHXlf9zGYYf+4UjPkCR6yd+HcCmX6vOhdGAgAAVOoCAAybDsQ== Date: Wed, 13 Oct 2021 21:37:17 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-GB, en-US Content-Language: en-GB X-MS-Has-Attach: X-MS-TNEF-Correlator: suggested_attachment_session_id: 8ca2d7a7-d57c-ef79-19f4-e593ccaf3e34 x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 895e2119-1a39-43fa-903e-08d98e91a117 x-ms-traffictypediagnostic: VE1PR01MB5918: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:1284; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: ciaIZqk+q58GkV+x1U7ao7PfMP/V1qjvXbQ795YmzBWVD+sNKy7hN3OOlP27B6gcePaP2eyBW2VMdHnHw3MjebAhkKDNGxvIZnO2JacNw0A4Wd+H3rOVBwGksu8EvImZGTlI/rPbf/sKiSAeZ2yhz/rrujmmieikbRbiC/NBJ9ag2YDuoVDTw9ssF+fKM7YunKrZGTY+JpmWOT9XSyY4pEjkMsfk5w9yqRkEPXprKKtGZ6zgpC1W6gL3EZFatdsBJElSIUCrKo2WFP5W1NzWuA1EySaE11l+wVCVlXihAQhZlUDJH97jIH9IOASAOoeB9ONg0t7tI6fwdOnL0IgAphjeG16W/4ozFwMYJ/nRWVhQllA0AANoVSEWqMlnbilXQ5Z5+coplqdHtpdBgcXEGOXey+D1LQeTOpSGXBPHViUo+nNno5Gv862wLhZe/QYv/5bW6aSlat0jZjQrW9wmFfAA2NvV/rtxUK94FRn6p6AxpNaC3YBLzmQAhftFahvu+NC3quVGc9HRmi9OnDftFqtGjfyGM9uPxIqZemTv2Iman5UbcbaQEGcXzF2MPYaoKU26H+yhWKaTDCnwH8OhU/ksLiZsdAKiFgwSACIAVOOFBLrVMmvcasQsa5yyqOaHFpybSl6dcK46jvBFXE3TOlgtwNpu5ZAQfeBfunBSdlZDy2R4nIGhInsoAUuaLAc8wsIcEZL69Rj911Pr+1rB+kD3j65wpfQeO0vN/dHJr1d2HYDqOEzLhZETJtbmtrMS5WgYqdCo53gkJe4ImcnwLl8yIFh+k7u7VCCs3+HiFO8= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR0102MB3197.eurprd01.prod.exchangelabs.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(66946007)(7696005)(450100002)(6506007)(186003)(53546011)(8936002)(9686003)(71200400001)(76116006)(83380400001)(66556008)(55016002)(66446008)(91956017)(66476007)(26005)(109986005)(2906002)(38100700002)(52536014)(54906003)(64756008)(5660300002)(316002)(38070700005)(786003)(86362001)(8676002)(508600001)(4326008)(122000001)(33656002)(966005); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?W+fZlPyrqKLF5kumRBQCfR3BTBihUrArQQbGQT9FxeaRJBNZVGMj2cYr+ss1?= =?us-ascii?Q?xCVlzxSTv5q8iMToiUlHx3M6GYlVDs+e/2CPEW14jE4iMvddixqWEUD5ASoQ?= =?us-ascii?Q?2JCVGQ8cduUL+hUgRD8f+/Dp2x+XWqgMzEILYxAw76Mue4yzVSl3XCpM7ffp?= =?us-ascii?Q?sSB74nFjnFih4iOljjcIAfkSESXOYIYBJWj66K+agN8mHEI04KhTH7pFox2K?= =?us-ascii?Q?dQO1duwFteuJJ/8ji/RXmZwm+OaXVNN0mWoUuS7YsVNJrLRaTBQMW4vBuJQ4?= =?us-ascii?Q?si8KC2rrWDVFOsUsEIS6scIIwy1oRbU0LPzF6KGdmjft5W0RyHuq5sFDfIIY?= =?us-ascii?Q?YFsbZBi3fg0k/Ve4kAte6YuV/C84/LQkAFd6Xgc7RbMNpHmGDXIcvYHOpix3?= =?us-ascii?Q?h+TIstx+oMxjDEd9iABxhyovdbt11yNd0+VSYWoWI4vY9qsGz4QRp+5iVIRg?= =?us-ascii?Q?kFWhctw5ju6GxI3FYWjy/pEJcYZ2rgBUAKx3ZwNn9tXsGppSQPCJ2YmhA9Ix?= =?us-ascii?Q?ztl0LdSiHMXlZJpcyzF+0p4RrowxBpR2mAwQSvM0r2ih1NXE1R0M1pYjuV65?= =?us-ascii?Q?JIgAmHiIYWCYFBH6jeta3pNOWkqMMJ97bI3VpqA3GzuSR511ALwq+rsLDGdy?= =?us-ascii?Q?GXKjpVv8zEM89UcLYPEc+nb/KAs3HwKmfA9m7f2YSvl2B8wRV99PnRmZZ31Y?= =?us-ascii?Q?+Nnej8lhPad/SQZfTXT15nNGpud/zOB5Na1C+jQ6WqEdgLkfMrhvGJISc7Mh?= =?us-ascii?Q?+f4R5Wd2G1BnJHrhaZtlEhJIVyIiUnSbjSjIFMFsl22nYo6r1P1w8hvGKe8f?= =?us-ascii?Q?9b7TexMp/rpmGFj8494X4OR0POZfH8kPrO0syZIEfoU6DvEJBy1x+QQ3ZRBZ?= =?us-ascii?Q?kvIqVRXIXVPy+dVsabayG61cw5p8FZHhTkbDuEHdyBjiWjyvAPlr9VOQ8ujy?= =?us-ascii?Q?OKlWjtaceRhHFQoWNxqoTyRvgbjL9On9OmsgjAPYP2EBJiHkvlWL9BIHIYcu?= =?us-ascii?Q?ofz2xl7BSJbjAcVs9Cj2I755N46z52sr9fdf6R9C3rUCpNujzCKM83rTSK+V?= =?us-ascii?Q?KkrqhHqLhrsg86a3cm5Td0gE1j/ofRzf0Et3NlGXUfJBE95bDLKqSFxw58WZ?= =?us-ascii?Q?mqL5AEs9j7NnYf2l7ND7r+IbLO8RhZp8FDOZXor0NUh5aX+wQhyjGN6cLKqE?= =?us-ascii?Q?vmonzURiv4Df+bILp9QM7fgooxbnzPHUO8tJ1OMkvrVi20430xvinptFxVG5?= =?us-ascii?Q?wI/nJ9FY8O1UQG4JEW3GNs/hOMPPkrmCpIEXZWQD4hImU0N7RR3ppY7odJ/c?= =?us-ascii?Q?Dt+Ef14yN9iO/jXWBbF3IGNi?= x-ms-exchange-transport-forked: True Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: uws.ac.uk X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: VI1PR0102MB3197.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-Network-Message-Id: 895e2119-1a39-43fa-903e-08d98e91a117 X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Oct 2021 21:37:17.1095 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: f89944b7-4a4e-4ea7-9156-3299f3411647 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: iB1U0Ay4YND88flBUYNPb5Rs5iNgiASZQcvg02opHlCDH0lhuqChVbk8IXcPjW4hmZpcMjM5ZlNok/fyKYTxyA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR01MB5918 X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, FORGED_SPF_HELO, HK_OBFDOM, MISSING_HEADERS, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_PASS, SPF_NONE, TXREP autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: libstdc++@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libstdc++ mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 13 Oct 2021 21:37:24 -0000 I'd like to cancel the request to apply that patch. At the time I had actually assumed that Clang was at fault, but your commen= t made me pause. I'll submit a bug report as you suggest. We can reconsider= the patch in future once that bug is resolved. ________________________________________ From: Jonathan Wakely Sent: 11 October 2021 22:04 To: Paul Keir Cc: gcc-patches@gcc.gnu.org; libstdc++@gcc.gnu.org Subject: Re: [PATCH] libstdc++: Fix compare_three_way for constexpr and Cla= ng The source of this email is EXTERNAL to UWS On Mon, 11 Oct 2021 at 20:48, Jonathan Wakely wrote: > > On Fri, 20 Aug 2021 at 21:19, Paul Keir wrote: > > > > Hi, > > > > The current compare_three_way implementation makes provision for consta= nt evaluation contexts (avoiding reinterpret_cast etc.), but the approach f= ails with Clang; when it compares two const volatile void pointers: "compar= ison between unequal pointers to void has unspecified result". I include a = fix and test. > > > > Could someone commit the attached patch for me? > > Sorry for dropping the ball on this again. I've applied the patch > locally and I'm testing it now. Unless I'm mistaken, you do not have a > copyright assignment on file with the FSF, is that right? Are you able > to certify that you have the right to submit this to GCC, as described > at https://eu-west-1.protection.sophos.com?d=3Dgnu.org&u=3DaHR0cHM6Ly9nY2= MuZ251Lm9yZy9kY28uaHRtbA=3D=3D&i=3DNWY2MGNhZjMzZTA5NzkwZGZlNmJhMzUy&t=3DSlR= 2SzN4czZueWZRZGdubVA0Z2M4M2FGbC9YLzIrWEVZaUpTMEhaZUJLND0=3D&h=3D929febb5b14= 4486493bd3fc3c8522cfe ? P.S. patches should not touch the ChangeLog file. It was always wrong, because it usually makes the patch fail to apply. Since we moved to Git the ChangeLog files are automatically generated from the Git commits anyway, so are never touched as part of the commit. The changelog entry is still needed, but should be in the Git commit message not as a patch against the actual ChangeLog file. > > Also, if GCC is failing to diagnose the invalid comparisons here then > that should be reported to bugzilla as a c++ "accepts-invalid" bug. Please consider the environment and think before you print. The University of the West of Scotland is a registered Scottish charity. Ch= arity number SC002520. This e-mail and any attachment is for authorised use by the intended recipi= ent(s) only. It may contain proprietary material, confidential information = and/or be subject to legal privilege. It should not be copied, disclosed to= , retained or used by, any other party. If you are not an intended recipien= t then please promptly delete this e-mail and any attachment and all copies= and inform the sender. Please note that any views or opinions presented in this email are solely t= hose of the author and do not necessarily represent those of the University= of the West of Scotland. As a public body, the University of the West of Scotland may be required to= make available emails as well as other written forms of information as a r= esult of a request made under the Freedom of Information (Scotland) Act 200= 2.