From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id A941A385483D for ; Wed, 17 May 2023 10:07:30 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A941A385483D Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=redhat.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1684318050; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=A0rVueh4Nqz2cCaJtOUJTb4BEHl/Z6k0k/DhzbIPl4Q=; b=hj7hyfQHR/RzCvVJhfi+DCwca3G4MAoycSwrfl20RMRTbuWgUWRbfDoCwkfllPalMEIz2B wgue7yl4ibnvxRUEB05t3vfABYjmTOPUF8KpFSUih0YJ/0d25yA6gJ5Nifv3P18yaUe4tT z4v6jOuktvzy8IJiK3xMJXO0CwVpKs8= Received: from mail-lj1-f199.google.com (mail-lj1-f199.google.com [209.85.208.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-374-SPpWz3yUM96d1_LWKEF6mQ-1; Wed, 17 May 2023 06:07:26 -0400 X-MC-Unique: SPpWz3yUM96d1_LWKEF6mQ-1 Received: by mail-lj1-f199.google.com with SMTP id 38308e7fff4ca-2acfaf0c1ffso2405201fa.3 for ; Wed, 17 May 2023 03:07:26 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1684318045; x=1686910045; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=A0rVueh4Nqz2cCaJtOUJTb4BEHl/Z6k0k/DhzbIPl4Q=; b=Xh7X5w1sg/5IahnEp3o+uzSKhlkP6xxtGo6/ozgkYxIwJe9SgM4bEPAhMKMJHpUlor ARG/oxWqesujsYUrwhf7sDbF6GnMARY5WOgufZxRmiHMyJkfjZRAMUbyhV8W8s9WMVm4 OAia7bqPmYuzf3boymNzmI4ayVLb55rEFL+dGUb07y38HWN8X2wGiJdnI42ODx2aU5WV /je9ryqXrHO3GcZbe4UwGhX1heu53LWyAIKKnRFgve7nCFxGjkbZ/4l/lLgNCaMa9CMD rJH2cnaDrVvlHUp1gtdHA1McXGZnWKGgiXg9keknWcqG7E2BzfB+qom9JVcvsuTCW553 7aDQ== X-Gm-Message-State: AC+VfDxMokQ43AYYalYkJ1aJzKn0qgOPbSKnRcsKRbqG4wJ+jovWfDwZ lQ2f/h6fQ+3mkPA+a7UXd26kE/icMTsNjPrTWk0j8CSej7xKgaJcaMi0KhVzq3E3zPNFotcb8O0 XgbyN4A3UOIbsFSycF53LVkRk5+qfm+G9nyYrtDw= X-Received: by 2002:a2e:894a:0:b0:2ac:8522:d601 with SMTP id b10-20020a2e894a000000b002ac8522d601mr10362337ljk.43.1684318045473; Wed, 17 May 2023 03:07:25 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ6ikt7SQUQYgAxLUWzcegMb7A0Ry6Vuo0bhZv64Wy7Qh7Bf4Yf9IOZy6eior0ohGGghxjmJiTwDGXtVujpZA80= X-Received: by 2002:a2e:894a:0:b0:2ac:8522:d601 with SMTP id b10-20020a2e894a000000b002ac8522d601mr10362330ljk.43.1684318045189; Wed, 17 May 2023 03:07:25 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Jonathan Wakely Date: Wed, 17 May 2023 11:07:14 +0100 Message-ID: Subject: Re: [Ping][PATCH] libstdc++: Add missing functions to [PR79700] To: Nathaniel Shead Cc: libstdc++@gcc.gnu.org, gcc-patches@gcc.gnu.org X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: multipart/alternative; boundary="000000000000c5cd6705fbe0da47" X-Spam-Status: No, score=-6.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,HTML_MESSAGE,KAM_SHORT,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE,TXREP,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: --000000000000c5cd6705fbe0da47 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Wed, 17 May 2023 at 10:38, Nathaniel Shead wrote: > On Wed, May 17, 2023 at 10:05:59AM +0100, Jonathan Wakely wrote: > > On Wed, 17 May 2023 at 09:37, Nathaniel Shead wrote: > > > > > Now that GCC13.1 is released is it ok to merge? Thanks! > > > > > > > Yes, I've been testing this locally, but I think it needs more work > (sorry!) > > > > Looking at it again, I'm not sure why I asked for the additional tests > > because if they fail, it's a problem in libc, and there's nothing we can > > actually do about it in libstdc++. We certainly do want std::expl(0.0L) > to > > return the same thing as std::exp(0.0L), but if it doesn't, we'll just > have > > a libstdc++ test failure caused by a bug in libc. But you wrote the test > > now, so let's keep it. If we get failures for the test it will allow us > to > > inform the relevant libc maintainers that they have a bug. > > Sounds good. > > > Also, since you're contributing this under the DCO terms the new test > > should not have the FSF copyright header, unless it's a derived work of > an > > existing test with that header (and in that case it should retain the > dates > > from the copied test). I don't actually bother putting the copyright and > > license header on new tests these days. There's nothing in that test th= at > > is novel or interesting, and I think it's arguably not useful or > meaningful > > to consider it copyrighted. > > Makes sense, I was just copying from other tests in the directory. I'll > keep this in mind for the future, thanks! > Yeah, we have a mix of tests using the old conventions (with copyright and GPL headers) and new conventions (don't bother, they're not really meaningful on tests). We're unlikely to *remove* the copyright notices from the old tests, because that would require all sorts of legal wrangling, and it's not clear that the copyright holder (the FSF) would agree to it anyway. > > Finally, and most importantly, the new using-declarations in are > > not guarded by any autoconf macro. That will break targets without full > C99 > > support, e.g. djgpp declares acosf but not acosl, so the new > > "using acosl;" would be a hard error as soon as is included (and > > might even prevent GCC building on that target). So I think we need a n= ew > > autoconf check for the existence of those functions. I'm in the process > of > > reworking the autoconf macros for (due to PR 109818), which is > why > > I didn't address it for this patch yet. > > Ah, I see; yes, that would be a problem. I'm not very familiar with > autoconf, so thanks for working this out. Let me know when you've done > that if there's anything else I should do for this patch. > I hope to have an updated patch by next week, so I'll let you know once that's ready. Thanks for your patience and for pining the patch. > > > > > > On Tue, Apr 18, 2023 at 6:48=E2=80=AFPM Jonathan Wakely > > > wrote: > > > > > > > > On Mon, 17 Apr 2023 at 09:11, Nathaniel Shead < > nathanieloshead@gmail.com> > > > wrote: > > > > > > > > > > Hi, just checking whether there were any issues with this patch? > > > > > > https://gcc.gnu.org/pipermail/gcc-patches/2023-February/612386.html > > > > > > > > > > Otherwise I assume it won't be in GCC13. > > > > > > > > That's right, it's too large and invasive a change to get into GCC = 13 > > > > when only submitted in February, sorry. I'll merge it to trunk once > > > > GCC 13.1 is released though. > > > > > > > > > > > > --000000000000c5cd6705fbe0da47--