removed redundant instanceof TruffleString from LocalsStorage

tomasstupka · tomasstupka · commit 193f1b7e0881 · 2022-06-17T13:13:52.000+02:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/common/LocalsStorage.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/common/LocalsStorage.java
@@ -254,12 +254,10 @@ static HashingStorage cached(LocalsStorage self, HashingStorage other,
             HashingStorage result = other;
             for (int slot = 0; slot < desc.getNumberOfSlots(); slot++) {
                 Object identifier = desc.getSlotName(slot);
-                if (identifier instanceof TruffleString) {
-                    if (isUserFrameSlot(identifier)) {
-                        Object value = self.getValue(slot);
-                        if (value != null) {
-                            result = lib.setItem(result, desc.getSlotName(slot), value);
-                        }
+                if (isUserFrameSlot(identifier)) {
+                    Object value = self.getValue(slot);
+                    if (value != null) {
+                        result = lib.setItem(result, desc.getSlotName(slot), value);
                     }
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -254,12 +254,10 @@ static HashingStorage cached(LocalsStorage self, HashingStorage other,`
`254`	`254`	`HashingStorage result = other;`
`255`	`255`	`for (int slot = 0; slot < desc.getNumberOfSlots(); slot++) {`
`256`	`256`	`Object identifier = desc.getSlotName(slot);`
`257`		`- if (identifier instanceof TruffleString) {`
`258`		`- if (isUserFrameSlot(identifier)) {`
`259`		`- Object value = self.getValue(slot);`
`260`		`- if (value != null) {`
`261`		`- result = lib.setItem(result, desc.getSlotName(slot), value);`
`262`		`- }`
	`257`	`+ if (isUserFrameSlot(identifier)) {`
	`258`	`+ Object value = self.getValue(slot);`
	`259`	`+ if (value != null) {`
	`260`	`+ result = lib.setItem(result, desc.getSlotName(slot), value);`
`263`	`261`	`}`
`264`	`262`	`}`
`265`	`263`	`}`