[libcxx-commits] [PATCH] D144994: [Draft][libc++][modules] Adds std module.

Wed Apr 19 19:02:01 PDT 2023

ChuanqiXu added inline comments.

================
Comment at: libcxx/docs/Modules.rst:83-98
+  * Clang
+
+    * ``__synth_three_way_result`` does not work. There is a work-around in libc++.
+
+      * `bug report <https://github.com/llvm/llvm-project/issues/57222>`__
+      * `clang patch <https://reviews.llvm.org/D140002>`__
+      * `libc++ patch </usr/share/c++/modules/v<x>/`__
----------------
Mordante wrote:
> ChuanqiXu wrote:
> > Will these problems be blockers actually? Specially for the "header-module" order problem, we can't fix it fundamentally. And I feel 'std module' is still workable in some degree with these defects. If these problems are not blockers actually, could we move the "limitations" section?
> I think this one isn't I think we should downgrade this one to limitation.
> 
What do you mean for `this one`?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144994