src/hotspot/share/ci/ciMethodBlocks.cpp
author tschatzl
Wed, 08 Aug 2018 15:31:06 +0200
changeset 51333 f6641fcf7b7e
parent 47216 71c04702a3d5
permissions -rw-r--r--
8208670: Compiler changes to allow enabling -Wreorder Reviewed-by: kvn
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: 38658
diff changeset
     2
 * Copyright (c) 2006, 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: 3261
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 3261
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: 3261
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: 6180
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6180
diff changeset
    26
#include "ci/ciMethodBlocks.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6180
diff changeset
    27
#include "ci/ciStreams.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6180
diff changeset
    28
#include "interpreter/bytecode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6180
diff changeset
    29
#include "utilities/copy.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    30
489c9b5090e2 Initial load
duke
parents:
diff changeset
    31
// ciMethodBlocks
489c9b5090e2 Initial load
duke
parents:
diff changeset
    32
489c9b5090e2 Initial load
duke
parents:
diff changeset
    33
489c9b5090e2 Initial load
duke
parents:
diff changeset
    34
489c9b5090e2 Initial load
duke
parents:
diff changeset
    35
ciBlock *ciMethodBlocks::block_containing(int bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    36
  ciBlock *blk = _bci_to_block[bci];
489c9b5090e2 Initial load
duke
parents:
diff changeset
    37
  return blk;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    38
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
bool ciMethodBlocks::is_block_start(int bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    41
  assert(bci >=0 && bci < _code_size, "valid bytecode range");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    42
  ciBlock *b = _bci_to_block[bci];
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
  assert(b != NULL, "must have block for bytecode");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
  return b->start_bci() == bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
// ------------------------------------------------------------------
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
// ciMethodBlocks::split_block_at
489c9b5090e2 Initial load
duke
parents:
diff changeset
    49
//
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
// Split the block spanning bci into two separate ranges.  The former
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
// block becomes the second half and a new range is created for the
489c9b5090e2 Initial load
duke
parents:
diff changeset
    52
// first half.  Returns the range beginning at bci.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    53
ciBlock *ciMethodBlocks::split_block_at(int bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    54
  ciBlock *former_block = block_containing(bci);
1399
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
    55
  ciBlock *new_block = new(_arena) ciBlock(_method, _num_blocks++, former_block->start_bci());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    56
  _blocks->append(new_block);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
  assert(former_block != NULL, "must not be NULL");
489c9b5090e2 Initial load
duke
parents:
diff changeset
    58
  new_block->set_limit_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    59
  former_block->set_start_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    60
  for (int pos=bci-1; pos >= 0; pos--) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    61
    ciBlock *current_block = block_containing(pos);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    62
    if (current_block == former_block) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
      // Replace it.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    64
      _bci_to_block[pos] = new_block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    65
    } else if (current_block == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    66
      // Non-bytecode start.  Skip.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    69
      // We are done with our backwards walk
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
  }
199
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    73
  // Move an exception handler information if needed.
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    74
  if (former_block->is_handler()) {
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    75
    int ex_start = former_block->ex_start_bci();
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    76
    int ex_end = former_block->ex_limit_bci();
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    77
    new_block->set_exception_range(ex_start, ex_end);
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    78
    // Clear information in former_block.
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    79
    former_block->clear_exception_handler();
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
    80
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
  return former_block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    82
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
ciBlock *ciMethodBlocks::make_block_at(int bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
  ciBlock *cb = block_containing(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
  if (cb == NULL ) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
    // This is our first time visiting this bytecode.  Create
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
    // a fresh block and assign it this starting point.
1399
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
    89
    ciBlock *nb = new(_arena) ciBlock(_method, _num_blocks++, bci);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
    _blocks->append(nb);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
     _bci_to_block[bci] = nb;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    92
    return nb;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    93
  } else if (cb->start_bci() == bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
    // The block begins at bci.  Simply return it.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
    return cb;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
    // We have already created a block containing bci but
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
    // not starting at bci.  This existing block needs to
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
    // be split into two.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
    return split_block_at(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
1399
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
   104
ciBlock *ciMethodBlocks::make_dummy_block() {
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
   105
  ciBlock *dum = new(_arena) ciBlock(_method, -1, 0);
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
   106
  return dum;
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
   107
}
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
   108
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
void ciMethodBlocks::do_analysis() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
  ciBytecodeStream s(_method);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
  ciBlock *cur_block = block_containing(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  int limit_bci = _method->code_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
  while (s.next() != ciBytecodeStream::EOBC()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
    int bci = s.cur_bci();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
    // Determine if a new block has been made at the current bci.  If
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
    // this block differs from our current range, switch to the new
489c9b5090e2 Initial load
duke
parents:
diff changeset
   118
    // one and end the old one.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   119
    assert(cur_block != NULL, "must always have a current block");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   120
    ciBlock *new_block = block_containing(bci);
199
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   121
    if (new_block == NULL || new_block == cur_block) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
      // We have not marked this bci as the start of a new block.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
      // Keep interpreting the current_range.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   124
      _bci_to_block[bci] = cur_block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   125
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   126
      cur_block->set_limit_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   127
      cur_block = new_block;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   129
489c9b5090e2 Initial load
duke
parents:
diff changeset
   130
    switch (s.cur_bc()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
      case Bytecodes::_ifeq        :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
      case Bytecodes::_ifne        :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
      case Bytecodes::_iflt        :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
      case Bytecodes::_ifge        :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
      case Bytecodes::_ifgt        :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   136
      case Bytecodes::_ifle        :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   137
      case Bytecodes::_if_icmpeq   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
      case Bytecodes::_if_icmpne   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
      case Bytecodes::_if_icmplt   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
      case Bytecodes::_if_icmpge   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
      case Bytecodes::_if_icmpgt   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
      case Bytecodes::_if_icmple   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
      case Bytecodes::_if_acmpeq   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   144
      case Bytecodes::_if_acmpne   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   145
      case Bytecodes::_ifnull      :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   146
      case Bytecodes::_ifnonnull   :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   147
      {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   148
        cur_block->set_control_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
        ciBlock *fall_through = make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
        int dest_bci = s.get_dest();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
        ciBlock *dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   153
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
      case Bytecodes::_goto        :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
      {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   157
        cur_block->set_control_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   158
        if (s.next_bci() < limit_bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   159
          (void) make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   160
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   161
        int dest_bci = s.get_dest();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   162
        ciBlock *dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   165
489c9b5090e2 Initial load
duke
parents:
diff changeset
   166
      case Bytecodes::_jsr         :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
      {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
        cur_block->set_control_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   169
        ciBlock *ret = make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   170
        int dest_bci = s.get_dest();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   171
        ciBlock *dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   172
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   173
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   174
489c9b5090e2 Initial load
duke
parents:
diff changeset
   175
      case Bytecodes::_tableswitch :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
        {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
          cur_block->set_control_bci(bci);
7913
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   178
          Bytecode_tableswitch sw(&s);
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   179
          int len = sw.length();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
          ciBlock *dest;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
          int dest_bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   182
          for (int i = 0; i < len; i++) {
7913
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   183
            dest_bci = s.cur_bci() + sw.dest_offset_at(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   184
            dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   185
          }
7913
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   186
          dest_bci = s.cur_bci() + sw.default_offset();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   187
          make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   188
          if (s.next_bci() < limit_bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   189
            dest = make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   190
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
      case Bytecodes::_lookupswitch:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
        {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
          cur_block->set_control_bci(bci);
7913
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   197
          Bytecode_lookupswitch sw(&s);
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   198
          int len = sw.number_of_pairs();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
          ciBlock *dest;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
          int dest_bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
          for (int i = 0; i < len; i++) {
7913
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   202
            dest_bci = s.cur_bci() + sw.pair_at(i).offset();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
            dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
          }
7913
dd096a83bdbb 4926272: methodOopDesc::method_from_bcp is unsafe
never
parents: 7397
diff changeset
   205
          dest_bci = s.cur_bci() + sw.default_offset();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   206
          dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
          if (s.next_bci() < limit_bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   208
            dest = make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
      case Bytecodes::_goto_w      :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
      {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
        cur_block->set_control_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
        if (s.next_bci() < limit_bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   217
          (void) make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
        int dest_bci = s.get_far_dest();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
        ciBlock *dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
      case Bytecodes::_jsr_w       :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
      {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
        cur_block->set_control_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
        ciBlock *ret = make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
        int dest_bci = s.get_far_dest();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
        ciBlock *dest = make_block_at(dest_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   232
489c9b5090e2 Initial load
duke
parents:
diff changeset
   233
      case Bytecodes::_athrow      :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
        cur_block->set_may_throw();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
        // fall-through
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
      case Bytecodes::_ret         :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
      case Bytecodes::_ireturn     :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      case Bytecodes::_lreturn     :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
      case Bytecodes::_freturn     :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
      case Bytecodes::_dreturn     :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
      case Bytecodes::_areturn     :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
      case Bytecodes::_return      :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
        cur_block->set_control_bci(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
        if (s.next_bci() < limit_bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   245
          (void) make_block_at(s.next_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   246
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
        break;
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 38658
diff changeset
   248
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 38658
diff changeset
   249
      default:
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 38658
diff changeset
   250
        break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   253
  //  End the last block
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  cur_block->set_limit_bci(limit_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
ciMethodBlocks::ciMethodBlocks(Arena *arena, ciMethod *meth): _method(meth),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
                          _arena(arena), _num_blocks(0), _code_size(meth->code_size()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  int block_estimate = _code_size / 8;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
6180
53c1bf468c81 6973963: SEGV in ciBlock::start_bci() with EA
kvn
parents: 5547
diff changeset
   261
  _blocks =  new(_arena) GrowableArray<ciBlock *>(_arena, block_estimate, 0, NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
  int b2bsize = _code_size * sizeof(ciBlock **);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
  _bci_to_block = (ciBlock **) arena->Amalloc(b2bsize);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
  Copy::zero_to_words((HeapWord*) _bci_to_block, b2bsize / sizeof(HeapWord));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
489c9b5090e2 Initial load
duke
parents:
diff changeset
   266
  // create initial block covering the entire method
1399
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
   267
  ciBlock *b = new(arena) ciBlock(_method, _num_blocks++, 0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
  _blocks->append(b);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
  _bci_to_block[0] = b;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
  // create blocks for exception handlers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
  if (meth->has_exception_handlers()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
    for(ciExceptionHandlerStream str(meth); !str.is_done(); str.next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
      ciExceptionHandler* handler = str.handler();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
      ciBlock *eb = make_block_at(handler->handler_bci());
199
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   276
      //
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   277
      // Several exception handlers can have the same handler_bci:
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   278
      //
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   279
      //  try {
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   280
      //    if (a.foo(b) < 0) {
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   281
      //      return a.error();
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   282
      //    }
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   283
      //    return CoderResult.UNDERFLOW;
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   284
      //  } finally {
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   285
      //      a.position(b);
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   286
      //  }
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   287
      //
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   288
      //  The try block above is divided into 2 exception blocks
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   289
      //  separated by 'areturn' bci.
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   290
      //
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
      int ex_start = handler->start();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
      int ex_end = handler->limit();
2258
020bda12ecd4 6772368: REGRESSION:tomcat crashed twice with JDK 7
kvn
parents: 1399
diff changeset
   293
      // ensure a block at the start of exception range and start of following code
020bda12ecd4 6772368: REGRESSION:tomcat crashed twice with JDK 7
kvn
parents: 1399
diff changeset
   294
      (void) make_block_at(ex_start);
020bda12ecd4 6772368: REGRESSION:tomcat crashed twice with JDK 7
kvn
parents: 1399
diff changeset
   295
      if (ex_end < _code_size)
020bda12ecd4 6772368: REGRESSION:tomcat crashed twice with JDK 7
kvn
parents: 1399
diff changeset
   296
        (void) make_block_at(ex_end);
020bda12ecd4 6772368: REGRESSION:tomcat crashed twice with JDK 7
kvn
parents: 1399
diff changeset
   297
199
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   298
      if (eb->is_handler()) {
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   299
        // Extend old handler exception range to cover additional range.
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   300
        int old_ex_start = eb->ex_start_bci();
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   301
        int old_ex_end   = eb->ex_limit_bci();
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   302
        if (ex_start > old_ex_start)
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   303
          ex_start = old_ex_start;
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   304
        if (ex_end < old_ex_end)
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   305
          ex_end = old_ex_end;
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   306
        eb->clear_exception_handler(); // Reset exception information
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   307
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
      eb->set_exception_range(ex_start, ex_end);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
  // scan the bytecodes and identify blocks
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
  do_analysis();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
  // mark blocks that have exception handlers
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
  if (meth->has_exception_handlers()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
    for(ciExceptionHandlerStream str(meth); !str.is_done(); str.next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   318
      ciExceptionHandler* handler = str.handler();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
      int ex_start = handler->start();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
      int ex_end = handler->limit();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
      int bci = ex_start;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
      while (bci < ex_end) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   324
        ciBlock *b = block_containing(bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
        b->set_has_handler();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
        bci = b->limit_bci();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
void ciMethodBlocks::clear_processed() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   333
  for (int i = 0; i < _blocks->length(); i++)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   334
    _blocks->at(i)->clear_processed();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   335
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   336
489c9b5090e2 Initial load
duke
parents:
diff changeset
   337
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
void ciMethodBlocks::dump() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   339
  tty->print("---- blocks for method: ");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
  _method->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
  tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
  for (int i = 0; i < _blocks->length(); i++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
    tty->print("  B%d: ", i); _blocks->at(i)->dump();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   347
1399
9648dfd4ce09 6384206: Phis which are later unneeded are impairing our ability to inline based on static types
never
parents: 958
diff changeset
   348
ciBlock::ciBlock(ciMethod *method, int index, int start_bci) :
51333
f6641fcf7b7e 8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents: 47216
diff changeset
   349
                         _idx(index), _start_bci(start_bci), _limit_bci(-1), _control_bci(fall_through_bci),
f6641fcf7b7e 8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents: 47216
diff changeset
   350
                         _flags(0), _ex_start_bci(-1), _ex_limit_bci(-1)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
#ifndef PRODUCT
51333
f6641fcf7b7e 8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents: 47216
diff changeset
   352
                         , _method(method)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
#endif
51333
f6641fcf7b7e 8208670: Compiler changes to allow enabling -Wreorder
tschatzl
parents: 47216
diff changeset
   354
{
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
void ciBlock::set_exception_range(int start_bci, int limit_bci)  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   358
   assert(limit_bci >= start_bci, "valid range");
199
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   359
   assert(!is_handler() && _ex_start_bci == -1 && _ex_limit_bci == -1, "must not be handler");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   360
   _ex_start_bci = start_bci;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   361
   _ex_limit_bci = limit_bci;
199
fb51d01039ff 6621084: ciMethodBlocks::split_block_at() is broken for methods with exception handler
kvn
parents: 1
diff changeset
   362
   set_handler();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   363
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   364
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
#ifndef PRODUCT
958
4c4709e8b7ee 6712835: Server compiler fails with assertion (loop_count < K,"infinite loop in PhaseIterGVN::transform")
never
parents: 670
diff changeset
   366
static const char *flagnames[] = {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
  "Processed",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
  "Handler",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  "MayThrow",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
  "Jsr",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
  "Ret",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
  "RetTarget",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
  "HasHandler",
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
void ciBlock::dump() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
  tty->print(" [%d .. %d), {", _start_bci, _limit_bci);
38658
34f9c45625d8 8140594: Various minor code improvements (compiler)
goetz
parents: 7913
diff changeset
   378
  for (int i = 0; i < 7; i++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
    if ((_flags & (1 << i)) != 0) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
      tty->print(" %s", flagnames[i]);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
  tty->print(" ]");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
  if (is_handler())
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
    tty->print(" handles(%d..%d)", _ex_start_bci, _ex_limit_bci);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
  tty->cr();
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
// ciBlock::print_on
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
void ciBlock::print_on(outputStream* st) const {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
  st->print_cr("--------------------------------------------------------");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
  st->print   ("ciBlock [%d - %d) control : ", start_bci(), limit_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
  if (control_bci() == fall_through_bci) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
    st->print_cr("%d:fall through", limit_bci());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   397
    st->print_cr("%d:%s", control_bci(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   398
        Bytecodes::name(method()->java_code_at_bci(control_bci())));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   400
489c9b5090e2 Initial load
duke
parents:
diff changeset
   401
  if (Verbose || WizardMode) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
    method()->print_codes_on(start_bci(), limit_bci(), st);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   403
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
#endif