[clang] [APINotes] Add Where.Object selectors for C++ methods (PR #216148)

[Gábor Horváth via cfe-commits]

================
@@ -1279,25 +1313,23 @@ class YAMLConverter {
     llvm::StringSet<> KnownNameOnlyFunctions;
     llvm::StringSet<> KnownFunctionSelectors;
     for (const auto &Function : TLItems.Functions) {
-      auto WhereParameters = getWhereParameters(Function);
-      if (!WhereParameters.first)
+      auto WhereSelector = getWhereSelector(Function, /*AllowObject=*/false);
+      if (!WhereSelector)
         continue;
 
-      if (WhereParameters.second) {
-        if (!KnownFunctionSelectors
-                 .insert(getFunctionSelectorKey(Function.Name,
-                                                *WhereParameters.second))
-                 .second) {
+      if (WhereSelector->Parameters) {
+        std::string DuplicateKey =
----------------
Xazax-hun wrote:

Some question about the naming + potentially having hash and equality on the selectors and putting them in a set directly instead of a serialized string. 

https://github.com/llvm/llvm-project/pull/216148