[all-commits] [llvm/llvm-project] 26f7f3: [libc++][TZDB] Fixes reverse time lookups.

Mark de Wever via All-commits all-commits at lists.llvm.org
Sat Apr 20 06:06:46 PDT 2024


  Branch: refs/heads/users/mordante/makes_saturation_functions_available
  Home:   https://github.com/llvm/llvm-project
  Commit: 26f7f3abb6398a749810230f08c2ef4db5150e6f
      https://github.com/llvm/llvm-project/commit/26f7f3abb6398a749810230f08c2ef4db5150e6f
  Author: Mark de Wever <koraq at xs4all.nl>
  Date:   2024-04-20 (Sat, 20 Apr 2024)

  Changed paths:
    M libcxx/src/time_zone.cpp
    M libcxx/test/std/time/time.zone/time.zone.timezone/time.zone.members/get_info.sys_time.pass.cpp

  Log Message:
  -----------
  [libc++][TZDB] Fixes reverse time lookups.

Testing with the get_info() returning a local_info revealed some issues in
the reverse lookup. This needed an additional quirck. Also the skipping
when not in the current continuation optimization was wrong. It prevented
merging two sys_info objects.


  Commit: d1f131136d005699a882bf30af9520b06fc898b4
      https://github.com/llvm/llvm-project/commit/d1f131136d005699a882bf30af9520b06fc898b4
  Author: Mark de Wever <koraq at xs4all.nl>
  Date:   2024-04-20 (Sat, 20 Apr 2024)

  Changed paths:
    M libcxx/include/__numeric/saturation_arithmetic.h

  Log Message:
  -----------
  [libc++] Makes saturation functions privately available.

These functions are useful in the implementation of the time zone
database. So expose them with private names.

The functions could be exposed before C++ 20, but since libc++ is mostly
C++ 17 complete it seems less useful to allow earlier.


Compare: https://github.com/llvm/llvm-project/compare/26f7f3abb639%5E...d1f131136d00

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list