https://github.com/mordante approved this pull request.
Thanks, LGTM!
https://github.com/llvm/llvm-project/pull/126352
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante approved this pull request.
Thanks! The changes LGTM, but I haven't verified the commands work as described.
https://github.com/llvm/llvm-project/pull/89042
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://l
mordante wrote:
Thanks for the fix!
https://github.com/llvm/llvm-project/pull/88312
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante closed
https://github.com/llvm/llvm-project/pull/88613
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante commented:
Thanks for the review!
https://github.com/llvm/llvm-project/pull/88613
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,194 @@
+//===--
LibCxxProxyArray.cpp---===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/88613
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante closed
https://github.com/llvm/llvm-project/pull/88312
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante created
https://github.com/llvm/llvm-project/pull/88613
These proxies are returned by operator[](...). These proxies all "behave" the
same. They store a pointer to the data of the valarray they are a proxy for and
they have an internal array of indices. This interna
Author: Mark de Wever
Date: 2024-04-13T12:03:18+02:00
New Revision: a33a754cff02fda7520fa63fbc6c85c274a44baa
URL:
https://github.com/llvm/llvm-project/commit/a33a754cff02fda7520fa63fbc6c85c274a44baa
DIFF:
https://github.com/llvm/llvm-project/commit/a33a754cff02fda7520fa63fbc6c85c274a44baa.diff
https://github.com/mordante closed
https://github.com/llvm/llvm-project/pull/88178
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/88178
>From 5140ce2929ca56c865fc9c02fb0849ba06727c56 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Tue, 19 Mar 2024 20:31:41 +0100
Subject: [PATCH] [lldb][libc++] Adds local_t clock data formatters.
---
.../Lan
@@ -1068,6 +1068,29 @@ static void
LoadLibCxxFormatters(lldb::TypeCategoryImplSP cpp_category_sp) {
eTypeOptionCascade,
true);
+ AddCXXSummary(
+ cpp_category_sp,
+ lldb_private::formatters::LibcxxChronoLocalSecondsSummaryProvide
https://github.com/mordante ready_for_review
https://github.com/llvm/llvm-project/pull/88312
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -751,6 +751,8 @@ def setUpCommands(cls):
"settings set symbols.enable-external-lookup false",
# Inherit the TCC permissions from the inferior's parent.
"settings set target.inherit-tcc true",
+# Based on
https://discourse.llvm
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/88312
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante created
https://github.com/llvm/llvm-project/pull/88312
None
>From a94781dd9a993d88dc1eb0897eade23fb4acdf4e Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Wed, 10 Apr 2024 21:12:29 +0200
Subject: [PATCH] [libc++][CI] Tests LLDB libc++ data formatters.
---
libc
https://github.com/mordante created
https://github.com/llvm/llvm-project/pull/88178
None
>From 656ee030e38be24732dd78f96204ce12fe40e788 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Tue, 19 Mar 2024 20:31:41 +0100
Subject: [PATCH] [lldb][libc++] Adds local_t clock data formatters.
---
.
https://github.com/mordante approved this pull request.
I wasn't aware LLDB's print accepts `%zu`, thanks for the fix!
https://github.com/llvm/llvm-project/pull/85763
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-
mordante wrote:
Thanks for the review!
https://github.com/llvm/llvm-project/pull/85544
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante closed
https://github.com/llvm/llvm-project/pull/85544
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/85544
>From 52d525c8377fe36d2c8c7705736482ee6a021366 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Sat, 16 Mar 2024 17:25:32 +0100
Subject: [PATCH 1/2] [lldb][libc++] Adds slice_array data formatters.
---
.../P
https://github.com/mordante created
https://github.com/llvm/llvm-project/pull/85544
None
>From 52d525c8377fe36d2c8c7705736482ee6a021366 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Sat, 16 Mar 2024 17:25:32 +0100
Subject: [PATCH] [lldb][libc++] Adds slice_array data formatters.
---
...
mordante wrote:
> Looks good. I see the PR test for "Test documentation build" is failing, but
> that's an issue with the bot and pexpect.
Is this a known issue and is somebody already working on fixing this?
https://github.com/llvm/llvm-project/pull/84630
_
https://github.com/mordante closed
https://github.com/llvm/llvm-project/pull/84630
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante created
https://github.com/llvm/llvm-project/pull/84630
Recently building libc++ requires building libunwind too. This updates the LLDB
instructions.
I noticed this recently and it was separately filed as
https://github.com/llvm/llvm-project/issues/84053
>From 994
https://github.com/mordante closed
https://github.com/llvm/llvm-project/pull/80609
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,140 @@
+//===-- LibCxxValarray.cpp
===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/80609
>From 661f43e0aa40b29ca8bde6c643ee1f3241ecfb17 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Sun, 4 Feb 2024 18:46:50 +0100
Subject: [PATCH] [lldb][libc++] Adds valarray data formatters.
The code is heavil
@@ -750,6 +750,11 @@ static void LoadLibCxxFormatters(lldb::TypeCategoryImplSP
cpp_category_sp) {
lldb_private::formatters::LibcxxStdVectorSyntheticFrontEndCreator,
"libc++ std::vector synthetic children",
"^std::__[[:alnum:]]+::vector<.+>$", stl_deref_flags,
mordante wrote:
Thanks for the fix!
https://github.com/llvm/llvm-project/pull/81078
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante created
https://github.com/llvm/llvm-project/pull/80609
The code is heavily based on the vector data formatter.
>From 12b03c9ff9f6ddb0a011d8f958e3a826bd7603b9 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Sun, 4 Feb 2024 18:46:50 +0100
Subject: [PATCH] [lldb][l
mordante wrote:
@jasonmolenda I'm not using a mac, but based on your results I expect that
`std::size_t` is a 64-bit _unsigned_ integer type. Is that correct?
https://github.com/llvm/llvm-project/pull/78609
___
lldb-commits mailing list
lldb-commits@l
mordante wrote:
> @mordante I guess the PR missed LLVM18 release. I'll update the release notes
> accordingly. Is that OK?
Yes. It would have been nice to get this in LLVM 18, but I don't feel it's
critical to backport it. (Especially since it's a C++26 only feature and I
expect very few peop
https://github.com/mordante closed
https://github.com/llvm/llvm-project/pull/78609
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante updated
https://github.com/llvm/llvm-project/pull/78609
>From 31a7604f872116521558dcb9f1b4c2c4f5f9d439 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Thu, 18 Jan 2024 19:21:09 +0100
Subject: [PATCH 1/2] [lldb][libc++] Adds system_clock data formatters.
---
.../
@@ -32,6 +32,106 @@ def test_with_run_command(self):
self.expect("frame variable m", substrs=["m = 4321 months"])
self.expect("frame variable y", substrs=["y = 321 years"])
+self.expect(
+"frame variable ss_tp",
+substrs=["ss_tp
@@ -1073,18 +1073,79 @@ bool
lldb_private::formatters::LibcxxWStringViewSummaryProvider(
bool success;
ValueObjectSP dataobj;
size_t size;
- std::tie( success, dataobj, size ) = LibcxxExtractStringViewData(valobj);
+ std::tie(success, dataobj, size) = LibcxxExtractStri
@@ -0,0 +1,397 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/77967
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante approved this pull request.
LGTM modulo one typo.
https://github.com/llvm/llvm-project/pull/77967
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,458 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,172 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,458 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,172 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
https://github.com/mordante requested changes to this pull request.
> I did some re-imagining of the tests, there is a some redundancy but it is
> cleared that nothing was missed. The tests are formatted in tabular form
> manually to make them easier to read.
Redundant code in tests when if i
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/77967
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante approved this pull request.
Still LGTM!
https://github.com/llvm/llvm-project/pull/76449
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/76447
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1513,6 +1519,11 @@
__sanitizer_verify_double_ended_contiguous_container(const void*, const void*, c
#define _LIBCPP_DISABLE_UBSAN_UNSIGNED_INTEGER_CHECK
# endif
+// Clang-18 has support for deducing this, but it does not set the FTM.
+#if defined(__cpp_explicit_this_
@@ -7,7 +7,8 @@
// UNSUPPORTED: c++03, c++11, c++14, c++17, c++20, c++23
// UNSUPPORTED: GCC-ALWAYS_INLINE-FIXME
-// XFAIL: clang-16 || clang-17
+// The tested functionality needs deducing this.
+// UNSUPPORTED: clang-16 || clang-17 || apple-clang
mordante wro
https://github.com/mordante approved this pull request.
Thanks LGTM modulo a few minor comment.
https://github.com/llvm/llvm-project/pull/76449
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
@@ -470,4 +470,9 @@ inline Tp const& DoNotOptimize(Tp const& value) {
# define TEST_IF_AIX(arg_true, arg_false) arg_false
#endif
+// Clang-18 has support for deducing this, but it does not set the FTM.
+#if defined(__cpp_explicit_this_parameter) || (defined(_LIBCPP_CLANG_VER)
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/76449
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1273,6 +1293,24 @@ public:
__impl_.__swap(__that.__impl_);
}
+# if _LIBCPP_STD_VER >= 26 && defined(_LIBCPP_HAS_EXPLICIT_THIS_PARAMETER)
+ struct __variant_visit_barrier_tag { // unnamable when using standard
library modules
mordante wrote:
I'm n
@@ -1130,6 +1137,19 @@ using __best_match_t = typename
invoke_result_t<_MakeOverloads<_Types...>, _Tp,
} // namespace __variant_detail
+template < class _Visitor, class... _Vs, typename =
void_t()))...> >
+_LIBCPP_HIDE_FROM_ABI _LIBCPP_AVAILABILITY_THROW_BAD_VARIANT_ACCESS
@@ -0,0 +1,357 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -1267,7 +1267,11 @@ def add_version_header(tc):
},
{
"name": "__cpp_lib_variant",
-"values": {"c++17": 202102},
+"values": {
+"c++17": 202102, # std::visit for classes derived from
std::variant
+
@@ -0,0 +1,48 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apac
https://github.com/mordante requested changes to this pull request.
Mostly LGTM, I like to see it again for a quick look.
https://github.com/llvm/llvm-project/pull/76447
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/c
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/76447
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -1130,6 +1137,19 @@ using __best_match_t = typename
invoke_result_t<_MakeOverloads<_Types...>, _Tp,
} // namespace __variant_detail
+template < class _Visitor, class... _Vs, typename =
void_t()))...> >
mordante wrote:
Clang-format needs to be compatible
@@ -1130,6 +1137,19 @@ using __best_match_t = typename
invoke_result_t<_MakeOverloads<_Types...>, _Tp,
} // namespace __variant_detail
+template < class _Visitor, class... _Vs, typename =
void_t()))...> >
+_LIBCPP_HIDE_FROM_ABI _LIBCPP_AVAILABILITY_THROW_BAD_VARIANT_ACCESS
@@ -144,7 +144,8 @@ _LIBCPP_HIDE_FROM_ABI decltype(auto)
__visit_format_arg(_Visitor&& __vis, basic_
__libcpp_unreachable();
}
-# if _LIBCPP_STD_VER >= 26
+# if _LIBCPP_STD_VER >= 26 && (!defined(_LIBCPP_COMPILER_CLANG_BASED) ||
_LIBCPP_CLANG_VER >= 1800)
---
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/78215
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -555,6 +556,7 @@ inline _LIBCPP_HIDE_FROM_ABI
_LIBCPP_AVAILABILITY_THROW_BAD_ANY_CAST _ValueType
template
inline _LIBCPP_HIDE_FROM_ABI add_pointer_t>
any_cast(any const* __any) _NOEXCEPT {
+ static_assert(!is_void_v<_ValueType>, "_ValueType may not be void.");
-
https://github.com/mordante approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/78215
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -144,7 +144,8 @@ _LIBCPP_HIDE_FROM_ABI decltype(auto)
__visit_format_arg(_Visitor&& __vis, basic_
__libcpp_unreachable();
}
-# if _LIBCPP_STD_VER >= 26
+# if _LIBCPP_STD_VER >= 26 && (!defined(_LIBCPP_COMPILER_CLANG_BASED) ||
_LIBCPP_CLANG_VER >= 1800)
---
@@ -0,0 +1,36 @@
+//===--===//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2
@@ -267,7 +366,8 @@ class _LIBCPP_TEMPLATE_VIS
basic_format_arg<_Context>::handle {
// This function is user facing, so it must wrap the non-standard types of
// the "variant" in a handle to stay conforming. See __arg_t for more details.
template
-_LIBCPP_HIDE_FROM_ABI declty
@@ -19,19 +19,25 @@
#include "test_macros.h"
#include "make_string.h"
+#if _LIBCPP_STD_VER >= 26
+TEST_CLANG_DIAGNOSTIC_IGNORED("-Wdeprecated-declarations")
+#endif
+
template
void test(From value) {
auto store = std::make_format_args(value);
const std::basic_format_a
@@ -7,6 +7,7 @@
// UNSUPPORTED: c++03, c++11, c++14, c++17, c++20, c++23
// UNSUPPORTED: GCC-ALWAYS_INLINE-FIXME
+// XFAIL: clang-16 || clang-17
mordante wrote:
We know it fails so there is no real need to test that.
Quite often we do XFAIL when we want to up
@@ -163,20 +163,26 @@ class _LIBCPP_TEMPLATE_VIS basic_format_context basic_format_arg {
-if constexpr (same_as)
- return {};
-else if constexpr (same_as::handle>)
- // At the moment it's not possible for formatting
@@ -193,7 +194,8 @@ _LIBCPP_HIDE_FROM_ABI _Rp __visit_format_arg(_Visitor&&
__vis, basic_format_arg<
__libcpp_unreachable();
}
-# endif
+
+# endif // if _LIBCPP_STD_VER >= 26
mordante wrote:
```suggestion
# endif // if _LIBCPP_STD_VER >= 26 && defined(
@@ -19,19 +19,25 @@
#include "test_macros.h"
#include "make_string.h"
+#if _LIBCPP_STD_VER >= 26
+TEST_CLANG_DIAGNOSTIC_IGNORED("-Wdeprecated-declarations")
+#endif
+
template
void test(From value) {
auto store = std::make_format_args(value);
const std::basic_format_a
@@ -163,20 +163,26 @@ class _LIBCPP_TEMPLATE_VIS basic_format_context basic_format_arg {
-if constexpr (same_as)
- return {};
-else if constexpr (same_as::handle>)
- // At the moment it's not possible for formatting
https://github.com/mordante requested changes to this pull request.
In general this looks good. I'd like to have another look after addressing the
comments.
https://github.com/llvm/llvm-project/pull/76449
___
lldb-commits mailing list
lldb-commits@lis
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/76449
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante created
https://github.com/llvm/llvm-project/pull/78609
None
>From 31a7604f872116521558dcb9f1b4c2c4f5f9d439 Mon Sep 17 00:00:00 2001
From: Mark de Wever
Date: Thu, 18 Jan 2024 19:21:09 +0100
Subject: [PATCH] [lldb][libc++] Adds system_clock data formatters.
---
..
@@ -0,0 +1,281 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,267 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,281 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,119 @@
+// -*- C++ -*-
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License
@@ -0,0 +1,267 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,267 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,281 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,157 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/77967
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
@@ -0,0 +1,76 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apac
@@ -0,0 +1,75 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apac
https://github.com/mordante requested changes to this pull request.
Thanks I think this is getting close, I would like to see it again after
addressing the comments.
https://github.com/llvm/llvm-project/pull/77967
___
lldb-commits mailing list
lldb-co
@@ -5,39 +5,134 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===--===//
+
// UNSUPPORTED: c++03, c++11, c++14, c++17
//
-#include
+// constexpr explicit(extent != dynamic_extent)
@@ -0,0 +1,38 @@
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apac
https://github.com/mordante edited
https://github.com/llvm/llvm-project/pull/78157
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
https://github.com/mordante approved this pull request.
> Thanks. The documentation build fails due to some CMake error consistently
> otherwise I believe I addressed all comments.
I think I fixed this a few days ago.
LGTM modulo 1 nit.
https://github.com/llvm/llvm-project/pull/78157
___
mordante wrote:
> > I see you implemented this code only for C++26. In general we retroactively
> > apply LWG issues to all previous standard versions. Is there a reason why
> > you only applied this to C++26?
>
> I am not familiar with this process. I'm happy to do what's needed to be
> done
@@ -0,0 +1,119 @@
+// -*- C++ -*-
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License
@@ -0,0 +1,119 @@
+// -*- C++ -*-
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License
@@ -0,0 +1,119 @@
+// -*- C++ -*-
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License
@@ -0,0 +1,119 @@
+// -*- C++ -*-
+//===--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License
1 - 100 of 153 matches
Mail list logo