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.129.124]) by sourceware.org (Postfix) with ESMTPS id 82245384645E for ; Wed, 17 May 2023 10:07:28 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 82245384645E 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=1684318048; 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=bGFp16chj9jtPVd3O7Yy9owEb6ggKkG8kDdjLBxrZkC+3fnXqs1zb10IaG1Fic3Y7a2MjN OWOOtuh+AKTOFtzt0jvOsDGY3q9r5HcC//mv52HlfABzAv80Jf+oFx4kgrnIECd0PGlrE4 1lLP7lVhFkLggVpvEEfPbywOufIgVEM= Received: from mail-lj1-f200.google.com (mail-lj1-f200.google.com [209.85.208.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-15-ISi59r0jPv-O4woS1nMOjw-1; Wed, 17 May 2023 06:07:26 -0400 X-MC-Unique: ISi59r0jPv-O4woS1nMOjw-1 Received: by mail-lj1-f200.google.com with SMTP id 38308e7fff4ca-2ac89cef5e8so2440271fa.1 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=fxG7+nd3A5MQZi81fx7CygQoiytv640RD2BA40CbErnUzOMpt303ZrkqmBd4mS8sbR TV7Ur+X1YKqeCszh9aexZc2iHV4hrBuAfEuXb+7yBLnJMA2j1DtYYE34Z9C0pLfmhpbK zahgnpZ/sTXxe6t4jmoy5YoagNDBCLs7FMB7qdtx1gmllr+QaNOmWKpmYuLJ5XeHKkm8 /pc9WW0hSyNGyHR/nqcsx7muXj08iuWdN8MSYaRIGrkyamBUgsSJc8x911LVTrPcKflH hG2WXtf+6iNv9KImi/dd2cc74PQ1xVtecFEA1jkkQhnetu308odZjW17NjmaRASJn1jZ NBug== X-Gm-Message-State: AC+VfDzfKR33KrWe3Bv8JLmxTJirfeLdwGwSCXr5b/VLBINBNjlNhzbn 9EOFyIz6C+qCaVEIC0HMSl+qfDzZmEiGB0tDX85OT6LFnGC+I/Dk+ys/1Ct3rUynGCfLRSzULFz yDkSyWKPfcMovI8+dIbTsj6rQZr7cV5clfQ== X-Received: by 2002:a2e:894a:0:b0:2ac:8522:d601 with SMTP id b10-20020a2e894a000000b002ac8522d601mr10362339ljk.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.2 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,SPF_HELO_NONE,SPF_NONE,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 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--