[PATCH] D125899: [ADT] Add copy constructor to IntervalMap

[Dimitry Andric via Phabricator via llvm-commits]

dim added a comment.

In D125899#3531250 <https://reviews.llvm.org/D125899#3531250>, @kpn wrote:

> Will this fix FreeBSD 12 (and maybe 11?). FreeBSD 12 is still very much supported.

Yes, we have been carrying the `_LIBCPP_TRIVIAL_PAIR_COPY_CTOR` define since ~2013, so this copy constructor workaround will work for pretty old FreeBSD versions.

That said, we do want to get rid of this wart for FreeBSD 14.0, so I'm planning a libc++ review to make the define optional. Then we can turn it off in time for 14.0, and bump libc++.so to .2, so we don't get stuck with it for another few years. :)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125899/new/

https://reviews.llvm.org/D125899