ChuanqiXu added a comment. In D108696#3069723 <https://reviews.llvm.org/D108696#3069723>, @avogelsgesang wrote:
> Hi @ChuanqiXu > > sorry for what might be naive questions, but just to make sure I understand > the context of this patch correctly: Hi, you are welcome : ) > This patch fixes the look up of the `coroutine_traits`, so that clang now > search both the `std` and the `std::experimental` namespace. As such, it must > land before D109433 <https://reviews.llvm.org/D109433> since otherwise the > new `std::coroutine_traits` wouldn't be found by the compiler, correct? Yes, correct. > Also, does this patch enable me to use `clang-cl` to build programs which use > the coroutine header from Microsoft's STL > <https://github.com/microsoft/STL/blob/main/stl/inc/coroutine>? Or are there > still other incompatibilities between clang and the Microsoft STL with > regards to coroutines? After this patch, it should be able to compile <coroutine> MS STL. Otherwise, there must be a bug in either the clang part or in the MS STL part. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108696/new/ https://reviews.llvm.org/D108696 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
