[libcxx] [llvm] [libc++] Diagnose when nullptrs are passed to string APIs (PR #122790)

[Mark de Wever via llvm-commits]

================
@@ -1037,13 +1037,14 @@ public:
 #  endif // _LIBCPP_CXX03_LANG
 
   template <__enable_if_t<__is_allocator<_Allocator>::value, int> = 0>
-  _LIBCPP_HIDE_FROM_ABI _LIBCPP_CONSTEXPR_SINCE_CXX20 basic_string(const _CharT* __s) {
+  _LIBCPP_HIDE_FROM_ABI _LIBCPP_CONSTEXPR_SINCE_CXX20 basic_string(const _CharT* _LIBCPP_DIAGNOSE_NONNULL __s) {
----------------
mordante wrote:

Can the `_LIBCPP_ASSERT_NON_NULL` still trigger when using `_Nonnull`?

https://github.com/llvm/llvm-project/pull/122790