https://github.com/kadircet closed
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vvd170501 wrote:
@kadircet, ping. Could you commit these changes, please?
I've rebased to make sure that 68daf7d27ecc085fe7347552736197db6453f71c didn't
cause any conflicts.
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing lis
https://github.com/vvd170501 updated
https://github.com/llvm/llvm-project/pull/113612
>From 1baa1c2aebd37da2f2ccc3918fa3425698860566 Mon Sep 17 00:00:00 2001
From: vvd170501 <36827317+vvd170...@users.noreply.github.com>
Date: Sat, 26 Oct 2024 00:17:26 +0300
Subject: [PATCH] Update std symbol map
vvd170501 wrote:
> thanks a lot, lgtm!
>
> LMK if i should commit this for you
Yes, commit it please, I don't have the permissions.
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
https://github.com/vvd170501 updated
https://github.com/llvm/llvm-project/pull/113612
>From 853881983faeeb5b7e0582d3e94037c12df12c7c Mon Sep 17 00:00:00 2001
From: vvd170501 <36827317+vvd170...@users.noreply.github.com>
Date: Sat, 26 Oct 2024 00:17:26 +0300
Subject: [PATCH 1/5] Fix variant parsi
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -115,15 +115,17 @@ static int initialize(Lang Language) {
NSLen = 0;
}
-if (SymIndex >= 0 &&
-Mapping->SymbolNames[SymIndex].qualifiedName() == QName) {
- // Not a new symbol, use the same index.
+if (SymIndex > 0) {
assert(llvm::none_o
https://github.com/kadircet approved this pull request.
thanks a lot, lgtm!
LMK if i should commit this for you
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
https://github.com/vvd170501 updated
https://github.com/llvm/llvm-project/pull/113612
>From 853881983faeeb5b7e0582d3e94037c12df12c7c Mon Sep 17 00:00:00 2001
From: vvd170501 <36827317+vvd170...@users.noreply.github.com>
Date: Sat, 26 Oct 2024 00:17:26 +0300
Subject: [PATCH 1/4] Fix variant parsi
@@ -232,6 +232,37 @@ SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
+// C++ [range.access.general]: ... the customization point objects
+// in [range.access] are available when the header is included.
+SYMBOL(begin, std::ranges::, )
vvd170501 wrote:
I've reverted the optimizations, will create a new PR when this one is merged
(the commit depends on other changes that I've left in this PR)
https://github.com/llvm/llvm-project/pull/113612
___
cfe-
@@ -352,20 +383,23 @@ SYMBOL(get, std::, /*no headers*/)
// providing the type.
SYMBOL(make_error_code, std::, /*no headers*/)
SYMBOL(make_error_condition, std::, /*no headers*/)
+// Similar to std::get, has variants for multiple containers
+// (vector, deque, list, etc.)
+SYMB
@@ -352,20 +383,23 @@ SYMBOL(get, std::, /*no headers*/)
// providing the type.
SYMBOL(make_error_code, std::, /*no headers*/)
SYMBOL(make_error_condition, std::, /*no headers*/)
+// Similar to std::get, has variants for multiple containers
+// (vector, deque, list, etc.)
+SYMB
kadircet wrote:
can you send this in a separate PR ?
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -232,6 +232,37 @@ SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
+// C++ [range.access.general]: ... the customization point objects
+// in [range.access] are available when the header is included.
+SYMBOL(begin, std::ranges::, )
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -115,15 +115,17 @@ static int initialize(Lang Language) {
NSLen = 0;
}
-if (SymIndex >= 0 &&
-Mapping->SymbolNames[SymIndex].qualifiedName() == QName) {
- // Not a new symbol, use the same index.
+if (SymIndex > 0) {
assert(llvm::none_o
@@ -232,6 +232,47 @@ SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
SYMBOL(ssize, std::, )
+// C++ [range.access.general]: ... the customization point objects
+// in [range.access] are available when the header is included.
+SYMBOL(begin, std::ranges::, )
https://github.com/kadircet requested changes to this pull request.
thanks a ton for the comprehensive change, i think this looks amazing!
can you just move the parser changes into a separate PR and revert changes in
the assertion and rather delete the duplicates from stdspecialsymbolmap?
http
https://github.com/vvd170501 edited
https://github.com/llvm/llvm-project/pull/113612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
20 matches
Mail list logo
