[GR-19220] Use the owner module instead of origin with UnboundMethod#{inspect,==} (#3437)

PullRequest: truffleruby/4163

Author: eregon

CHANGELOG.md

@@ -48,6 +48,7 @@ Compatibility:
 * Support `connect_timeout` keyword argument to `TCPSocket.{new,open}` (#3421, @manefz, @patricklinpl, @nirvdrum, @rwstauner).
 * Add `File.lutime` and `Pathname#lutime` methods (#3039, @andrykonchin).
 * Add a deprecation warning for `Encoding#replicate` (#3039, @patricklinpl, @manefz, @nirvdrum).
+* Change `UnboundMethod#{==,inspect}` to use the owner module rather than the origin (#3039, @rwstauner, @manefz, @patricklinpl)
 
 Performance:
 

spec/tags/core/unboundmethod/equal_value_tags.txt

@@ -52,8 +52,7 @@ public abstract static class EqualNode extends CoreMethodArrayArgumentsNode {
 
         @Specialization
         boolean equal(RubyUnboundMethod self, RubyUnboundMethod other) {
-            return self.origin == other.origin &&
-                    MethodNodes.areInternalMethodEqual(self.method, other.method);
+            return MethodNodes.areInternalMethodEqual(self.method, other.method);
         }
 
         @Specialization(guards = "!isRubyUnboundMethod(other)")
@@ -134,16 +133,6 @@ RubySymbol name(RubyUnboundMethod unboundMethod) {
 
     }
 
-    @Primitive(name = "unbound_method_origin")
-    public abstract static class OriginNode extends PrimitiveArrayArgumentsNode {
-
-        @Specialization
-        RubyModule origin(RubyUnboundMethod unboundMethod) {
-            return unboundMethod.origin;
-        }
-
-    }
-
     @CoreMethod(names = "original_name")
     public abstract static class OriginalNameNode extends CoreMethodArrayArgumentsNode {
 

spec/tags/core/unboundmethod/inspect_tags.txt

@@ -10,7 +10,7 @@
 
 class UnboundMethod
   def inspect
-    Truffle::MethodOperations.inspect_method(self, Primitive.unbound_method_origin(self), owner)
+    Truffle::MethodOperations.inspect_method(self, owner, owner)
   end
   alias_method :to_s, :inspect