[GR-20835] Operations lt, le, gt and ge for two complex numbers should return NotImplemented.

Author: ppisl

graalpython/com.oracle.graal.python.test/src/tests/unittest_tags/test_complex.txt

@@ -10,4 +10,5 @@
 *ComplexTest.test_overflow
 *ComplexTest.test_plus_minus_0j
 *ComplexTest.test_repr_str
+*ComplexTest.test_richcompare
 *ComplexTest.test_truediv

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/complex/ComplexBuiltins.java

@@ -498,8 +498,8 @@ PNotImplemented doGeneric(Object left, Object right) {
     @Builtin(name = __GE__, minNumOfPositionalArgs = 2)
     abstract static class GeNode extends PythonBinaryBuiltinNode {
         @Specialization
-        boolean doComplex(PComplex left, PComplex right) {
-            return left.greaterEqual(right);
+        PNotImplemented doComplex(PComplex left, PComplex right) {
+            return PNotImplemented.NOT_IMPLEMENTED;
         }
 
         @SuppressWarnings("unused")
@@ -513,8 +513,8 @@ PNotImplemented doGeneric(Object left, Object right) {
     @Builtin(name = __GT__, minNumOfPositionalArgs = 2)
     abstract static class GtNode extends PythonBinaryBuiltinNode {
         @Specialization
-        boolean doComplex(PComplex left, PComplex right) {
-            return left.greaterThan(right);
+        PNotImplemented doComplex(PComplex left, PComplex right) {
+            return PNotImplemented.NOT_IMPLEMENTED;
         }
 
         @SuppressWarnings("unused")
@@ -528,8 +528,8 @@ PNotImplemented doGeneric(Object left, Object right) {
     @Builtin(name = __LT__, minNumOfPositionalArgs = 2)
     abstract static class LtNode extends PythonBinaryBuiltinNode {
         @Specialization
-        boolean doComplex(PComplex left, PComplex right) {
-            return left.lessThan(right);
+        PNotImplemented doComplex(PComplex left, PComplex right) {
+            return PNotImplemented.NOT_IMPLEMENTED;
         }
 
         @SuppressWarnings("unused")
@@ -543,8 +543,8 @@ PNotImplemented doGeneric(Object left, Object right) {
     @Builtin(name = __LE__, minNumOfPositionalArgs = 2)
     abstract static class LeNode extends PythonBinaryBuiltinNode {
         @Specialization
-        boolean doComplex(PComplex left, PComplex right) {
-            return left.lessEqual(right);
+        PNotImplemented doComplex(PComplex left, PComplex right) {
+            return PNotImplemented.NOT_IMPLEMENTED;
         }
 
         @SuppressWarnings("unused")
@@ -567,6 +567,16 @@ boolean doComplex(PComplex left, PComplex right) {
         boolean doComplex(PComplex left, long right) {
             return left.getImag() != 0 || left.getReal() != right;
         }
+        
+        @Specialization
+        boolean doComplex(PComplex left, PInt right) {
+            return left.getImag() != 0 || left.getReal() != right.doubleValue();
+        }
+        
+        @Specialization
+        boolean doComplex(PComplex left, double right) {
+            return left.getImag() != 0 || left.getReal() != right;
+        }
 
         @SuppressWarnings("unused")
         @Fallback