Mordante added a comment.

Since this patch requires both an update to both Clang and libc++ I think it 
would land the Clang part in a separate patch. Then wait for the CI to have ToT 
with this version and land that separately.
For now can you run the CI with the Bootstrap build?



================
Comment at: libcxx/include/__memory/construct_at.h:40
+    // This implements https://wg21.link/LWG3436, which hasn't been voted into 
the Standard yet.
+    if constexpr (is_array_v<_Tp>)
+        return _VSTD::launder(__location);
----------------
Please add here also you implement the resolution of 2021-01-16.
I was wondering whether we should add it to the status list for C++23, but 
looking at the issue it's not certain it will be in C++23.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114649

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to