https://github.com/philnik777 ready_for_review
https://github.com/llvm/llvm-project/pull/147681
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 ready_for_review
https://github.com/llvm/llvm-project/pull/147679
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/147681
>From 2335111d19903652229a8e3fb25cd03b5ffa5e48 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Tue, 10 Jun 2025 05:56:20 +0200
Subject: [PATCH] [libc++] Add ABI flag to make __tree nodes more compact
--
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/147681
>From 920e83463665654fd7b48b5e70591fb978b7b50e Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Tue, 10 Jun 2025 05:56:20 +0200
Subject: [PATCH] [libc++] Add ABI flag to make __tree nodes more compact
--
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/147679
>From 8e5e6f010dd8c3aaab2de28b7c33c10d3cbf8a55 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Mon, 9 Jun 2025 18:17:31 +0200
Subject: [PATCH] [libc++] Add accessor functions to __tree_node_base
---
li
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/147681
>From bed3297d32d7a270d216ee3efb706b914fc32f2b Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Tue, 10 Jun 2025 05:56:20 +0200
Subject: [PATCH] [libc++] Add ABI flag to make __tree nodes more compact
--
https://github.com/philnik777 created
https://github.com/llvm/llvm-project/pull/147681
None
>From bd3919aed4aa6cae451541d2ad29f914e2785dc0 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Tue, 10 Jun 2025 05:56:20 +0200
Subject: [PATCH] [libc++] Add ABI flag to make __tree nodes more compa
https://github.com/philnik777 created
https://github.com/llvm/llvm-project/pull/147679
None
>From f1fc048e52e5c57898fcb226a1fe3c8519cb5923 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Mon, 9 Jun 2025 18:17:31 +0200
Subject: [PATCH] [libc++] Add accessor functions to __tree_node_base
-
https://github.com/philnik777 created
https://github.com/llvm/llvm-project/pull/144117
This fixes/removes a bunch of random tests. They all failed in relatively
simple to fix ways.
Specificially (all inside `libcxx/test/libcxx-03`):
- `utilities/template.bitset/includes.pass.cpp`: the header g
https://github.com/philnik777 created
https://github.com/llvm/llvm-project/pull/144110
Quite a few of the frozen header tests only fail because the include path is
incorrect due to copying the headers. This patch fixes the tests where that's
the only problem.
This is part of https://discourse
https://github.com/philnik777 created
https://github.com/llvm/llvm-project/pull/144101
The tests in `libcxx/test/libcxx` aren't run against the frozen headers
anymore, so we can remove any XFAILs in them.
This is part of https://discourse.llvm.org/t/rfc-freezing-c-03-headers-in-libc.
>From e
philnik777 wrote:
@tstellar The CI failures are most certainly unrelated.
https://github.com/llvm/llvm-project/pull/138880
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bran
https://github.com/philnik777 approved this pull request.
LGTM with added comment.
https://github.com/llvm/llvm-project/pull/142925
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/142925
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -451,6 +451,7 @@ namespace std {
# if _LIBCPP_STD_VER >= 23
#include <__fwd/mdspan.h>
+#include <__fwd/span.h>
philnik777 wrote:
Can you add a comment with the LWG issue number? If the answer is that we
indeed expect users to include `` we should
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/136752
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
@ldionne Should make sure this is correct when he is back. I'm not entirely
sure whether "latest stable release" means latest minor or latest major (also,
we're not up-to-date in the docs which XCode we're supporting).
https://github.co
philnik777 wrote:
It looks like your merge base is wrong.
https://github.com/llvm/llvm-project/pull/132291
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
LGTM assuming the diff landed is the same I see. I'm really not a fan of
complicating things unnecessarily though.
https://github.com/llvm/llvm-project/pull/132291
___
llvm-branch-commits mail
philnik777 wrote:
Could you not use spr in that case? I can't see the diff as it would be merged
AFAICT.
https://github.com/llvm/llvm-project/pull/132291
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cg
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/133010
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 ready_for_review
https://github.com/llvm/llvm-project/pull/133010
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/128366
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -0,0 +1,47 @@
+//===--===//
+//
+// 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/philnik777 requested changes to this pull request.
@denzor200 It's a lot easier for us to add a libc++-speicific clang-tidy check
than a general one. libc++ checks have significantly lower quality
requirements, since they only have to work for libc++ (making "seems to work
f
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/130433
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/129305
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/129303
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/127732
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/125996
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 milestoned
https://github.com/llvm/llvm-project/pull/125185
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 created
https://github.com/llvm/llvm-project/pull/125185
This aligns the builtins with how implementations work which don't use
the buitins.
>From a729e9190f8047b455d80224e92844b12e493e3c Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Thu, 30 Jan 2025 20:34
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/124922
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/109002
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -587,42 +587,48 @@ template
*/
-#include <__config>
-
-#include <__atomic/aliases.h>
-#include <__atomic/atomic.h>
-#include <__atomic/atomic_base.h>
-#include <__atomic/atomic_flag.h>
-#include <__atomic/atomic_init.h>
-#include <__atomic/atomic_lock_free.h>
-#include <_
@@ -1827,232 +1827,147 @@ template
*/
-#include <__config>
-
-#include <__algorithm/adjacent_find.h>
-#include <__algorithm/all_of.h>
-#include <__algorithm/any_of.h>
-#include <__algorithm/binary_search.h>
-#include <__algorithm/copy.h>
-#include <__algorithm/copy_backward.
@@ -1021,17 +1021,8 @@ set(files
configure_file("__config_site.in"
"${LIBCXX_GENERATED_INCLUDE_TARGET_DIR}/__config_site" @ONLY)
philnik777 wrote:
I think this change is fine. I'll have to modify this anyways and removing the
copying now means I won't introduc
@@ -1021,17 +1021,8 @@ set(files
configure_file("__config_site.in"
"${LIBCXX_GENERATED_INCLUDE_TARGET_DIR}/__config_site" @ONLY)
philnik777 wrote:
This isn't used _yet_.
https://github.com/llvm/llvm-project/pull/115380
_
https://github.com/philnik777 ready_for_review
https://github.com/llvm/llvm-project/pull/109001
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 ready_for_review
https://github.com/llvm/llvm-project/pull/109002
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 ready_for_review
https://github.com/llvm/llvm-project/pull/109000
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
philnik777 wrote:
> [!WARNING]
> This pull request is not mergeable via GitHub because a downstack PR is
> open. Once all requirements are satisfied, merge this PR as a stack href="https://app.graphite.dev/github/pr/llvm/llvm-project/109002?utm_source=stack-comment-downstack-mergeability-warnin
philnik777 wrote:
> [!WARNING]
> This pull request is not mergeable via GitHub because a downstack PR is
> open. Once all requirements are satisfied, merge this PR as a stack href="https://app.graphite.dev/github/pr/llvm/llvm-project/109001?utm_source=stack-comment-downstack-mergeability-warnin
philnik777 wrote:
> [!WARNING]
> This pull request is not mergeable via GitHub because a downstack PR is
> open. Once all requirements are satisfied, merge this PR as a stack href="https://app.graphite.dev/github/pr/llvm/llvm-project/109000?utm_source=stack-comment-downstack-mergeability-warnin
philnik777 wrote:
It's also worth noting that compilers can constant-fold `strlen`, so in cases
where the string is known this doesn't even result in multiple iterations.
https://github.com/llvm/llvm-project/pull/101805
___
llvm-branch-commits mailing
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/102466
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
Author: Nikolas Klauser
Date: 2024-08-02T10:53:33+02:00
New Revision: b2eab3486499656ec6ef30ace5033f80d4d9dfc9
URL:
https://github.com/llvm/llvm-project/commit/b2eab3486499656ec6ef30ace5033f80d4d9dfc9
DIFF:
https://github.com/llvm/llvm-project/commit/b2eab3486499656ec6ef30ace5033f80d4d9dfc9.dif
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/101638
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/101638
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/101638
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/101638
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/100604
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -0,0 +1,93 @@
+//===--===//
+//
+// 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
@@ -18,5 +18,9 @@
#include
#include
-// expected-error@+1 {{template template argument has different template
parameters than its corresponding template template parameter}}
-static_assert(!std::__is_specialization_v,
std::array>);
+#if defined(__clang__) && __clang_major_
@@ -0,0 +1,233 @@
+//===--===//
+//
+// 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,233 @@
+//===--===//
+//
+// 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,233 @@
+//===--===//
+//
+// 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
philnik777 wrote:
@EricWF When switching to `__libcpp_operator_{new,delete}` everything gets
compiled exactly the same on my system. Could you disclose which test you used
for this code? You also claim that the `__pointer_int_pair` causes the
optimizer to lose track of the function pointer cau
@@ -0,0 +1,46 @@
+//===--===//
+//
+// 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,233 @@
+//===--===//
+//
+// 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,46 @@
+//===--===//
+//
+// 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,233 @@
+//===--===//
+//
+// 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/philnik777 edited
https://github.com/llvm/llvm-project/pull/94670
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/89181
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/89181
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/83723
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/83723
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/80720
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/philnik777 approved this pull request.
I'm OK with this as a quick fix, since Mark promised to fix it properly in the
long term.
https://github.com/llvm/llvm-project/pull/76268
___
llvm-branch-commits mailing list
llvm-branch-commit
philnik777 wrote:
> > > Oh shit. I just realized that this is most likely a latent bug no matter
> > > what. We build the module with Clang 18, and then essentially try to load
> > > it with Clang 17 (aka Clang Tidy 17). AFAIK that's not guaranteed to
> > > work, and probably just happens to w
philnik777 wrote:
> > Oh shit. I just realized that this is most likely a latent bug no matter
> > what. We build the module with Clang 18, and then essentially try to load
> > it with Clang 17 (aka Clang Tidy 17). AFAIK that's not guaranteed to work,
> > and probably just happens to work curr
https://github.com/philnik777 approved this pull request.
https://github.com/llvm/llvm-project/pull/76659
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
philnik777 wrote:
Oh shit. I just realized that this is most likely a latent bug no matter what.
We build the module with Clang 18, and then essentially try to load it with
Clang 17 (aka Clang Tidy 17). AFAIK that's not guaranteed to work, and probably
just happens to work currently with Clang
philnik777 wrote:
I'd like to understand what the fixed issues are you're relying on. From what I
can tell, this could just as much be a bug in our setup for the clang-tidy
plugin as an actual fix in trunk.
https://github.com/llvm/llvm-project/pull/76268
___
https://github.com/philnik777 requested changes to this pull request.
I'm really not happy with bumping the clang-tidy version we use all the time to
the trunk version. We agreed to using the latest stable version, which we've
not done way too many times now. I'd really like to first understand
Author: Nikolas Klauser
Date: 2023-12-02T14:04:01+01:00
New Revision: 0e823b02ba8a0f61ddd14bb128cf8e12fe14602a
URL:
https://github.com/llvm/llvm-project/commit/0e823b02ba8a0f61ddd14bb128cf8e12fe14602a
DIFF:
https://github.com/llvm/llvm-project/commit/0e823b02ba8a0f61ddd14bb128cf8e12fe14602a.dif
76 matches
Mail list logo