Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(910)

Unified Diff: pkg/compiler/lib/src/ssa/interceptor_simplifier.dart

Issue 1859343004: dartfmt pkg/compiler (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/compiler/lib/src/ssa/interceptor_simplifier.dart
diff --git a/pkg/compiler/lib/src/ssa/interceptor_simplifier.dart b/pkg/compiler/lib/src/ssa/interceptor_simplifier.dart
index 328111124ba61ac1f42381ac54b32f8837297f2c..1777d961666c6a23f5f3676f6d6d92c2f586286e 100644
--- a/pkg/compiler/lib/src/ssa/interceptor_simplifier.dart
+++ b/pkg/compiler/lib/src/ssa/interceptor_simplifier.dart
@@ -96,9 +96,8 @@ class SsaSimplifyInterceptors extends HBaseVisitor
return false;
}
- bool canUseSelfForInterceptor(HInstruction receiver,
- Set<ClassElement> interceptedClasses) {
-
+ bool canUseSelfForInterceptor(
+ HInstruction receiver, Set<ClassElement> interceptedClasses) {
if (receiver.canBePrimitive(compiler)) {
// Primitives always need interceptors.
return false;
@@ -116,8 +115,7 @@ class SsaSimplifyInterceptors extends HBaseVisitor
}
HInstruction tryComputeConstantInterceptor(
- HInstruction input,
- Set<ClassElement> interceptedClasses) {
+ HInstruction input, Set<ClassElement> interceptedClasses) {
if (input == graph.explicitReceiverParameter) {
// If `explicitReceiverParameter` is set it means the current method is an
// interceptor method, and `this` is the interceptor. The caller just did
@@ -143,9 +141,7 @@ class SsaSimplifyInterceptors extends HBaseVisitor
}
ClassElement tryComputeConstantInterceptorFromType(
- TypeMask type,
- Set<ClassElement> interceptedClasses) {
-
+ TypeMask type, Set<ClassElement> interceptedClasses) {
if (type.isNullable) {
if (type.isNull) {
return helpers.jsNullClass;
@@ -230,13 +226,13 @@ class SsaSimplifyInterceptors extends HBaseVisitor
node == dominator.receiver &&
useCount(dominator, node) == 1) {
interceptedClasses =
- backend.getInterceptedClassesOn(dominator.selector.name);
+ 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(helpers.jsNumberClass) &&
!(interceptedClasses.contains(helpers.jsDoubleClass) ||
- interceptedClasses.contains(helpers.jsIntClass))) {
+ interceptedClasses.contains(helpers.jsIntClass))) {
for (HInstruction user in node.usedBy) {
if (user is! HInvoke) continue;
Set<ClassElement> intercepted =
@@ -256,14 +252,14 @@ class SsaSimplifyInterceptors extends HBaseVisitor
user.isCallOnInterceptor(compiler) &&
node == user.receiver &&
useCount(user, node) == 1) {
- interceptedClasses.addAll(
- backend.getInterceptedClassesOn(user.selector.name));
+ interceptedClasses
+ .addAll(backend.getInterceptedClassesOn(user.selector.name));
} else if (user is HInvokeSuper &&
- user.isCallOnInterceptor(compiler) &&
- node == user.receiver &&
- useCount(user, node) == 1) {
- interceptedClasses.addAll(
- backend.getInterceptedClassesOn(user.selector.name));
+ user.isCallOnInterceptor(compiler) &&
+ node == user.receiver &&
+ useCount(user, node) == 1) {
+ interceptedClasses
+ .addAll(backend.getInterceptedClassesOn(user.selector.name));
} else {
// Use a most general interceptor for other instructions, example,
// is-checks and escaping interceptors.
@@ -311,10 +307,9 @@ class SsaSimplifyInterceptors extends HBaseVisitor
ClassElement interceptorClass = tryComputeConstantInterceptorFromType(
receiver.instructionType.nonNullable(), interceptedClasses);
if (interceptorClass != null) {
- HInstruction constantInstruction =
- graph.addConstant(
- new InterceptorConstantValue(interceptorClass.thisType),
- compiler);
+ HInstruction constantInstruction = graph.addConstant(
+ new InterceptorConstantValue(interceptorClass.thisType),
+ compiler);
node.conditionalConstantInterceptor = constantInstruction;
constantInstruction.usedBy.add(node);
return false;
@@ -358,8 +353,11 @@ class SsaSimplifyInterceptors extends HBaseVisitor
List<HInstruction> inputs = new List<HInstruction>.from(user.inputs);
inputs[0] = nullConstant;
HOneShotInterceptor oneShotInterceptor = new HOneShotInterceptor(
- user.selector, user.mask,
- inputs, user.instructionType, interceptedClasses);
+ user.selector,
+ user.mask,
+ inputs,
+ user.instructionType,
+ interceptedClasses);
oneShotInterceptor.sourceInformation = user.sourceInformation;
oneShotInterceptor.sourceElement = user.sourceElement;
return replaceUserWith(oneShotInterceptor);
@@ -386,8 +384,8 @@ class SsaSimplifyInterceptors extends HBaseVisitor
}
bool visitOneShotInterceptor(HOneShotInterceptor node) {
- HInstruction constant = tryComputeConstantInterceptor(
- node.inputs[1], node.interceptedClasses);
+ HInstruction constant =
+ tryComputeConstantInterceptor(node.inputs[1], node.interceptedClasses);
if (constant == null) return false;
@@ -395,12 +393,8 @@ class SsaSimplifyInterceptors extends HBaseVisitor
TypeMask mask = node.mask;
HInstruction instruction;
if (selector.isGetter) {
- instruction = new HInvokeDynamicGetter(
- selector,
- mask,
- node.element,
- <HInstruction>[constant, node.inputs[1]],
- node.instructionType);
+ instruction = new HInvokeDynamicGetter(selector, mask, node.element,
+ <HInstruction>[constant, node.inputs[1]], node.instructionType);
} else if (selector.isSetter) {
instruction = new HInvokeDynamicSetter(
selector,
« no previous file with comments | « pkg/compiler/lib/src/ssa/codegen_helpers.dart ('k') | pkg/compiler/lib/src/ssa/invoke_dynamic_specializers.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698