[all-commits] [llvm/llvm-project] d956ed: [libc++] Guard the fix to CityHash behind ABI v2

Owen via All-commits all-commits at lists.llvm.org
Thu Feb 9 23:20:00 PST 2023

  Branch: refs/heads/release/16.x
  Home:   https://github.com/llvm/llvm-project
  Commit: d956ed0d76afd1a0f778faad920cc4a52618b5f8
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2023-02-10 (Fri, 10 Feb 2023)

  Changed paths:
    M libcxx/include/__config
    M libcxx/include/__functional/hash.h
    M libcxx/test/libcxx/utilities/utility/__murmur2_or_cityhash.pass.cpp

  Log Message:
  [libc++] Guard the fix to CityHash behind ABI v2

As explained in a comment in https://reviews.llvm.org/D134124, we tried
landing this unconditionally but this actually bit some users who were
sharing std::unordered_map across an ABI boundary. This shows that the
ABI break is not benign and it should be guarded behind ABI v2.

Differential Revision: https://reviews.llvm.org/D143688

(cherry picked from commit 02718433a0dd3f8d3f3719b97aa1685b699ac785)

  Commit: 073506d8c15ccd67307f2740015bbdcbb6e2f69f
  Author: Owen Pan <owenpiano at gmail.com>
  Date:   2023-02-10 (Fri, 10 Feb 2023)

  Changed paths:
    M clang/lib/Format/TokenAnnotator.cpp
    M clang/unittests/Format/FormatTest.cpp

  Log Message:
  [clang-format] Insert a space between a numeric UDL and a dot

Fixes #60576.

Differential Revision: https://reviews.llvm.org/D143546

(cherry picked from commit b4e35c63711b4bde10b83607d04dab4bfdb86482)

Compare: https://github.com/llvm/llvm-project/compare/5036912174ac...073506d8c15c

More information about the All-commits mailing list