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 9473D3858425 for ; Fri, 23 Dec 2022 09:29:24 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 9473D3858425 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=1671787764; 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=PmbbkIjR3uE8nIZha5N0AHIhtzDxic+4LdyjYdo8jrY=; b=L5xirAJcn4fNlUL8JM+fC1Ybcfgf1GO45fVy9HetX+BG7ATPKf7vyAt072Glqt/l2VMzDj phahYb/G9PNJ17QiQSpwSS/bIT+Mb1uSNuI9AfeowBHRsOzat0xC2xzKAMg14wV5N6kxW1 8+d/DyTX25eK3fKDw/7d/82Vw1Ac4o0= Received: from mail-lf1-f70.google.com (mail-lf1-f70.google.com [209.85.167.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-498-e0YDIsSdPsqdqlNsoMUpiw-1; Fri, 23 Dec 2022 04:29:22 -0500 X-MC-Unique: e0YDIsSdPsqdqlNsoMUpiw-1 Received: by mail-lf1-f70.google.com with SMTP id z8-20020ac25de8000000b004c66d7fb5baso1853771lfq.21 for ; Fri, 23 Dec 2022 01:29:22 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=PmbbkIjR3uE8nIZha5N0AHIhtzDxic+4LdyjYdo8jrY=; b=TLM/M8/ULXCJ+rpHD8sm7qolIBhJvqYs+7EDUrbUHrb5Goa1Vp5outeBmPap165UIw XysPL1wpdJYsCYCEcJkn2YsWUj9rvne+qDGeESzqdeupfKkcERjoQZ5+J/aueefsVW1b HsVK0HoD0qSClBK/utlFGFDbbF7IoblbhBuVQcn94i/Rxk0hd8zVVJKfcNB+MRACbnNa mjxNVMECn3q1vma70KPeAD1oBUbldmQAAfztF80qzkKLlvpi5ucYH/cyf6VmWOPhvfBq ZW3zEDE5P+jevF2lZDVbscQz7lQSaWqkSihYMNDU0tMx5566TGXquNuR9VkNXujcWDp1 Oywg== X-Gm-Message-State: AFqh2kqyeprJUahDhofOEdPnOw0me/syturr/gTuM5uCGx959vf3maIh F3wJ1NXTV7gHAqVDJHq1aosz+3H5TGv40JqCIOu5ssw3riKlfT6K/bOS9W7Dblm/TqF3SiWUvbG RPwwVOCth+2rQieA3atrLoYeTTTtMxN4= X-Received: by 2002:a05:6512:614:b0:4b0:65b0:7f30 with SMTP id b20-20020a056512061400b004b065b07f30mr452756lfe.385.1671787761525; Fri, 23 Dec 2022 01:29:21 -0800 (PST) X-Google-Smtp-Source: AMrXdXuSq6RNuiT54N32pbTJ1qaxBKv/wUkj1Rj8RTOvJRf3PmMhTIjKh4hIadCafWVgdqUlrI4R1ksAcD8qAn9ejNE= X-Received: by 2002:a05:6512:614:b0:4b0:65b0:7f30 with SMTP id b20-20020a056512061400b004b065b07f30mr452752lfe.385.1671787761153; Fri, 23 Dec 2022 01:29:21 -0800 (PST) MIME-Version: 1.0 References: <20221222233704.772013-1-jwakely@redhat.com> <20221223021520.B89212043F@pchp3.se.axis.com> In-Reply-To: <20221223021520.B89212043F@pchp3.se.axis.com> From: Jonathan Wakely Date: Fri, 23 Dec 2022 09:29:09 +0000 Message-ID: Subject: Re: build broke, cris-elf: [committed] libstdc++: Implement C++20 time zone support in To: Hans-Peter Nilsson Cc: libstdc++@gcc.gnu.org, gcc-patches@gcc.gnu.org X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" 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,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE,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 Fri, 23 Dec 2022 at 02:15, Hans-Peter Nilsson via Libstdc++ wrote: > > > From: Jonathan Wakely via Gcc-patches > > Date: Fri, 23 Dec 2022 00:37:04 +0100 > > > This is the largest missing piece of C++20 support. Only the cxx11 ABI > > is supported, due to the use of std::string in the API for time zones. > > > libstdc++-v3/ChangeLog: > > > > * acinclude.m4 (GLIBCXX_ZONEINFO_DIR): New macro. > > * config.h.in: Regenerate. > > * config/abi/pre/gnu.ver: Export new symbols. > > * configure: Regenerate. > > * configure.ac (GLIBCXX_ZONEINFO_DIR): Use new macro. > > * include/std/chrono (utc_clock::from_sys): Correct handling > > of leap seconds. > > (nonexistent_local_time::_M_make_what_str): Define. > > (ambiguous_local_time::_M_make_what_str): Define. > > (__throw_bad_local_time): Define new function. > > (time_zone, tzdb_list, tzdb): Implement all members. > > (remote_version, zoned_time, get_leap_second_info): Define. > > * include/std/version: Add comment for __cpp_lib_chrono. > > * src/c++20/Makefile.am: Add new file. > > * src/c++20/Makefile.in: Regenerate. > > * src/c++20/tzdb.cc: New file. > > * testsuite/lib/libstdc++.exp: Define effective target tzdb. > > * testsuite/std/time/clock/file/members.cc: Check file_time > > alias and file_clock::now() member. > > * testsuite/std/time/clock/gps/1.cc: Likewise for gps_clock. > > * testsuite/std/time/clock/tai/1.cc: Likewise for tai_clock. > > * testsuite/std/time/syn_c++20.cc: Uncomment everything except > > parse. > > * testsuite/std/time/clock/utc/leap_second_info.cc: New test. > > * testsuite/std/time/exceptions.cc: New test. > > * testsuite/std/time/time_zone/get_info_local.cc: New test. > > * testsuite/std/time/time_zone/get_info_sys.cc: New test. > > * testsuite/std/time/time_zone/requirements.cc: New test. > > * testsuite/std/time/tzdb/1.cc: New test. > > * testsuite/std/time/tzdb/leap_seconds.cc: New test. > > * testsuite/std/time/tzdb_list/1.cc: New test. > > * testsuite/std/time/tzdb_list/requirements.cc: New test. > > * testsuite/std/time/zoned_time/1.cc: New test. > > * testsuite/std/time/zoned_time/custom.cc: New test. > > * testsuite/std/time/zoned_time/deduction.cc: New test. > > * testsuite/std/time/zoned_time/req_neg.cc: New test. > > * testsuite/std/time/zoned_time/requirements.cc: New test. > > * testsuite/std/time/zoned_traits.cc: New test. > > > > +++ b/libstdc++-v3/src/c++20/tzdb.cc > > > + static_assert(sizeof(datetime) == 8 && alignof(datetime) == 4); > > This broke build for cris-elf: > x/autotest/hpautotest-gcc1/gcc/libstdc++-v3/src/c++20/tzdb.cc:451:38: error: static assertion failed > 451 | static_assert(sizeof(datetime) == 8 && alignof(datetime) == 4); > | ~~~~~~~~~~~~~~~~~^~~~ > x/autotest/hpautotest-gcc1/gcc/libstdc++-v3/src/c++20/tzdb.cc:451:38: note: the comparison reduces to '(7 == 8)' > make[5]: *** [Makefile:562: tzdb.lo] Error 1 > > (and I don't think "alignof(datetime) == 4" is true either) Sorry about that, I can just remove the assertion now. I'll commit that ASAP.