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