Use entities in BackendClasses

pkg/compiler/lib/src/ssa/interceptor_simplifier.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 import '../common/backend_api.dart' show BackendClasses;
 import '../compiler.dart' show Compiler;
 import '../constants/constant_system.dart';
 import '../constants/values.dart';
 import '../elements/entities.dart';
 import '../js_backend/backend.dart';
@@ skipping 83 matching lines @@
     return false;
   }
 
   bool canUseSelfForInterceptor(
       HInstruction receiver, Set<ClassEntity> interceptedClasses) {
     if (receiver.canBePrimitive(closedWorld)) {
       // Primitives always need interceptors.
       return false;
     }
     if (receiver.canBeNull() &&
-        interceptedClasses.contains(backendClasses.nullImplementation)) {
+        interceptedClasses.contains(backendClasses.nullClass)) {
       // Need the JSNull interceptor.
       return false;
     }
 
     // All intercepted classes extend `Interceptor`, so if the receiver can't be
     // a class extending `Interceptor` then it can be called directly.
     return new TypeMask.nonNullSubclass(
             backend.helpers.jsInterceptorClass, closedWorld)
         .isDisjoint(receiver.instructionType, closedWorld);
   }
@@ skipping 20 matching lines @@
     }
 
     ConstantValue constant = new InterceptorConstantValue(constantInterceptor);
     return graph.addConstant(constant, closedWorld);
   }
 
   ClassEntity tryComputeConstantInterceptorFromType(
       TypeMask type, Set<ClassEntity> interceptedClasses) {
     if (type.isNullable) {
       if (type.isNull) {
-        return backendClasses.nullImplementation;
+        return backendClasses.nullClass;
       }
     } else if (type.containsOnlyInt(closedWorld)) {
-      return backendClasses.intImplementation;
+      return backendClasses.intClass;
     } else if (type.containsOnlyDouble(closedWorld)) {
-      return backendClasses.doubleImplementation;
+      return backendClasses.doubleClass;
     } else if (type.containsOnlyBool(closedWorld)) {
-      return backendClasses.boolImplementation;
+      return backendClasses.boolClass;
     } else if (type.containsOnlyString(closedWorld)) {
-      return backendClasses.stringImplementation;
-    } else if (type.satisfies(backendClasses.listImplementation, closedWorld)) {
-      return backendClasses.listImplementation;
+      return backendClasses.stringClass;
+    } else if (type.satisfies(backendClasses.listClass, closedWorld)) {
+      return backendClasses.listClass;
     } else if (type.containsOnlyNum(closedWorld) &&
-        !interceptedClasses.contains(backendClasses.intImplementation) &&
-        !interceptedClasses.contains(backendClasses.doubleImplementation)) {
+        !interceptedClasses.contains(backendClasses.intClass) &&
+        !interceptedClasses.contains(backendClasses.doubleClass)) {
       // If the method being intercepted is not defined in [int] or [double] we
       // can safely use the number interceptor.  This is because none of the
       // [int] or [double] methods are called from a method defined on [num].
-      return backendClasses.numImplementation;
+      return backendClasses.numClass;
     } else {
       // Try to find constant interceptor for a native class.  If the receiver
       // is constrained to a leaf native class, we can use the class's
       // interceptor directly.
 
       // TODO(sra): Key DOM classes like Node, Element and Event are not leaf
       // classes.  When the receiver type is not a leaf class, we might still be
       // able to use the receiver class as a constant interceptor.  It is
       // usually the case that methods defined on a non-leaf class don't test
       // for a subclass or call methods defined on a subclass.  Provided the
@@ skipping 48 matching lines @@
     // selector of that instruction.
     if (dominator is HInvokeDynamic &&
         dominator.isCallOnInterceptor(closedWorld) &&
         node == dominator.receiver &&
         useCount(dominator, node) == 1) {
       interceptedClasses =
           backend.getInterceptedClassesOn(dominator.selector.name);
 
       // If we found that we need number, we must still go through all
       // uses to check if they require int, or double.
-      if (interceptedClasses.contains(backendClasses.numImplementation) &&
-          !(interceptedClasses.contains(backendClasses.doubleImplementation) ||
-              interceptedClasses.contains(backendClasses.intImplementation))) {
+      if (interceptedClasses.contains(backendClasses.numClass) &&
+          !(interceptedClasses.contains(backendClasses.doubleClass) ||
+              interceptedClasses.contains(backendClasses.intClass))) {
         Set<ClassEntity> required;
         for (HInstruction user in node.usedBy) {
           if (user is! HInvoke) continue;
           Set<ClassEntity> intercepted =
               backend.getInterceptedClassesOn(user.selector.name);
-          if (intercepted.contains(backendClasses.intImplementation)) {
+          if (intercepted.contains(backendClasses.intClass)) {
             // TODO(johnniwinther): Use type argument when all uses of
             // intercepted classes expect entities instead of elements.
             required ??= new Set/*<ClassEntity>*/();
-            required.add(backendClasses.intImplementation);
+            required.add(backendClasses.intClass);
           }
-          if (intercepted.contains(backendClasses.doubleImplementation)) {
+          if (intercepted.contains(backendClasses.doubleClass)) {
             // TODO(johnniwinther): Use type argument when all uses of
             // intercepted classes expect entities instead of elements.
             required ??= new Set/*<ClassEntity>*/();
-            required.add(backendClasses.doubleImplementation);
+            required.add(backendClasses.doubleClass);
           }
         }
         // Don't modify the result of [backend.getInterceptedClassesOn].
         if (required != null) {
           interceptedClasses = interceptedClasses.union(required);
         }
       }
     } else {
       // TODO(johnniwinther): Use type argument when all uses of intercepted
       // classes expect entities instead of elements.
@@ skipping 47 matching lines @@
     // If it is a conditional constant interceptor and was not strengthened to a
     // constant interceptor then there is nothing more we can do.
     if (node.isConditionalConstantInterceptor) return false;
 
     // Do we have an 'almost constant' interceptor?  The receiver could be
     // `null` but not any other JavaScript falsy value, `null` values cause
     // `NoSuchMethodError`s, and if the receiver was not null we would have a
     // constant interceptor `C`.  Then we can use `(receiver && C)` for the
     // interceptor.
     if (receiver.canBeNull()) {
-      if (!interceptedClasses.contains(backendClasses.nullImplementation)) {
+      if (!interceptedClasses.contains(backendClasses.nullClass)) {
         // Can use `(receiver && C)` only if receiver is either null or truthy.
         if (!(receiver.canBePrimitiveNumber(closedWorld) ||
             receiver.canBePrimitiveBoolean(closedWorld) ||
             receiver.canBePrimitiveString(closedWorld))) {
           ClassEntity interceptorClass = tryComputeConstantInterceptorFromType(
               receiver.instructionType.nonNullable(), interceptedClasses);
           if (interceptorClass != null) {
             HInstruction constantInstruction = graph.addConstant(
                 new InterceptorConstantValue(interceptorClass), closedWorld);
             node.conditionalConstantInterceptor = constantInstruction;
@@ skipping 94 matching lines @@
       instruction = new HInvokeDynamicMethod(
           selector, mask, inputs, node.instructionType, true);
     }
 
     HBasicBlock block = node.block;
     block.addAfter(node, instruction);
     block.rewrite(node, instruction);
     return true;
   }
 }