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

Louis Dionne via All-commits all-commits at lists.llvm.org
Thu Feb 9 17:10:34 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 02718433a0dd3f8d3f3719b97aa1685b699ac785
      https://github.com/llvm/llvm-project/commit/02718433a0dd3f8d3f3719b97aa1685b699ac785
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2023-02-09 (Thu, 09 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




More information about the All-commits mailing list