hotspot/src/share/vm/c1/c1_Canonicalizer.cpp
author thartmann
Tue, 07 Jun 2016 18:20:44 +0200
changeset 39263 d139a133ba27
parent 38060 954c9575f653
child 43673 bf2f6d3f8f5e
permissions -rw-r--r--
8158228: C1 incorrectly folds mismatched loads from stable arrays Summary: Disable constant folding for mismatched loads from stable arrays. Reviewed-by: vlivanov
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
39263
d139a133ba27 8158228: C1 incorrectly folds mismatched loads from stable arrays
thartmann
parents: 38060
diff changeset
     2
 * Copyright (c) 1999, 2016, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5046
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5046
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 5046
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6745
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6745
diff changeset
    26
#include "c1/c1_Canonicalizer.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6745
diff changeset
    27
#include "c1/c1_InstructionPrinter.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6745
diff changeset
    28
#include "c1/c1_ValueStack.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6745
diff changeset
    29
#include "ci/ciArray.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6745
diff changeset
    30
#include "runtime/sharedRuntime.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    31
489c9b5090e2 Initial load
duke
parents:
diff changeset
    32
5707
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    33
class PrintValueVisitor: public ValueVisitor {
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    34
  void visit(Value* vp) {
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    35
    (*vp)->print_line();
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    36
  }
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    37
};
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
void Canonicalizer::set_canonical(Value x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
  assert(x != NULL, "value must exist");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
  // Note: we can not currently substitute root nodes which show up in
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
  // the instruction stream (because the instruction list is embedded
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
  // in the instructions).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  if (canonical() != x) {
12947
ba8ba6c2dd83 7172843: C1: fix "assert(has_printable_bci()) failed: _printable_bci should have been set"
twisti
parents: 11886
diff changeset
    45
#ifndef PRODUCT
ba8ba6c2dd83 7172843: C1: fix "assert(has_printable_bci()) failed: _printable_bci should have been set"
twisti
parents: 11886
diff changeset
    46
    if (!x->has_printable_bci()) {
ba8ba6c2dd83 7172843: C1: fix "assert(has_printable_bci()) failed: _printable_bci should have been set"
twisti
parents: 11886
diff changeset
    47
      x->set_printable_bci(bci());
ba8ba6c2dd83 7172843: C1: fix "assert(has_printable_bci()) failed: _printable_bci should have been set"
twisti
parents: 11886
diff changeset
    48
    }
ba8ba6c2dd83 7172843: C1: fix "assert(has_printable_bci()) failed: _printable_bci should have been set"
twisti
parents: 11886
diff changeset
    49
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
    if (PrintCanonicalization) {
5707
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    51
      PrintValueVisitor do_print_value;
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    52
      canonical()->input_values_do(&do_print_value);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
      canonical()->print_line();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
      tty->print_cr("canonicalized to:");
5707
6c66849ed24e 6958292: C1: Enable parallel compilation
iveresov
parents: 5547
diff changeset
    55
      x->input_values_do(&do_print_value);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
      x->print_line();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
      tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
    assert(_canonical->type()->tag() == x->type()->tag(), "types must match");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
    _canonical = x;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
void Canonicalizer::move_const_to_right(Op2* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
  if (x->x()->type()->is_constant() && x->is_commutative()) x->swap_operands();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
void Canonicalizer::do_Op2(Op2* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  if (x->x() == x->y()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
    switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    73
    case Bytecodes::_isub: set_constant(0); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
    case Bytecodes::_lsub: set_constant(jlong_cast(0)); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
    case Bytecodes::_iand: // fall through
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
    case Bytecodes::_land: // fall through
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
    case Bytecodes::_ior:  // fall through
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
    case Bytecodes::_lor : set_canonical(x->x()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    79
    case Bytecodes::_ixor: set_constant(0); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    80
    case Bytecodes::_lxor: set_constant(jlong_cast(0)); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  if (x->x()->type()->is_constant() && x->y()->type()->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
    // do constant folding for selected operations
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
    switch (x->type()->tag()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
      case intTag:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
        { jint a = x->x()->type()->as_IntConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
          jint b = x->y()->type()->as_IntConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
          switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
            case Bytecodes::_iadd: set_constant(a + b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
            case Bytecodes::_isub: set_constant(a - b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
            case Bytecodes::_imul: set_constant(a * b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
            case Bytecodes::_idiv:
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
              if (b != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
                if (a == min_jint && b == -1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
                  set_constant(min_jint);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
                } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
                  set_constant(a / b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
                }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
                return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
              break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
            case Bytecodes::_irem:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
              if (b != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
                if (a == min_jint && b == -1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
                  set_constant(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
                } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
                  set_constant(a % b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
                }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
                return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
              break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
            case Bytecodes::_iand: set_constant(a & b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
            case Bytecodes::_ior : set_constant(a | b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
            case Bytecodes::_ixor: set_constant(a ^ b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
      case longTag:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   121
        { jlong a = x->x()->type()->as_LongConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
          jlong b = x->y()->type()->as_LongConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
          switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
            case Bytecodes::_ladd: set_constant(a + b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
            case Bytecodes::_lsub: set_constant(a - b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
            case Bytecodes::_lmul: set_constant(a * b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
            case Bytecodes::_ldiv:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
              if (b != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
                set_constant(SharedRuntime::ldiv(b, a));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
                return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
              break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
            case Bytecodes::_lrem:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
              if (b != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
                set_constant(SharedRuntime::lrem(b, a));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
                return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
              }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
              break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
            case Bytecodes::_land: set_constant(a & b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
            case Bytecodes::_lor : set_constant(a | b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
            case Bytecodes::_lxor: set_constant(a ^ b); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
      // other cases not implemented (must be extremely careful with floats & doubles!)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
  // make sure constant is on the right side, if any
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
  move_const_to_right(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
  if (x->y()->type()->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
    // do constant folding for selected operations
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
    switch (x->type()->tag()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
      case intTag:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
        if (x->y()->type()->as_IntConstant()->value() == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
          switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
            case Bytecodes::_iadd: set_canonical(x->x()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
            case Bytecodes::_isub: set_canonical(x->x()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
            case Bytecodes::_imul: set_constant(0); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
              // Note: for div and rem, make sure that C semantics
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
              //       corresponds to Java semantics!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
            case Bytecodes::_iand: set_constant(0); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
            case Bytecodes::_ior : set_canonical(x->x()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
      case longTag:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
        if (x->y()->type()->as_LongConstant()->value() == (jlong)0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
          switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
            case Bytecodes::_ladd: set_canonical(x->x()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
            case Bytecodes::_lsub: set_canonical(x->x()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
            case Bytecodes::_lmul: set_constant((jlong)0); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
              // Note: for div and rem, make sure that C semantics
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
              //       corresponds to Java semantics!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
            case Bytecodes::_land: set_constant((jlong)0); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
            case Bytecodes::_lor : set_canonical(x->x()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   183
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
void Canonicalizer::do_Phi            (Phi*             x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   186
void Canonicalizer::do_Constant       (Constant*        x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
void Canonicalizer::do_Local          (Local*           x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
void Canonicalizer::do_LoadField      (LoadField*       x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
// checks if v is in the block that is currently processed by
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
// GraphBuilder. This is the only block that has not BlockEnd yet.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
static bool in_current_block(Value v) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
  int max_distance = 4;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
  while (max_distance > 0 && v != NULL && v->as_BlockEnd() == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
    v = v->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
    max_distance--;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
  return v == NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
void Canonicalizer::do_StoreField     (StoreField*      x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   202
  // If a value is going to be stored into a field or array some of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
  // the conversions emitted by javac are unneeded because the fields
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
  // are packed to their natural size.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
  Convert* conv = x->value()->as_Convert();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
  if (conv) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
    Value value = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
    BasicType type = x->field()->type()->basic_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
    switch (conv->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
    case Bytecodes::_i2b: if (type == T_BYTE)  value = conv->value(); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
    case Bytecodes::_i2s: if (type == T_SHORT || type == T_BYTE) value = conv->value(); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
    case Bytecodes::_i2c: if (type == T_CHAR  || type == T_BYTE)  value = conv->value(); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
    // limit this optimization to current block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
    if (value != NULL && in_current_block(conv)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
      set_canonical(new StoreField(x->obj(), x->offset(), x->field(), value, x->is_static(),
8671
13ffa40a2f0a 6965570: assert(!needs_patching && x->is_loaded(),"how do we know it's volatile if it's not loaded")
never
parents: 8065
diff changeset
   217
                                   x->state_before(), x->needs_patching()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
void Canonicalizer::do_ArrayLength    (ArrayLength*     x) {
36310
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   225
  NewArray*  na;
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   226
  Constant*  ct;
36325
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   227
  LoadField* lf;
36310
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   228
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   229
  if ((na = x->array()->as_NewArray()) != NULL) {
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   230
    // New arrays might have the known length.
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   231
    // Do not use the Constant itself, but create a new Constant
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   232
    // with same value Otherwise a Constant is live over multiple
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   233
    // blocks without being registered in a state array.
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   234
    Constant* length;
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   235
    if (na->length() != NULL &&
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   236
        (length = na->length()->as_Constant()) != NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
      assert(length->type()->as_IntConstant() != NULL, "array length must be integer");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      set_constant(length->type()->as_IntConstant()->value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
    }
36310
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   240
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   241
  } else if ((ct = x->array()->as_Constant()) != NULL) {
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   242
    // Constant arrays have constant lengths.
36317
5c78a6c34390 8150514: C1 crashes in Canonicalizer::do_ArrayLength() after fix for JDK-8150102
shade
parents: 36310
diff changeset
   243
    ArrayConstant* cnst = ct->type()->as_ArrayConstant();
5c78a6c34390 8150514: C1 crashes in Canonicalizer::do_ArrayLength() after fix for JDK-8150102
shade
parents: 36310
diff changeset
   244
    if (cnst != NULL) {
5c78a6c34390 8150514: C1 crashes in Canonicalizer::do_ArrayLength() after fix for JDK-8150102
shade
parents: 36310
diff changeset
   245
      set_constant(cnst->value()->length());
5c78a6c34390 8150514: C1 crashes in Canonicalizer::do_ArrayLength() after fix for JDK-8150102
shade
parents: 36310
diff changeset
   246
    }
36310
bcc01156b370 8150102: C1 should fold arraylength for constant/trusted arrays
shade
parents: 36058
diff changeset
   247
36325
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   248
  } else if ((lf = x->array()->as_LoadField()) != NULL) {
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   249
    ciField* field = lf->field();
38030
93f24e7b3c43 8152590: C2: @Stable support doesn't always work w/ incremental inlining
vlivanov
parents: 36604
diff changeset
   250
    if (field->is_static_constant()) {
36325
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   251
      assert(PatchALot || ScavengeRootsInCode < 2, "Constant field loads are folded during parsing");
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   252
      ciObject* c = field->constant_value().as_object();
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   253
      if (!c->is_null_object()) {
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   254
        set_constant(c->as_array()->length());
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   255
      }
1330e79162ac 8150534: C1 compilation fails with "Constant field loads are folded during parsing"
shade
parents: 36317
diff changeset
   256
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
36604
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   260
void Canonicalizer::do_LoadIndexed    (LoadIndexed*     x) {
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   261
  StableArrayConstant* array = x->array()->type()->as_StableArrayConstant();
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   262
  IntConstant* index = x->index()->type()->as_IntConstant();
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   263
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   264
  assert(array == NULL || FoldStableValues, "not enabled");
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   265
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   266
  // Constant fold loads from stable arrays.
39263
d139a133ba27 8158228: C1 incorrectly folds mismatched loads from stable arrays
thartmann
parents: 38060
diff changeset
   267
  if (!x->mismatched() && array != NULL && index != NULL) {
36604
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   268
    jint idx = index->value();
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   269
    if (idx < 0 || idx >= array->value()->length()) {
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   270
      // Leave the load as is. The range check will handle it.
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   271
      return;
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   272
    }
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   273
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   274
    ciConstant field_val = array->value()->element_value(idx);
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   275
    if (!field_val.is_null_or_zero()) {
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   276
      jint dimension = array->dimension();
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   277
      assert(dimension <= array->value()->array_type()->dimension(), "inconsistent info");
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   278
      ValueType* value = NULL;
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   279
      if (dimension > 1) {
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   280
        // Preserve information about the dimension for the element.
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   281
        assert(field_val.as_object()->is_array(), "not an array");
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   282
        value = new StableArrayConstant(field_val.as_object()->as_array(), dimension - 1);
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   283
      } else {
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   284
        assert(dimension == 1, "sanity");
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   285
        value = as_ValueType(field_val);
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   286
      }
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   287
      set_canonical(new Constant(value));
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   288
    }
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   289
  }
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   290
}
e9c073b0c19e 8143407: C1: @Stable array support
vlivanov
parents: 36552
diff changeset
   291
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
void Canonicalizer::do_StoreIndexed   (StoreIndexed*    x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
  // If a value is going to be stored into a field or array some of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
  // the conversions emitted by javac are unneeded because the fields
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  // are packed to their natural size.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  Convert* conv = x->value()->as_Convert();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
  if (conv) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
    Value value = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
    BasicType type = x->elt_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
    switch (conv->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
    case Bytecodes::_i2b: if (type == T_BYTE)  value = conv->value(); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
    case Bytecodes::_i2s: if (type == T_SHORT || type == T_BYTE) value = conv->value(); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
    case Bytecodes::_i2c: if (type == T_CHAR  || type == T_BYTE) value = conv->value(); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
    // limit this optimization to current block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
    if (value != NULL && in_current_block(conv)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
      set_canonical(new StoreIndexed(x->array(), x->index(), x->length(),
37480
291ee208fb72 8132051: Better byte behavior
coleenp
parents: 36604
diff changeset
   308
                                     x->elt_type(), value, x->state_before(),
291ee208fb72 8132051: Better byte behavior
coleenp
parents: 36604
diff changeset
   309
                                     x->check_boolean()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
void Canonicalizer::do_NegateOp(NegateOp* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
  ValueType* t = x->x()->type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
  if (t->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
    switch (t->tag()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
      case intTag   : set_constant(-t->as_IntConstant   ()->value()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
      case longTag  : set_constant(-t->as_LongConstant  ()->value()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
      case floatTag : set_constant(-t->as_FloatConstant ()->value()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
      case doubleTag: set_constant(-t->as_DoubleConstant()->value()); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
      default       : ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
void Canonicalizer::do_ArithmeticOp   (ArithmeticOp*    x) { do_Op2(x); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
void Canonicalizer::do_ShiftOp        (ShiftOp*         x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
  ValueType* t = x->x()->type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
  ValueType* t2 = x->y()->type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
  if (t->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
    switch (t->tag()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
    case intTag   : if (t->as_IntConstant()->value() == 0)         { set_constant(0); return; } break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
    case longTag  : if (t->as_LongConstant()->value() == (jlong)0) { set_constant(jlong_cast(0)); return; } break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
    default       : ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
    if (t2->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
      if (t->tag() == intTag) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
        int value = t->as_IntConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
        int shift = t2->as_IntConstant()->value() & 31;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
        jint mask = ~(~0 << (32 - shift));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
        if (shift == 0) mask = ~0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   348
        switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
          case Bytecodes::_ishl:  set_constant(value << shift); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
          case Bytecodes::_ishr:  set_constant(value >> shift); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
          case Bytecodes::_iushr: set_constant((value >> shift) & mask); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
      } else if (t->tag() == longTag) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
        jlong value = t->as_LongConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
        int shift = t2->as_IntConstant()->value() & 63;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
        jlong mask = ~(~jlong_cast(0) << (64 - shift));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
        if (shift == 0) mask = ~jlong_cast(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
        switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   359
          case Bytecodes::_lshl:  set_constant(value << shift); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
          case Bytecodes::_lshr:  set_constant(value >> shift); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
          case Bytecodes::_lushr: set_constant((value >> shift) & mask); return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   362
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
  if (t2->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
    switch (t2->tag()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
      case intTag   : if (t2->as_IntConstant()->value() == 0)  set_canonical(x->x()); return;
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   369
      case longTag  : if (t2->as_LongConstant()->value() == (jlong)0)  set_canonical(x->x()); return;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
      default       : ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
void Canonicalizer::do_LogicOp        (LogicOp*         x) { do_Op2(x); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
void Canonicalizer::do_CompareOp      (CompareOp*       x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
  if (x->x() == x->y()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
    switch (x->x()->type()->tag()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
      case longTag: set_constant(0); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
      case floatTag: {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
        FloatConstant* fc = x->x()->type()->as_FloatConstant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
        if (fc) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
          if (g_isnan(fc->value())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
            set_constant(x->op() == Bytecodes::_fcmpl ? -1 : 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
          } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
            set_constant(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
      case doubleTag: {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
        DoubleConstant* dc = x->x()->type()->as_DoubleConstant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
        if (dc) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
          if (g_isnan(dc->value())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
            set_constant(x->op() == Bytecodes::_dcmpl ? -1 : 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
          } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
            set_constant(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
  } else if (x->x()->type()->is_constant() && x->y()->type()->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
    switch (x->x()->type()->tag()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
      case longTag: {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
        jlong vx = x->x()->type()->as_LongConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
        jlong vy = x->y()->type()->as_LongConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
        if (vx == vy)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
          set_constant(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
        else if (vx < vy)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
          set_constant(-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
        else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
          set_constant(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
      case floatTag: {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
        float vx = x->x()->type()->as_FloatConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
        float vy = x->y()->type()->as_FloatConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
        if (g_isnan(vx) || g_isnan(vy))
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
          set_constant(x->op() == Bytecodes::_fcmpl ? -1 : 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
        else if (vx == vy)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
          set_constant(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
        else if (vx < vy)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
          set_constant(-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
        else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
          set_constant(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   431
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
      case doubleTag: {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   433
        double vx = x->x()->type()->as_DoubleConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   434
        double vy = x->y()->type()->as_DoubleConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
        if (g_isnan(vx) || g_isnan(vy))
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
          set_constant(x->op() == Bytecodes::_dcmpl ? -1 : 1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
        else if (vx == vy)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
          set_constant(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
        else if (vx < vy)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
          set_constant(-1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
        else
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
          set_constant(1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
void Canonicalizer::do_IfInstanceOf(IfInstanceOf*    x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
void Canonicalizer::do_IfOp(IfOp* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
  // Caution: do not use do_Op2(x) here for now since
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
  //          we map the condition to the op for now!
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
  move_const_to_right(x);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
void Canonicalizer::do_Intrinsic      (Intrinsic*       x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
  switch (x->id()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
  case vmIntrinsics::_floatToRawIntBits   : {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
    FloatConstant* c = x->argument_at(0)->type()->as_FloatConstant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
    if (c != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
      JavaValue v;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
      v.set_jfloat(c->value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
      set_constant(v.get_jint());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
  case vmIntrinsics::_intBitsToFloat      : {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
    IntConstant* c = x->argument_at(0)->type()->as_IntConstant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
    if (c != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
      JavaValue v;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
      v.set_jint(c->value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
      set_constant(v.get_jfloat());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
  case vmIntrinsics::_doubleToRawLongBits : {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
    DoubleConstant* c = x->argument_at(0)->type()->as_DoubleConstant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
    if (c != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
      JavaValue v;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
      v.set_jdouble(c->value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
      set_constant(v.get_jlong());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
  case vmIntrinsics::_longBitsToDouble    : {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
    LongConstant* c = x->argument_at(0)->type()->as_LongConstant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
    if (c != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
      JavaValue v;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
      v.set_jlong(c->value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
      set_constant(v.get_jdouble());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
    break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
  }
12949
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   498
  case vmIntrinsics::_isInstance          : {
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   499
    assert(x->number_of_arguments() == 2, "wrong type");
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   500
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   501
    InstanceConstant* c = x->argument_at(0)->type()->as_InstanceConstant();
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   502
    if (c != NULL && !c->value()->is_null_object()) {
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   503
      // ciInstance::java_mirror_type() returns non-NULL only for Java mirrors
36552
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   504
      ciType* t = c->value()->java_mirror_type();
12949
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   505
      if (t->is_klass()) {
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   506
        // substitute cls.isInstance(obj) of a constant Class into
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   507
        // an InstantOf instruction
12950
ab4f4afb3988 7174884: C1: failures after 7171890: assert(cur_state != NULL) failed: state_before must be set
twisti
parents: 12949
diff changeset
   508
        InstanceOf* i = new InstanceOf(t->as_klass(), x->argument_at(1), x->state_before());
12949
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   509
        set_canonical(i);
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   510
        // and try to canonicalize even further
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   511
        do_InstanceOf(i);
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   512
      } else {
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   513
        assert(t->is_primitive_type(), "should be a primitive type");
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   514
        // cls.isInstance(obj) always returns false for primitive classes
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   515
        set_constant(0);
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   516
      }
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   517
    }
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   518
    break;
e125ba4c16bd 7171890: C1: add Class.isInstance intrinsic
roland
parents: 12947
diff changeset
   519
  }
36552
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   520
  case vmIntrinsics::_isPrimitive        : {
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   521
    assert(x->number_of_arguments() == 1, "wrong type");
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   522
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   523
    // Class.isPrimitive is known on constant classes:
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   524
    InstanceConstant* c = x->argument_at(0)->type()->as_InstanceConstant();
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   525
    if (c != NULL && !c->value()->is_null_object()) {
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   526
      ciType* t = c->value()->java_mirror_type();
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   527
      set_constant(t->is_primitive_type());
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   528
    }
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   529
    break;
0db9be43cebb 8150669: C1 intrinsic for Class.isPrimitive
shade
parents: 36325
diff changeset
   530
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
void Canonicalizer::do_Convert        (Convert*         x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
  if (x->value()->type()->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
    switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
    case Bytecodes::_i2b:  set_constant((int)((x->value()->type()->as_IntConstant()->value() << 24) >> 24)); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
    case Bytecodes::_i2s:  set_constant((int)((x->value()->type()->as_IntConstant()->value() << 16) >> 16)); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
    case Bytecodes::_i2c:  set_constant((int)(x->value()->type()->as_IntConstant()->value() & ((1<<16)-1))); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
    case Bytecodes::_i2l:  set_constant((jlong)(x->value()->type()->as_IntConstant()->value()));             break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
    case Bytecodes::_i2f:  set_constant((float)(x->value()->type()->as_IntConstant()->value()));             break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
    case Bytecodes::_i2d:  set_constant((double)(x->value()->type()->as_IntConstant()->value()));            break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
    case Bytecodes::_l2i:  set_constant((int)(x->value()->type()->as_LongConstant()->value()));              break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
    case Bytecodes::_l2f:  set_constant(SharedRuntime::l2f(x->value()->type()->as_LongConstant()->value())); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
    case Bytecodes::_l2d:  set_constant(SharedRuntime::l2d(x->value()->type()->as_LongConstant()->value())); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
    case Bytecodes::_f2d:  set_constant((double)(x->value()->type()->as_FloatConstant()->value()));          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
    case Bytecodes::_f2i:  set_constant(SharedRuntime::f2i(x->value()->type()->as_FloatConstant()->value())); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
    case Bytecodes::_f2l:  set_constant(SharedRuntime::f2l(x->value()->type()->as_FloatConstant()->value())); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
    case Bytecodes::_d2f:  set_constant((float)(x->value()->type()->as_DoubleConstant()->value()));          break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
    case Bytecodes::_d2i:  set_constant(SharedRuntime::d2i(x->value()->type()->as_DoubleConstant()->value())); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
    case Bytecodes::_d2l:  set_constant(SharedRuntime::d2l(x->value()->type()->as_DoubleConstant()->value())); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
    default:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
      ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
  Value value = x->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
  BasicType type = T_ILLEGAL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
  LoadField* lf = value->as_LoadField();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
  if (lf) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
    type = lf->field_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
    LoadIndexed* li = value->as_LoadIndexed();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
    if (li) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
      type = li->elt_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
      Convert* conv = value->as_Convert();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
      if (conv) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
        switch (conv->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
          case Bytecodes::_i2b: type = T_BYTE;  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
          case Bytecodes::_i2s: type = T_SHORT; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
          case Bytecodes::_i2c: type = T_CHAR;  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
  if (type != T_ILLEGAL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
    switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
      case Bytecodes::_i2b: if (type == T_BYTE)                    set_canonical(x->value()); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
      case Bytecodes::_i2s: if (type == T_SHORT || type == T_BYTE) set_canonical(x->value()); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
      case Bytecodes::_i2c: if (type == T_CHAR)                    set_canonical(x->value()); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
    Op2* op2 = x->value()->as_Op2();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
    if (op2 && op2->op() == Bytecodes::_iand && op2->y()->type()->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
      jint safebits = 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   587
      jint mask = op2->y()->type()->as_IntConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
      switch (x->op()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
        case Bytecodes::_i2b: safebits = 0x7f;   break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
        case Bytecodes::_i2s: safebits = 0x7fff; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
        case Bytecodes::_i2c: safebits = 0xffff; break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
      // When casting a masked integer to a smaller signed type, if
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
      // the mask doesn't include the sign bit the cast isn't needed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
      if (safebits && (mask & ~safebits) == 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
        set_canonical(x->value());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   602
489c9b5090e2 Initial load
duke
parents:
diff changeset
   603
void Canonicalizer::do_NullCheck      (NullCheck*       x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
  if (x->obj()->as_NewArray() != NULL || x->obj()->as_NewInstance() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
    set_canonical(x->obj());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
    Constant* con = x->obj()->as_Constant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
    if (con) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
      ObjectType* c = con->type()->as_ObjectType();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
      if (c && c->is_loaded()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
        ObjectConstant* oc = c->as_ObjectConstant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
        if (!oc || !oc->value()->is_null_object()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   613
          set_canonical(con);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   614
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
13391
30245956af37 7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents: 12950
diff changeset
   620
void Canonicalizer::do_TypeCast       (TypeCast*        x) {}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
void Canonicalizer::do_Invoke         (Invoke*          x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
void Canonicalizer::do_NewInstance    (NewInstance*     x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   623
void Canonicalizer::do_NewTypeArray   (NewTypeArray*    x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   624
void Canonicalizer::do_NewObjectArray (NewObjectArray*  x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   625
void Canonicalizer::do_NewMultiArray  (NewMultiArray*   x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   626
void Canonicalizer::do_CheckCast      (CheckCast*       x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   627
  if (x->klass()->is_loaded()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   628
    Value obj = x->obj();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   629
    ciType* klass = obj->exact_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
    if (klass == NULL) klass = obj->declared_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   631
    if (klass != NULL && klass->is_loaded() && klass->is_subtype_of(x->klass())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   632
      set_canonical(obj);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   635
    // checkcast of null returns null
489c9b5090e2 Initial load
duke
parents:
diff changeset
   636
    if (obj->as_Constant() && obj->type()->as_ObjectType()->constant_value()->is_null_object()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   637
      set_canonical(obj);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   638
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   639
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   640
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   641
void Canonicalizer::do_InstanceOf     (InstanceOf*      x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
  if (x->klass()->is_loaded()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
    Value obj = x->obj();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
    ciType* exact = obj->exact_type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
    if (exact != NULL && exact->is_loaded() && (obj->as_NewInstance() || obj->as_NewArray())) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
      set_constant(exact->is_subtype_of(x->klass()) ? 1 : 0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
    // instanceof null returns false
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
    if (obj->as_Constant() && obj->type()->as_ObjectType()->constant_value()->is_null_object()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
      set_constant(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
void Canonicalizer::do_MonitorEnter   (MonitorEnter*    x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
void Canonicalizer::do_MonitorExit    (MonitorExit*     x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   658
void Canonicalizer::do_BlockBegin     (BlockBegin*      x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   659
void Canonicalizer::do_Goto           (Goto*            x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   660
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
static bool is_true(jlong x, If::Condition cond, jlong y) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
  switch (cond) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
    case If::eql: return x == y;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
    case If::neq: return x != y;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
    case If::lss: return x <  y;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
    case If::leq: return x <= y;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   668
    case If::gtr: return x >  y;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
    case If::geq: return x >= y;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
  ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
11787
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   675
static bool is_safepoint(BlockEnd* x, BlockBegin* sux) {
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   676
  // An Instruction with multiple successors, x, is replaced by a Goto
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   677
  // to a single successor, sux. Is a safepoint check needed = was the
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   678
  // instruction being replaced a safepoint and the single remaining
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   679
  // successor a back branch?
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   680
  return x->is_safepoint() && (sux->bci() < x->state_before()->bci());
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   681
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   682
489c9b5090e2 Initial load
duke
parents:
diff changeset
   683
void Canonicalizer::do_If(If* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   684
  // move const to right
489c9b5090e2 Initial load
duke
parents:
diff changeset
   685
  if (x->x()->type()->is_constant()) x->swap_operands();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   686
  // simplify
489c9b5090e2 Initial load
duke
parents:
diff changeset
   687
  const Value l = x->x(); ValueType* lt = l->type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   688
  const Value r = x->y(); ValueType* rt = r->type();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   689
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
  if (l == r && !lt->is_float_kind()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
    // pattern: If (a cond a) => simplify to Goto
33589
7cbd1b2c139b 8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents: 28954
diff changeset
   692
    BlockBegin* sux = NULL;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
    switch (x->cond()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   694
    case If::eql: sux = x->sux_for(true);  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
    case If::neq: sux = x->sux_for(false); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   696
    case If::lss: sux = x->sux_for(false); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   697
    case If::leq: sux = x->sux_for(true);  break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   698
    case If::gtr: sux = x->sux_for(false); break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   699
    case If::geq: sux = x->sux_for(true);  break;
33589
7cbd1b2c139b 8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents: 28954
diff changeset
   700
    default: ShouldNotReachHere();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   701
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
    // If is a safepoint then the debug information should come from the state_before of the If.
11787
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   703
    set_canonical(new Goto(sux, x->state_before(), is_safepoint(x, sux)));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
  if (lt->is_constant() && rt->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   708
    if (x->x()->as_Constant() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   709
      // pattern: If (lc cond rc) => simplify to: Goto
489c9b5090e2 Initial load
duke
parents:
diff changeset
   710
      BlockBegin* sux = x->x()->as_Constant()->compare(x->cond(), x->y(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   711
                                                       x->sux_for(true),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   712
                                                       x->sux_for(false));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   713
      if (sux != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   714
        // If is a safepoint then the debug information should come from the state_before of the If.
11787
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   715
        set_canonical(new Goto(sux, x->state_before(), is_safepoint(x, sux)));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   716
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   717
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   718
  } else if (rt->as_IntConstant() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   719
    // pattern: If (l cond rc) => investigate further
489c9b5090e2 Initial load
duke
parents:
diff changeset
   720
    const jint rc = rt->as_IntConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   721
    if (l->as_CompareOp() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
      // pattern: If ((a cmp b) cond rc) => simplify to: If (x cond y) or: Goto
489c9b5090e2 Initial load
duke
parents:
diff changeset
   723
      CompareOp* cmp = l->as_CompareOp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   724
      bool unordered_is_less = cmp->op() == Bytecodes::_fcmpl || cmp->op() == Bytecodes::_dcmpl;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   725
      BlockBegin* lss_sux = x->sux_for(is_true(-1, x->cond(), rc)); // successor for a < b
489c9b5090e2 Initial load
duke
parents:
diff changeset
   726
      BlockBegin* eql_sux = x->sux_for(is_true( 0, x->cond(), rc)); // successor for a = b
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
      BlockBegin* gtr_sux = x->sux_for(is_true(+1, x->cond(), rc)); // successor for a > b
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
      BlockBegin* nan_sux = unordered_is_less ? lss_sux : gtr_sux ; // successor for unordered
489c9b5090e2 Initial load
duke
parents:
diff changeset
   729
      // Note: At this point all successors (lss_sux, eql_sux, gtr_sux, nan_sux) are
489c9b5090e2 Initial load
duke
parents:
diff changeset
   730
      //       equal to x->tsux() or x->fsux(). Furthermore, nan_sux equals either
489c9b5090e2 Initial load
duke
parents:
diff changeset
   731
      //       lss_sux or gtr_sux.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   732
      if (lss_sux == eql_sux && eql_sux == gtr_sux) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
        // all successors identical => simplify to: Goto
489c9b5090e2 Initial load
duke
parents:
diff changeset
   734
        set_canonical(new Goto(lss_sux, x->state_before(), x->is_safepoint()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   735
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   736
        // two successors differ and two successors are the same => simplify to: If (x cmp y)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   737
        // determine new condition & successors
33589
7cbd1b2c139b 8139040: Fix initializations before ShouldNotReachHere() etc. and enable -Wuninitialized on linux.
goetz
parents: 28954
diff changeset
   738
        If::Condition cond = If::eql;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   739
        BlockBegin* tsux = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   740
        BlockBegin* fsux = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   741
             if (lss_sux == eql_sux) { cond = If::leq; tsux = lss_sux; fsux = gtr_sux; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   742
        else if (lss_sux == gtr_sux) { cond = If::neq; tsux = lss_sux; fsux = eql_sux; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   743
        else if (eql_sux == gtr_sux) { cond = If::geq; tsux = eql_sux; fsux = lss_sux; }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   744
        else                         { ShouldNotReachHere();                           }
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   745
        If* canon = new If(cmp->x(), cond, nan_sux == tsux, cmp->y(), tsux, fsux, cmp->state_before(), x->is_safepoint());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   746
        if (cmp->x() == cmp->y()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   747
          do_If(canon);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   748
        } else {
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   749
          if (compilation()->profile_branches()) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   750
            // TODO: If profiling, leave floating point comparisons unoptimized.
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   751
            // We currently do not support profiling of the unordered case.
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   752
            switch(cmp->op()) {
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   753
              case Bytecodes::_fcmpl: case Bytecodes::_fcmpg:
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   754
              case Bytecodes::_dcmpl: case Bytecodes::_dcmpg:
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   755
                set_canonical(x);
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   756
                return;
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   757
            }
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
   758
          }
12947
ba8ba6c2dd83 7172843: C1: fix "assert(has_printable_bci()) failed: _printable_bci should have been set"
twisti
parents: 11886
diff changeset
   759
          set_bci(cmp->state_before()->bci());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
          set_canonical(canon);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
    } else if (l->as_InstanceOf() != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
      // NOTE: Code permanently disabled for now since it leaves the old InstanceOf
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
      //       instruction in the graph (it is pinned). Need to fix this at some point.
6461
cfc616b49f58 6919069: client compiler needs to capture more profile information for tiered work
iveresov
parents: 6453
diff changeset
   766
      //       It should also be left in the graph when generating a profiled method version or Goto
cfc616b49f58 6919069: client compiler needs to capture more profile information for tiered work
iveresov
parents: 6453
diff changeset
   767
      //       has to know that it was an InstanceOf.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   768
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   769
      // pattern: If ((obj instanceof klass) cond rc) => simplify to: IfInstanceOf or: Goto
489c9b5090e2 Initial load
duke
parents:
diff changeset
   770
      InstanceOf* inst = l->as_InstanceOf();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   771
      BlockBegin* is_inst_sux = x->sux_for(is_true(1, x->cond(), rc)); // successor for instanceof == 1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   772
      BlockBegin* no_inst_sux = x->sux_for(is_true(0, x->cond(), rc)); // successor for instanceof == 0
489c9b5090e2 Initial load
duke
parents:
diff changeset
   773
      if (is_inst_sux == no_inst_sux && inst->is_loaded()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   774
        // both successors identical and klass is loaded => simplify to: Goto
489c9b5090e2 Initial load
duke
parents:
diff changeset
   775
        set_canonical(new Goto(is_inst_sux, x->state_before(), x->is_safepoint()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
      } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   777
        // successors differ => simplify to: IfInstanceOf
6745
a34ef8968a84 6986046: C1 valuestack cleanup
roland
parents: 6461
diff changeset
   778
        set_canonical(new IfInstanceOf(inst->klass(), inst->obj(), true, inst->state_before()->bci(), is_inst_sux, no_inst_sux));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   779
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   780
    }
33633
8a83967eb351 8141044: C1 should fold (this == null) to false
shade
parents: 28954
diff changeset
   781
  } else if (rt == objectNull &&
8a83967eb351 8141044: C1 should fold (this == null) to false
shade
parents: 28954
diff changeset
   782
           (l->as_NewInstance() || l->as_NewArray() ||
36058
caff909b4e1f 8149356: Leftover from JDK-8141044: UseNewCode usage
shade
parents: 33638
diff changeset
   783
             (l->as_Local() && l->as_Local()->is_receiver()))) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   784
    if (x->cond() == Instruction::eql) {
11787
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   785
      BlockBegin* sux = x->fsux();
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   786
      set_canonical(new Goto(sux, x->state_before(), is_safepoint(x, sux)));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   787
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   788
      assert(x->cond() == Instruction::neq, "only other valid case");
11787
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   789
      BlockBegin* sux = x->tsux();
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   790
      set_canonical(new Goto(sux, x->state_before(), is_safepoint(x, sux)));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   791
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   792
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   793
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   794
489c9b5090e2 Initial load
duke
parents:
diff changeset
   795
489c9b5090e2 Initial load
duke
parents:
diff changeset
   796
void Canonicalizer::do_TableSwitch(TableSwitch* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   797
  if (x->tag()->type()->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   798
    int v = x->tag()->type()->as_IntConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   799
    BlockBegin* sux = x->default_sux();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   800
    if (v >= x->lo_key() && v <= x->hi_key()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   801
      sux = x->sux_at(v - x->lo_key());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   802
    }
11787
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   803
    set_canonical(new Goto(sux, x->state_before(), is_safepoint(x, sux)));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   804
  } else if (x->number_of_sux() == 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   805
    // NOTE: Code permanently disabled for now since the switch statement's
489c9b5090e2 Initial load
duke
parents:
diff changeset
   806
    //       tag expression may produce side-effects in which case it must
489c9b5090e2 Initial load
duke
parents:
diff changeset
   807
    //       be executed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   808
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   809
    // simplify to Goto
489c9b5090e2 Initial load
duke
parents:
diff changeset
   810
    set_canonical(new Goto(x->default_sux(), x->state_before(), x->is_safepoint()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   811
  } else if (x->number_of_sux() == 2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
    // NOTE: Code permanently disabled for now since it produces two new nodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   813
    //       (Constant & If) and the Canonicalizer cannot return them correctly
489c9b5090e2 Initial load
duke
parents:
diff changeset
   814
    //       yet. For now we copied the corresponding code directly into the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
    //       GraphBuilder (i.e., we should never reach here).
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
    // simplify to If
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
    assert(x->lo_key() == x->hi_key(), "keys must be the same");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
    Constant* key = new Constant(new IntConstant(x->lo_key()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
    set_canonical(new If(x->tag(), If::eql, true, key, x->sux_at(0), x->default_sux(), x->state_before(), x->is_safepoint()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   822
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   823
489c9b5090e2 Initial load
duke
parents:
diff changeset
   824
489c9b5090e2 Initial load
duke
parents:
diff changeset
   825
void Canonicalizer::do_LookupSwitch(LookupSwitch* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   826
  if (x->tag()->type()->is_constant()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   827
    int v = x->tag()->type()->as_IntConstant()->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   828
    BlockBegin* sux = x->default_sux();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   829
    for (int i = 0; i < x->length(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   830
      if (v == x->key_at(i)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   831
        sux = x->sux_at(i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   832
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
    }
11787
cfb5950b7706 7126041: jdk7u4 b05 and b06 crash with RubyMine 3.2.4, works well with b04
roland
parents: 8671
diff changeset
   834
    set_canonical(new Goto(sux, x->state_before(), is_safepoint(x, sux)));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   835
  } else if (x->number_of_sux() == 1) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
    // NOTE: Code permanently disabled for now since the switch statement's
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
    //       tag expression may produce side-effects in which case it must
489c9b5090e2 Initial load
duke
parents:
diff changeset
   838
    //       be executed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
    // simplify to Goto
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
    set_canonical(new Goto(x->default_sux(), x->state_before(), x->is_safepoint()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   842
  } else if (x->number_of_sux() == 2) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
    // NOTE: Code permanently disabled for now since it produces two new nodes
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
    //       (Constant & If) and the Canonicalizer cannot return them correctly
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
    //       yet. For now we copied the corresponding code directly into the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   846
    //       GraphBuilder (i.e., we should never reach here).
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
    return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
    // simplify to If
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
    assert(x->length() == 1, "length must be the same");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
    Constant* key = new Constant(new IntConstant(x->key_at(0)));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
    set_canonical(new If(x->tag(), If::eql, true, key, x->sux_at(0), x->default_sux(), x->state_before(), x->is_safepoint()));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
489c9b5090e2 Initial load
duke
parents:
diff changeset
   856
void Canonicalizer::do_Return         (Return*          x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   857
void Canonicalizer::do_Throw          (Throw*           x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   858
void Canonicalizer::do_Base           (Base*            x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
void Canonicalizer::do_OsrEntry       (OsrEntry*        x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
void Canonicalizer::do_ExceptionObject(ExceptionObject* x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
489c9b5090e2 Initial load
duke
parents:
diff changeset
   862
static bool match_index_and_scale(Instruction*  instr,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   863
                                  Instruction** index,
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   864
                                  int*          log2_scale) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   865
  // Skip conversion ops. This works only on 32bit because of the implicit l2i that the
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   866
  // unsafe performs.
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   867
#ifndef _LP64
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
  Convert* convert = instr->as_Convert();
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   869
  if (convert != NULL && convert->op() == Bytecodes::_i2l) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   870
    assert(convert->value()->type() == intType, "invalid input type");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
    instr = convert->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
  }
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   873
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
489c9b5090e2 Initial load
duke
parents:
diff changeset
   875
  ShiftOp* shift = instr->as_ShiftOp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
  if (shift != NULL) {
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   877
    if (shift->op() == Bytecodes::_lshl) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   878
      assert(shift->x()->type() == longType, "invalid input type");
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   879
    } else {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   880
#ifndef _LP64
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   881
      if (shift->op() == Bytecodes::_ishl) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   882
        assert(shift->x()->type() == intType, "invalid input type");
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   883
      } else {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   884
        return false;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   885
      }
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   886
#else
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   887
      return false;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   888
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   889
    }
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   890
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   891
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   892
    // Constant shift value?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   893
    Constant* con = shift->y()->as_Constant();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   894
    if (con == NULL) return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   895
    // Well-known type and value?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   896
    IntConstant* val = con->type()->as_IntConstant();
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   897
    assert(val != NULL, "Should be an int constant");
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   898
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
    *index = shift->x();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
    int tmp_scale = val->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   901
    if (tmp_scale >= 0 && tmp_scale < 4) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   902
      *log2_scale = tmp_scale;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   906
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   907
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   908
489c9b5090e2 Initial load
duke
parents:
diff changeset
   909
  ArithmeticOp* arith = instr->as_ArithmeticOp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   910
  if (arith != NULL) {
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   911
    // See if either arg is a known constant
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   912
    Constant* con = arith->x()->as_Constant();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   913
    if (con != NULL) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   914
      *index = arith->y();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   915
    } else {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   916
      con = arith->y()->as_Constant();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   917
      if (con == NULL) return false;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   918
      *index = arith->x();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   919
    }
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   920
    long const_value;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   921
    // Check for integer multiply
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   922
    if (arith->op() == Bytecodes::_lmul) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   923
      assert((*index)->type() == longType, "invalid input type");
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   924
      LongConstant* val = con->type()->as_LongConstant();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   925
      assert(val != NULL, "expecting a long constant");
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   926
      const_value = val->value();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   927
    } else {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   928
#ifndef _LP64
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   929
      if (arith->op() == Bytecodes::_imul) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   930
        assert((*index)->type() == intType, "invalid input type");
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   931
        IntConstant* val = con->type()->as_IntConstant();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   932
        assert(val != NULL, "expecting an int constant");
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   933
        const_value = val->value();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   934
      } else {
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   935
        return false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   936
      }
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   937
#else
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   938
      return false;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   939
#endif
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   940
    }
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   941
    switch (const_value) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   942
    case 1: *log2_scale = 0; return true;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   943
    case 2: *log2_scale = 1; return true;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   944
    case 4: *log2_scale = 2; return true;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   945
    case 8: *log2_scale = 3; return true;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   946
    default:            return false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   947
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   948
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
  // Unknown instruction sequence; don't touch it
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
  return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   952
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   953
489c9b5090e2 Initial load
duke
parents:
diff changeset
   954
489c9b5090e2 Initial load
duke
parents:
diff changeset
   955
static bool match(UnsafeRawOp* x,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
                  Instruction** base,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   957
                  Instruction** index,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   958
                  int*          log2_scale) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   959
  ArithmeticOp* root = x->base()->as_ArithmeticOp();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   960
  if (root == NULL) return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   961
  // Limit ourselves to addition for now
489c9b5090e2 Initial load
duke
parents:
diff changeset
   962
  if (root->op() != Bytecodes::_ladd) return false;
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   963
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   964
  bool match_found = false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   965
  // Try to find shift or scale op
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   966
  if (match_index_and_scale(root->y(), index, log2_scale)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   967
    *base = root->x();
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   968
    match_found = true;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   969
  } else if (match_index_and_scale(root->x(), index, log2_scale)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   970
    *base = root->y();
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   971
    match_found = true;
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   972
  } else if (NOT_LP64(root->y()->as_Convert() != NULL) LP64_ONLY(false)) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   973
    // Skipping i2l works only on 32bit because of the implicit l2i that the unsafe performs.
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   974
    // 64bit needs a real sign-extending conversion.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   975
    Convert* convert = root->y()->as_Convert();
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   976
    if (convert->op() == Bytecodes::_i2l) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   977
      assert(convert->value()->type() == intType, "should be an int");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
      // pick base and index, setting scale at 1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
      *base  = root->x();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   980
      *index = convert->value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   981
      *log2_scale = 0;
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   982
      match_found = true;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   983
    }
26819
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   984
  }
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   985
  // The default solution
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   986
  if (!match_found) {
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   987
    *base = root->x();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   988
    *index = root->y();
9f09d23f946c 8058744: Crash in C1 OSRed method w/ Unsafe usage
iveresov
parents: 22234
diff changeset
   989
    *log2_scale = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   990
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   991
489c9b5090e2 Initial load
duke
parents:
diff changeset
   992
  // If the value is pinned then it will be always be computed so
489c9b5090e2 Initial load
duke
parents:
diff changeset
   993
  // there's no profit to reshaping the expression.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   994
  return !root->is_pinned();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   995
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   996
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
489c9b5090e2 Initial load
duke
parents:
diff changeset
   998
void Canonicalizer::do_UnsafeRawOp(UnsafeRawOp* x) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   999
  Instruction* base = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1000
  Instruction* index = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1001
  int          log2_scale;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1002
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1003
  if (match(x, &base, &index, &log2_scale)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1004
    x->set_base(base);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1005
    x->set_index(index);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1006
    x->set_log2_scale(log2_scale);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1007
    if (PrintUnsafeOptimization) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1008
      tty->print_cr("Canonicalizer: UnsafeRawOp id %d: base = id %d, index = id %d, log2_scale = %d",
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1009
                    x->id(), x->base()->id(), x->index()->id(), x->log2_scale());
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1010
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1011
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1012
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1013
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1014
void Canonicalizer::do_RoundFP(RoundFP* x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1015
void Canonicalizer::do_UnsafeGetRaw(UnsafeGetRaw* x) { if (OptimizeUnsafes) do_UnsafeRawOp(x); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1016
void Canonicalizer::do_UnsafePutRaw(UnsafePutRaw* x) { if (OptimizeUnsafes) do_UnsafeRawOp(x); }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1017
void Canonicalizer::do_UnsafeGetObject(UnsafeGetObject* x) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1018
void Canonicalizer::do_UnsafePutObject(UnsafePutObject* x) {}
13886
8d82c4dfa722 7023898: Intrinsify AtomicLongFieldUpdater.getAndIncrement()
roland
parents: 13391
diff changeset
  1019
void Canonicalizer::do_UnsafeGetAndSetObject(UnsafeGetAndSetObject* x) {}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1020
void Canonicalizer::do_ProfileCall(ProfileCall* x) {}
20709
034be898bf04 8026054: New type profiling points: type of return values at calls
roland
parents: 17011
diff changeset
  1021
void Canonicalizer::do_ProfileReturnType(ProfileReturnType* x) {}
6453
970dc585ab63 6953144: Tiered compilation
iveresov
parents: 5707
diff changeset
  1022
void Canonicalizer::do_ProfileInvoke(ProfileInvoke* x) {}
8065
7ca689ce3d32 6809483: hotspot:::method_entry are not correctly generated for "method()V"
never
parents: 7397
diff changeset
  1023
void Canonicalizer::do_RuntimeCall(RuntimeCall* x) {}
16611
6807a703dd6b 7153771: array bound check elimination for c1
roland
parents: 13963
diff changeset
  1024
void Canonicalizer::do_RangeCheckPredicate(RangeCheckPredicate* x) {}
17011
def8879c5b81 8011648: C1: optimized build is broken after 7153771
roland
parents: 16611
diff changeset
  1025
#ifdef ASSERT
16611
6807a703dd6b 7153771: array bound check elimination for c1
roland
parents: 13963
diff changeset
  1026
void Canonicalizer::do_Assert(Assert* x) {}
17011
def8879c5b81 8011648: C1: optimized build is broken after 7153771
roland
parents: 16611
diff changeset
  1027
#endif
11886
feebf5c9f40c 7120481: storeStore barrier in constructor with final field
jiangli
parents: 11787
diff changeset
  1028
void Canonicalizer::do_MemBar(MemBar* x) {}