[GR-17148] Avoid assertion, because it runs too early on SVM

PullRequest: graalpython/590

Author: timfel

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/object/PythonObject.java

@@ -55,8 +55,6 @@
 import com.oracle.truffle.api.object.Shape;
 import com.oracle.truffle.api.object.dsl.Layout;
 
-import org.graalvm.nativeimage.ImageInfo;
-
 @ExportLibrary(PythonObjectLibrary.class)
 public class PythonObject extends PythonAbstractObject {
     public static final HiddenKey HAS_DICT = new HiddenKey("has_dict");
@@ -242,9 +240,10 @@ protected static interface PythonObjectLayout {
     private static final Shape emptyShape = PythonObjectLayoutImpl.INSTANCE.createPythonObjectShape(null).getShape();
 
     public static Shape freshShape(LazyPythonClass klass) {
-        if (!ImageInfo.inImageCode()) {
-            assert (PythonLanguage.getCurrent().singleContextAssumption.isValid() && klass != null) || klass instanceof PythonBuiltinClassType;
-        }
+        // GR-17243: I would like to enable this assertion, but it breaks
+        // building native images under some circumstances.
+        // assert (PythonLanguage.getCurrent().singleContextAssumption.isValid()
+        // && klass != null) || klass instanceof PythonBuiltinClassType;
         return PythonObjectLayoutImpl.INSTANCE.createPythonObjectShape(klass).getShape();
     }