================
@@ -1185,29 +1224,24 @@ class YAMLConverter {
llvm::StringSet<> KnownMethodSelectors;
for (const auto &CXXMethod : T.Methods) {
- auto WhereParameters = getWhereParameters(CXXMethod);
- if (!WhereParameters.first)
+ auto WhereSelector = getWhereSelector(CXXMethod, /*AllowObject=*/true);
+ if (!WhereSelector)
continue;
- if (WhereParameters.second) {
- if (!KnownMethodSelectors
- .insert(getFunctionSelectorKey(CXXMethod.Name,
- *WhereParameters.second))
- .second) {
+ if (WhereSelector->Parameters || WhereSelector->Object) {
+ std::string DuplicateKey =
----------------
Xazax-hun wrote:
I wonder if this name of this variable (and the corresponding functions) could
be tweaked a bit. `DuplicateKey` to me sounds like the key itself is redundant.
But I suspect we use this for deduplication. How about `UniqueKey`, or
`DeduplicationKey` or something similar?
https://github.com/llvm/llvm-project/pull/216148
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
