From: Hans-Peter Nilsson <hp@axis.com>
To: Jonathan Wakely <jwakely@redhat.com>
Cc: <libstdc++@gcc.gnu.org>, <gcc-patches@gcc.gnu.org>
Subject: build broke, cris-elf: [committed] libstdc++: Implement C++20 time zone support in <chrono>
Date: Fri, 23 Dec 2022 03:15:20 +0100 [thread overview]
Message-ID: <20221223021520.B89212043F@pchp3.se.axis.com> (raw)
In-Reply-To: <20221222233704.772013-1-jwakely@redhat.com> (message from Jonathan Wakely via Gcc-patches on Fri, 23 Dec 2022 00:37:04 +0100)
> From: Jonathan Wakely via Gcc-patches <gcc-patches@gcc.gnu.org>
> 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)
Happy holidays.
brgds, H-P
next prev parent reply other threads:[~2022-12-23 2:15 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-12-22 23:37 Jonathan Wakely
2022-12-23 2:15 ` Hans-Peter Nilsson [this message]
2022-12-23 9:29 ` build broke, cris-elf: " Jonathan Wakely
2022-12-23 9:44 ` Jonathan Wakely
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20221223021520.B89212043F@pchp3.se.axis.com \
--to=hp@axis.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=jwakely@redhat.com \
--cc=libstdc++@gcc.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).