jdk/src/share/classes/sun/tools/tree/IfStatement.java
author ntoda
Thu, 31 Jul 2014 17:01:24 -0700
changeset 25799 1afc4675dc75
parent 5506 202f599c92aa
permissions -rw-r--r--
8044867: Fix raw and unchecked lint warnings in sun.tools.* Reviewed-by: darcy
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
     1
/*
5506
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 2
diff changeset
     2
 * Copyright (c) 1994, 2003, Oracle and/or its affiliates. All rights reserved.
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
90ce3da70b43 Initial load
duke
parents:
diff changeset
     4
 *
90ce3da70b43 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
90ce3da70b43 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
5506
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 2
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
5506
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 2
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    10
 *
90ce3da70b43 Initial load
duke
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
90ce3da70b43 Initial load
duke
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
90ce3da70b43 Initial load
duke
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
90ce3da70b43 Initial load
duke
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
90ce3da70b43 Initial load
duke
parents:
diff changeset
    15
 * accompanied this code).
90ce3da70b43 Initial load
duke
parents:
diff changeset
    16
 *
90ce3da70b43 Initial load
duke
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
90ce3da70b43 Initial load
duke
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
90ce3da70b43 Initial load
duke
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    20
 *
5506
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 2
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 2
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
202f599c92aa 6943119: Rebrand source copyright notices
ohair
parents: 2
diff changeset
    23
 * questions.
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    24
 */
90ce3da70b43 Initial load
duke
parents:
diff changeset
    25
90ce3da70b43 Initial load
duke
parents:
diff changeset
    26
package sun.tools.tree;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    27
90ce3da70b43 Initial load
duke
parents:
diff changeset
    28
import sun.tools.java.*;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    29
import sun.tools.asm.Assembler;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    30
import sun.tools.asm.Label;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    31
import java.io.PrintStream;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    32
import java.util.Hashtable;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    33
90ce3da70b43 Initial load
duke
parents:
diff changeset
    34
/**
90ce3da70b43 Initial load
duke
parents:
diff changeset
    35
 * WARNING: The contents of this source file are not part of any
90ce3da70b43 Initial load
duke
parents:
diff changeset
    36
 * supported API.  Code that depends on them does so at its own risk:
90ce3da70b43 Initial load
duke
parents:
diff changeset
    37
 * they are subject to change or removal without notice.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    38
 */
90ce3da70b43 Initial load
duke
parents:
diff changeset
    39
public
90ce3da70b43 Initial load
duke
parents:
diff changeset
    40
class IfStatement extends Statement {
90ce3da70b43 Initial load
duke
parents:
diff changeset
    41
    Expression cond;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    42
    Statement ifTrue;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    43
    Statement ifFalse;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    44
90ce3da70b43 Initial load
duke
parents:
diff changeset
    45
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
    46
     * Constructor
90ce3da70b43 Initial load
duke
parents:
diff changeset
    47
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
    48
    public IfStatement(long where, Expression cond, Statement ifTrue, Statement ifFalse) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
    49
        super(IF, where);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    50
        this.cond = cond;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    51
        this.ifTrue = ifTrue;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    52
        this.ifFalse = ifFalse;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    53
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
    54
90ce3da70b43 Initial load
duke
parents:
diff changeset
    55
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
    56
     * Check statement
90ce3da70b43 Initial load
duke
parents:
diff changeset
    57
     */
25799
1afc4675dc75 8044867: Fix raw and unchecked lint warnings in sun.tools.*
ntoda
parents: 5506
diff changeset
    58
    Vset check(Environment env, Context ctx, Vset vset, Hashtable<Object, Object> exp) {
2
90ce3da70b43 Initial load
duke
parents:
diff changeset
    59
        checkLabel(env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    60
        CheckContext newctx = new CheckContext(ctx, this);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    61
        // Vset vsExtra = vset.copy();  // See comment below.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    62
        ConditionVars cvars =
90ce3da70b43 Initial load
duke
parents:
diff changeset
    63
              cond.checkCondition(env, newctx, reach(env, vset), exp);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    64
        cond = convert(env, newctx, Type.tBoolean, cond);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    65
        // The following code, now deleted, was apparently an erroneous attempt
90ce3da70b43 Initial load
duke
parents:
diff changeset
    66
        // at providing better error diagnostics.  The comment read: 'If either
90ce3da70b43 Initial load
duke
parents:
diff changeset
    67
        // the true clause or the false clause is unreachable, do a reasonable
90ce3da70b43 Initial load
duke
parents:
diff changeset
    68
        // check on the child anyway.'
90ce3da70b43 Initial load
duke
parents:
diff changeset
    69
        //    Vset vsTrue  = cvars.vsTrue.isDeadEnd() ? vsExtra : cvars.vsTrue;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    70
        //    Vset vsFalse = cvars.vsFalse.isDeadEnd() ? vsExtra : cvars.vsFalse;
90ce3da70b43 Initial load
duke
parents:
diff changeset
    71
        // Unfortunately, this violates the rules laid out in the JLS, and leads to
90ce3da70b43 Initial load
duke
parents:
diff changeset
    72
        // blatantly incorrect results.  For example, 'i' will not be recognized
90ce3da70b43 Initial load
duke
parents:
diff changeset
    73
        // as definitely assigned following the statement 'if (true) i = 1;'.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    74
        // It is best to slavishly follow the JLS here.  A cleverer approach could
90ce3da70b43 Initial load
duke
parents:
diff changeset
    75
        // only correctly issue warnings, as JLS 16.2.6 is quite explicit, and it
90ce3da70b43 Initial load
duke
parents:
diff changeset
    76
        // is OK for a dead branch of an if-statement to omit an assignment that
90ce3da70b43 Initial load
duke
parents:
diff changeset
    77
        // would be required in the other branch.  A complication: This code also
90ce3da70b43 Initial load
duke
parents:
diff changeset
    78
        // had the effect of implementing the special-case rules for 'if-then' and
90ce3da70b43 Initial load
duke
parents:
diff changeset
    79
        // 'if-then-else' in JLS 14.19, "Unreachable Statements".  We now use
90ce3da70b43 Initial load
duke
parents:
diff changeset
    80
        // 'Vset.clearDeadEnd' to remove the dead-end status of unreachable branches
90ce3da70b43 Initial load
duke
parents:
diff changeset
    81
        // without affecting the definite-assignment status of the variables, thus
90ce3da70b43 Initial load
duke
parents:
diff changeset
    82
        // maintaining a correct implementation of JLS 16.2.6.  Fixes 4094353.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    83
        // Note that the code below will not consider the branches unreachable if
90ce3da70b43 Initial load
duke
parents:
diff changeset
    84
        // the entire statement is unreachable.  This is consistent with the error
90ce3da70b43 Initial load
duke
parents:
diff changeset
    85
        // recovery policy that reports the only the first unreachable statement
90ce3da70b43 Initial load
duke
parents:
diff changeset
    86
        // along an acyclic execution path.
90ce3da70b43 Initial load
duke
parents:
diff changeset
    87
        Vset vsTrue  = cvars.vsTrue.clearDeadEnd();
90ce3da70b43 Initial load
duke
parents:
diff changeset
    88
        Vset vsFalse = cvars.vsFalse.clearDeadEnd();
90ce3da70b43 Initial load
duke
parents:
diff changeset
    89
        vsTrue = ifTrue.check(env, newctx, vsTrue, exp);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    90
        if (ifFalse != null)
90ce3da70b43 Initial load
duke
parents:
diff changeset
    91
            vsFalse = ifFalse.check(env, newctx, vsFalse, exp);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    92
        vset = vsTrue.join(vsFalse.join(newctx.vsBreak));
90ce3da70b43 Initial load
duke
parents:
diff changeset
    93
        return ctx.removeAdditionalVars(vset);
90ce3da70b43 Initial load
duke
parents:
diff changeset
    94
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
    95
90ce3da70b43 Initial load
duke
parents:
diff changeset
    96
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
    97
     * Inline
90ce3da70b43 Initial load
duke
parents:
diff changeset
    98
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
    99
    public Statement inline(Environment env, Context ctx) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   100
        ctx = new Context(ctx, this);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   101
        cond = cond.inlineValue(env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   102
90ce3da70b43 Initial load
duke
parents:
diff changeset
   103
        // The compiler currently needs to perform inlining on both
90ce3da70b43 Initial load
duke
parents:
diff changeset
   104
        // branches of the if statement -- even if `cond' is a constant
90ce3da70b43 Initial load
duke
parents:
diff changeset
   105
        // true or false.  Why?  The compiler will later try to compile
90ce3da70b43 Initial load
duke
parents:
diff changeset
   106
        // all classes that it has seen; this includes classes that
90ce3da70b43 Initial load
duke
parents:
diff changeset
   107
        // appear in dead code.  If we don't inline the dead branch here
90ce3da70b43 Initial load
duke
parents:
diff changeset
   108
        // then the compiler will never perform inlining on any local
90ce3da70b43 Initial load
duke
parents:
diff changeset
   109
        // classes appearing on the dead code.  When the compiler tries
90ce3da70b43 Initial load
duke
parents:
diff changeset
   110
        // to compile an un-inlined local class with uplevel references,
90ce3da70b43 Initial load
duke
parents:
diff changeset
   111
        // it dies.  (bug 4059492)
90ce3da70b43 Initial load
duke
parents:
diff changeset
   112
        //
90ce3da70b43 Initial load
duke
parents:
diff changeset
   113
        // A better solution to this would be to walk the dead branch and
90ce3da70b43 Initial load
duke
parents:
diff changeset
   114
        // mark any local classes appearing therein as unneeded.  Then the
90ce3da70b43 Initial load
duke
parents:
diff changeset
   115
        // compilation phase could skip these classes.
90ce3da70b43 Initial load
duke
parents:
diff changeset
   116
        if (ifTrue != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   117
            ifTrue = ifTrue.inline(env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   118
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   119
        if (ifFalse != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   120
            ifFalse = ifFalse.inline(env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   121
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   122
        if (cond.equals(true)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   123
            return eliminate(env, ifTrue);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   124
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   125
        if (cond.equals(false)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   126
            return eliminate(env, ifFalse);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   127
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   128
        if ((ifTrue == null) && (ifFalse == null)) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   129
            return eliminate(env, new ExpressionStatement(where, cond).inline(env, ctx));
90ce3da70b43 Initial load
duke
parents:
diff changeset
   130
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   131
        if (ifTrue == null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   132
            cond = new NotExpression(cond.where, cond).inlineValue(env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   133
            return eliminate(env, new IfStatement(where, cond, ifFalse, null));
90ce3da70b43 Initial load
duke
parents:
diff changeset
   134
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   135
        return this;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   136
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   137
90ce3da70b43 Initial load
duke
parents:
diff changeset
   138
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
   139
     * Create a copy of the statement for method inlining
90ce3da70b43 Initial load
duke
parents:
diff changeset
   140
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
   141
    public Statement copyInline(Context ctx, boolean valNeeded) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   142
        IfStatement s = (IfStatement)clone();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   143
        s.cond = cond.copyInline(ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   144
        if (ifTrue != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   145
            s.ifTrue = ifTrue.copyInline(ctx, valNeeded);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   146
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   147
        if (ifFalse != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   148
            s.ifFalse = ifFalse.copyInline(ctx, valNeeded);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   149
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   150
        return s;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   151
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   152
90ce3da70b43 Initial load
duke
parents:
diff changeset
   153
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
   154
     * The cost of inlining this statement
90ce3da70b43 Initial load
duke
parents:
diff changeset
   155
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
   156
    public int costInline(int thresh, Environment env, Context ctx) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   157
        int cost = 1 + cond.costInline(thresh, env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   158
        if (ifTrue != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   159
            cost += ifTrue.costInline(thresh, env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   160
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   161
        if (ifFalse != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   162
            cost += ifFalse.costInline(thresh, env, ctx);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   163
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   164
        return cost;
90ce3da70b43 Initial load
duke
parents:
diff changeset
   165
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   166
90ce3da70b43 Initial load
duke
parents:
diff changeset
   167
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
   168
     * Code
90ce3da70b43 Initial load
duke
parents:
diff changeset
   169
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
   170
    public void code(Environment env, Context ctx, Assembler asm) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   171
        CodeContext newctx = new CodeContext(ctx, this);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   172
90ce3da70b43 Initial load
duke
parents:
diff changeset
   173
        Label l1 = new Label();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   174
        cond.codeBranch(env, newctx, asm, l1, false);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   175
        ifTrue.code(env, newctx, asm);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   176
        if (ifFalse != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   177
            Label l2 = new Label();
90ce3da70b43 Initial load
duke
parents:
diff changeset
   178
            asm.add(true, where, opc_goto, l2);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   179
            asm.add(l1);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   180
            ifFalse.code(env, newctx, asm);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   181
            asm.add(l2);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   182
        } else {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   183
            asm.add(l1);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   184
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   185
90ce3da70b43 Initial load
duke
parents:
diff changeset
   186
        asm.add(newctx.breakLabel);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   187
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   188
90ce3da70b43 Initial load
duke
parents:
diff changeset
   189
    /**
90ce3da70b43 Initial load
duke
parents:
diff changeset
   190
     * Print
90ce3da70b43 Initial load
duke
parents:
diff changeset
   191
     */
90ce3da70b43 Initial load
duke
parents:
diff changeset
   192
    public void print(PrintStream out, int indent) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   193
        super.print(out, indent);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   194
        out.print("if ");
90ce3da70b43 Initial load
duke
parents:
diff changeset
   195
        cond.print(out);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   196
        out.print(" ");
90ce3da70b43 Initial load
duke
parents:
diff changeset
   197
        ifTrue.print(out, indent);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   198
        if (ifFalse != null) {
90ce3da70b43 Initial load
duke
parents:
diff changeset
   199
            out.print(" else ");
90ce3da70b43 Initial load
duke
parents:
diff changeset
   200
            ifFalse.print(out, indent);
90ce3da70b43 Initial load
duke
parents:
diff changeset
   201
        }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   202
    }
90ce3da70b43 Initial load
duke
parents:
diff changeset
   203
}