ugly workaround for allocation reporter assertion

timfel · timfel · commit b1aac6d219c3 · 2018-10-09T14:14:08.000+02:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/object/PythonObjectFactory.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/runtime/object/PythonObjectFactory.java
@@ -167,7 +167,9 @@ private PythonContext getContext() {
     }
 
     private boolean reportAllocations() {
-        return getContext() != null && allocationReporter != null;
+        // tfel: the allocation reporter has an assertion that it is a PE constant. How the reporter
+        // works is subject to change, but until such time we don't report allocations if it isn't
+        return getContext() != null && allocationReporter != null && CompilerDirectives.isPartialEvaluationConstant(allocationReporter);
     }
 
     /*

Original file line number	Diff line number	Diff line change
`@@ -167,7 +167,9 @@ private PythonContext getContext() {`
`167`	`167`	`}`
`168`	`168`
`169`	`169`	`private boolean reportAllocations() {`
`170`		`- return getContext() != null && allocationReporter != null;`
	`170`	`+ // tfel: the allocation reporter has an assertion that it is a PE constant. How the reporter`
	`171`	`+ // works is subject to change, but until such time we don't report allocations if it isn't`
	`172`	`+ return getContext() != null && allocationReporter != null && CompilerDirectives.isPartialEvaluationConstant(allocationReporter);`
`171`	`173`	`}`
`172`	`174`
`173`	`175`	`/*`