[libcxx-commits] [PATCH] D131362: [libc++] Implement `operator<=> for `thread::id`

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added inline comments.


================
Comment at: libcxx/include/__threading_support:651
+#else // _LIBCPP_STD_VER <= 17
+    friend _LIBCPP_INLINE_VISIBILITY
+        strong_ordering operator<=>(__thread_id __x, __thread_id __y) noexcept
----------------
avogelsgesang wrote:
> Mordante wrote:
> > Mordante wrote:
> > > I read your reasoning about why you did it, but I think we should fix it. There's already a bug report regarding this issue.
> > > https://github.com/llvm/llvm-project/issues/56187
> > For new code please use `_LIBCPP_HIDE_FROM_ABI` instead, both do the same but this name is the new name.
> Do you want me to turn the friends into non-friends as part of this commit or should I use a separate commit for this?
I prefer a separate commit for the unfriending, but I like to do the spaceship operator the proper way.
Feel free to assign the bug to yourself I haven't started it; I mainly self-assigned since I intended to work on the spaceship operators. But I have enough other things on my todo list.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131362