cseriildi wrote:
Hi @kadircet,
when you have time could you take a look on this? I think it is done now.
https://github.com/llvm/llvm-project/pull/147720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
@@ -3426,7 +3851,6 @@ SYMBOL(duration, std::chrono::, )
SYMBOL(duration_cast, std::chrono::, )
SYMBOL(duration_values, std::chrono::, )
SYMBOL(file_clock, std::chrono::, )
-SYMBOL(file_seconds, std::chrono::, )
cseriildi wrote:
I honestly think this doesn't ex
@@ -690,6 +743,14 @@ SYMBOL(atol, None, )
SYMBOL(atoll, std::, )
SYMBOL(atoll, None, )
SYMBOL(atoll, None, )
+SYMBOL(atomic, std::, )
+SYMBOL(atomic, std::, )
+SYMBOL(atomic, None, )
cseriildi wrote:
fixed with
[977e152](https://github.com/llvm/llvm-project/p
@@ -3775,19 +4220,17 @@ SYMBOL(push_heap, std::ranges::, )
SYMBOL(random_access_range, std::ranges::, )
SYMBOL(range, std::ranges::, )
SYMBOL(range_adaptor_closure, std::ranges::, )
-SYMBOL(range_const_reference_t, std::ranges::, )
-SYMBOL(range_difference_t, std::ranges::, )
-
cseriildi wrote:
@kadircet should I modify the failing unit test or what is the best course of
action?
https://github.com/llvm/llvm-project/pull/147720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
@@ -279,154 +17,55 @@ SYMBOL(abs, None, )
SYMBOL(abs, None, )
SYMBOL(abs, None, )
-// Only add headers for the generic atomic template.
-// Ignore variants (std::weak_ptr, std::shared_ptr).
-SYMBOL(atomic, std::, )
-// atomic_* family symbols. is for C compatibility.
-SYMBOL(
@@ -1234,15 +1234,6 @@ SYMBOL(FILE, None, )
SYMBOL(_Exit, std::, )
SYMBOL(_Exit, None, )
SYMBOL(_Exit, None, )
-SYMBOL(abs, std::, )
cseriildi wrote:
For `abs` the only variants on cppreference are `std::complex` and
`std::valarray`, adding those would not pr
@@ -690,6 +1391,23 @@ SYMBOL(atol, None, )
SYMBOL(atoll, std::, )
SYMBOL(atoll, None, )
SYMBOL(atoll, None, )
+SYMBOL(atomic, std::, )
+SYMBOL(atomic, std::, )
+SYMBOL(atomic_bool, std::, )
+SYMBOL(atomic_bool, std::, )
+SYMBOL(atomic_bool, std::, )
cseriildi w
@@ -2525,6 +3589,10 @@ SYMBOL(remainderf, None, )
SYMBOL(remainderl, std::, )
SYMBOL(remainderl, None, )
SYMBOL(remainderl, None, )
+SYMBOL(remove, std::, )
+SYMBOL(remove, std::, )
cseriildi wrote:
I agree. We don't store the page for the symbols but I can ch
@@ -690,6 +1391,23 @@ SYMBOL(atol, None, )
SYMBOL(atoll, std::, )
SYMBOL(atoll, None, )
SYMBOL(atoll, None, )
+SYMBOL(atomic, std::, )
+SYMBOL(atomic, std::, )
+SYMBOL(atomic_bool, std::, )
+SYMBOL(atomic_bool, std::, )
+SYMBOL(atomic_bool, std::, )
cseriildi w
@@ -232,4 +247,4 @@ def GetSymbols(parse_pages):
finally:
pool.terminate()
pool.join()
-return sorted(symbols)
+return sorted(set(symbols))
cseriildi wrote:
Some symbols are on multiple pages, like `wstring_convert` on
`symbol_inde
cseriildi wrote:
Hi @kadircet,
this might be of interest to you.
https://github.com/llvm/llvm-project/pull/147720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cseriildi closed
https://github.com/llvm/llvm-project/pull/144751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cseriildi edited
https://github.com/llvm/llvm-project/pull/147720
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cseriildi updated
https://github.com/llvm/llvm-project/pull/144751
From eabf81f66a57e099139a579d1955195055a6f75b Mon Sep 17 00:00:00 2001
From: Ildiko Cseri
Date: Wed, 18 Jun 2025 12:58:34 +
Subject: [PATCH] [include-cleaner] Update generated include mappings
Update gene
https://github.com/cseriildi updated
https://github.com/llvm/llvm-project/pull/144748
From c4681dfa8da9ba90418525f32a3d1968366174d2 Mon Sep 17 00:00:00 2001
From: Ildiko Cseri
Date: Wed, 18 Jun 2025 12:58:34 +
Subject: [PATCH] [include-cleaner] Add POSIX symbol mappings
Add POSIX symbol-to
https://github.com/cseriildi edited
https://github.com/llvm/llvm-project/pull/144751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cseriildi edited
https://github.com/llvm/llvm-project/pull/144748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cseriildi edited
https://github.com/llvm/llvm-project/pull/144748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cseriildi edited
https://github.com/llvm/llvm-project/pull/144748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cseriildi wrote:
Hi @kadircet,
We noticed you’ve been involved in this area, so we thought this PR might be of
interest to you.
https://github.com/llvm/llvm-project/pull/144748
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
https://github.com/cseriildi updated
https://github.com/llvm/llvm-project/pull/144748
From c4681dfa8da9ba90418525f32a3d1968366174d2 Mon Sep 17 00:00:00 2001
From: Ildiko Cseri
Date: Wed, 18 Jun 2025 12:58:34 +
Subject: [PATCH] [include-cleaner] Add POSIX symbol mappings
Add POSIX symbol-to
https://github.com/cseriildi created
https://github.com/llvm/llvm-project/pull/144748
### **Summary**
This PR adds more symbol-header mappings that `clang-include-cleaner`, `clangd`
and `clang-tidy` use to detect symbols that are used but not directly included
([following the include-what-you
23 matches
Mail list logo
