changeset 21947:9f70fc90169d

Truffle: remove expensive assertion
author Andreas Woess <andreas.woess@oracle.com>
date Wed, 17 Jun 2015 04:09:30 +0200
parents a50fa3266a0a
children f0fe6f0cbccc
files graal/com.oracle.truffle.object/src/com/oracle/truffle/object/PropertyMap.java graal/com.oracle.truffle.object/src/com/oracle/truffle/object/ShapeImpl.java
diffstat 2 files changed, 0 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/graal/com.oracle.truffle.object/src/com/oracle/truffle/object/PropertyMap.java	Tue Jun 16 18:11:23 2015 -0700
+++ b/graal/com.oracle.truffle.object/src/com/oracle/truffle/object/PropertyMap.java	Wed Jun 17 04:09:30 2015 +0200
@@ -291,7 +291,6 @@
     }
 
     public PropertyMap putCopy(Property value) {
-        assert !this.containsValue(value);
         return new PropertyMap(this, value);
     }
 
--- a/graal/com.oracle.truffle.object/src/com/oracle/truffle/object/ShapeImpl.java	Tue Jun 16 18:11:23 2015 -0700
+++ b/graal/com.oracle.truffle.object/src/com/oracle/truffle/object/ShapeImpl.java	Wed Jun 17 04:09:30 2015 +0200
@@ -333,8 +333,6 @@
     private ShapeImpl addPropertyInternal(Property prop) {
         CompilerAsserts.neverPartOfCompilation();
         assert prop.isShadow() || !(this.hasProperty(prop.getKey())) : "duplicate property " + prop.getKey();
-        // assertion caused performance issues on e.g. testv8 regress-619.js
-        // assert !getPropertyListInternal(false).contains(prop);
 
         AddPropertyTransition addTransition = new AddPropertyTransition(prop);
         ShapeImpl cachedShape = queryTransition(addTransition);