[libcxx-commits] [PATCH] D96790: [libc++] Enable the synchronization library on Apple platforms

[Louis Dionne via Phabricator via libcxx-commits]

ldionne accepted this revision.
ldionne added a comment.
This revision is now accepted and ready to land.

In D96790#2734482 <https://reviews.llvm.org/D96790#2734482>, @jfb wrote:

> I would recommend checking with @MadCoder about the implementation and ABI, since the kernel has Opinions on this. IIRC he looked at Olivier's original patch.

Yup. What I'm mostly concerned about though is the ABI between the libc++ dylib and its headers. But it looks like this isn't going to change, so I'm going to commit this now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96790