[all-commits] [llvm/llvm-project] 2ac6ba: [libc++] Move __libcpp_erase_if_container into <it...

[Quuxplusone via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2ac6babcc007ccbe7f18d71cd6188c925cf25813
      https://github.com/llvm/llvm-project/commit/2ac6babcc007ccbe7f18d71cd6188c925cf25813
  Author: Arthur O'Dwyer <arthur.j.odwyer at gmail.com>
  Date:   2021-03-22 (Mon, 22 Mar 2021)

  Changed paths:
    M libcxx/include/functional
    M libcxx/include/iterator
    M libcxx/include/map
    M libcxx/include/set
    M libcxx/include/unordered_map
    M libcxx/include/unordered_set

  Log Message:
  -----------
  [libc++] Move __libcpp_erase_if_container into <iterator>, and ADL-proof it.

The container headers don't need to include <functional> for any other reason
(or at least, they wouldn't if we moved `less` and `equal_to` out of <functional>),
so let's put `__libcpp_erase_if_container` somewhere that's common to the
containers but outside of <functional>.

Also, calling `std::erase_if(c, pred)` should not trigger ADL.

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