Fix: Do not share native pointers.

Author: fangerer

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/CExtNodes.java

@@ -364,12 +364,7 @@ static PrimitiveNativeWrapper doIntegerSmall(@SuppressWarnings("unused") CExtCon
         static PrimitiveNativeWrapper doInteger(@SuppressWarnings("unused") CExtContext cextContext, int i,
                         @Shared("contextRef") @CachedContext(PythonLanguage.class) ContextReference<PythonContext> contextRef) {
             if (CApiGuards.isSmallInteger(i)) {
-                if (CompilerDirectives.inInterpreter()) {
-                    return doIntegerSmall(cextContext, i, contextRef);
-                } else {
-                    // for explanation: see method doc
-                    return doIntegerSmall(cextContext, i, contextRef).copy();
-                }
+                return doIntegerSmall(cextContext, i, contextRef);
             }
             return PrimitiveNativeWrapper.createInt(i);
         }
@@ -388,12 +383,7 @@ static PrimitiveNativeWrapper doLongSmall(@SuppressWarnings("unused") CExtContex
         static PrimitiveNativeWrapper doLong(@SuppressWarnings("unused") CExtContext cextContext, long l,
                         @Shared("contextRef") @CachedContext(PythonLanguage.class) ContextReference<PythonContext> contextRef) {
             if (CApiGuards.isSmallLong(l)) {
-                // for explanation of this construct: see 'ToSulongNode.doInteger'
-                if (CompilerDirectives.inInterpreter()) {
-                    return doLongSmall(cextContext, l, contextRef);
-                } else {
-                    return doLongSmall(cextContext, l, contextRef).copy();
-                }
+                return doLongSmall(cextContext, l, contextRef);
             }
             return PrimitiveNativeWrapper.createLong(l);
         }
@@ -622,12 +612,7 @@ static PrimitiveNativeWrapper doLongSmall(@SuppressWarnings("unused") CExtContex
         static PrimitiveNativeWrapper doInteger(CExtContext cextContext, int i,
                         @Shared("contextRef") @CachedContext(PythonLanguage.class) ContextReference<PythonContext> contextRef) {
             if (CApiGuards.isSmallInteger(i)) {
-                // for explanation of this construct: see 'ToSulongNode.doInteger'
-                if (CompilerDirectives.inInterpreter()) {
-                    return doIntegerSmall(cextContext, i, contextRef);
-                } else {
-                    return doIntegerSmall(cextContext, i, contextRef).copy();
-                }
+                return doIntegerSmall(cextContext, i, contextRef);
             }
             return PrimitiveNativeWrapper.createInt(i);
         }
@@ -636,12 +621,7 @@ static PrimitiveNativeWrapper doInteger(CExtContext cextContext, int i,
         static PrimitiveNativeWrapper doLong(CExtContext cextContext, long l,
                         @Shared("contextRef") @CachedContext(PythonLanguage.class) ContextReference<PythonContext> contextRef) {
             if (CApiGuards.isSmallLong(l)) {
-                // for explanation of this construct: see 'ToSulongNode.doInteger'
-                if (CompilerDirectives.inInterpreter()) {
-                    return doLongSmall(cextContext, l, contextRef);
-                } else {
-                    return doLongSmall(cextContext, l, contextRef).copy();
-                }
+                return doLongSmall(cextContext, l, contextRef);
             }
             return PrimitiveNativeWrapper.createLong(l);
         }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/DynamicObjectNativeWrapper.java

@@ -1475,12 +1475,6 @@ private PrimitiveNativeWrapper(double dvalue) {
             this.dvalue = dvalue;
         }
 
-        private PrimitiveNativeWrapper(byte state, long value, double dvalue) {
-            this.state = state;
-            this.value = value;
-            this.dvalue = dvalue;
-        }
-
         public byte getState() {
             return state;
         }
@@ -1563,10 +1557,6 @@ public int hashCode() {
             return (Long.hashCode(value) ^ Long.hashCode(Double.doubleToRawLongBits(dvalue)) ^ state);
         }
 
-        PrimitiveNativeWrapper copy() {
-            return new PrimitiveNativeWrapper(state, value, dvalue);
-        }
-
         @Override
         public String toString() {
             return "PrimitiveNativeWrapper(" + getTypeName() + "(" + value + ")" + ')';