Don't ignore overloaded friend functions that are also declared constexpr

Closes #2927

CHANGES.current

@@ -7,6 +7,10 @@ the issue number to the end of the URL: https://github.com/swig/swig/issues/
 Version 4.3.0 (in progress)
 ===========================
 
+2024-06-16: wsfulton
+            #2927 Don't ignore overloaded friend functions that are also declared
+            constexpr.
+
 2024-06-15: wsfulton
             [Python] Removed deprecated pytuplehlp.swg file and t_output_helper.
             Use SWIG_Python_AppendOutput instead of t_output_helper.

Examples/test-suite/common.mk

@@ -615,6 +615,7 @@ CPP11_TEST_CASES += \
 	cpp11_auto_variable \
 	cpp11_brackets_expression \
 	cpp11_constexpr \
+	cpp11_constexpr_friend \
 	cpp11_copyctor_delete \
 	cpp11_decltype \
 	cpp11_default_delete \

Examples/test-suite/cpp11_constexpr_friend.i

@@ -0,0 +1,35 @@
+%module cpp11_constexpr_friend
+
+%rename("isEqual") operator==(FriendA const& lhs, FriendA const& rhs) noexcept;
+%rename("isEqual") operator==(FriendB const& lhs, FriendB const& rhs) noexcept;
+
+%inline %{
+  class FriendA final
+  {
+    public:
+      FriendA(int const v) noexcept : _v(v)
+      {
+      }
+
+      constexpr friend bool operator==(FriendA const& lhs, FriendA const& rhs) noexcept
+      {
+        return lhs._v == rhs._v;
+      }
+    private:
+      int _v{};
+  };
+  class FriendB final
+  {
+    public:
+      FriendB(int const v) noexcept : _v(v)
+      {
+      }
+
+      constexpr friend bool operator==(FriendB const& lhs, FriendB const& rhs) noexcept
+      {
+        return lhs._v == rhs._v;
+      }
+    private:
+      int _v{};
+  };
+%}

Examples/test-suite/csharp/cpp11_constexpr_friend_runme.cs

@@ -0,0 +1,20 @@
+using System;
+using cpp11_constexpr_friendNamespace;
+
+public class cpp11_constexpr_friend_runme {
+  public static void Main() {
+    FriendA fa10 = new FriendA(10);
+    FriendA fa20 = new FriendA(20);
+    if (!cpp11_constexpr_friend.isEqual(fa10, fa10))
+      throw new Exception("Friend fa10 fa10 fail");
+    if (cpp11_constexpr_friend.isEqual(fa10, fa20))
+      throw new Exception("Friend fa10 fa20 fail");
+
+    FriendB fb10 = new FriendB(10);
+    FriendB fb20 = new FriendB(20);
+    if (!cpp11_constexpr_friend.isEqual(fb10, fb10))
+      throw new Exception("Friend fb10 fb10 fail");
+    if (cpp11_constexpr_friend.isEqual(fb10, fb20))
+      throw new Exception("Friend fb10 fb20 fail");
+  }
+}

Source/CParse/parser.y

@@ -420,7 +420,7 @@ static void add_symbols(Node *n) {
     String *symname = 0;
     String *old_prefix = 0;
     Symtab *old_scope = 0;
-    int isfriend = inclass && Checkattr(n, "storage", "friend");
+    int isfriend = inclass && Strstr(Getattr(n, "storage"), "friend") != NULL;
     int iscdecl = Cmp(nodeType(n),"cdecl") == 0;
     int only_csymbol = 0;