hotspot/src/share/vm/opto/stringopts.cpp
author jcoomes
Tue, 22 Mar 2011 13:36:33 -0700
changeset 8728 3f1bcd33068e
parent 7714 f078cdefa674
child 8734 2dae3d363359
permissions -rw-r--r--
6962931: move interned strings out of the perm gen Reviewed-by: never, coleenp, ysr, jwilhelm
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     1
/*
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4891
diff changeset
     2
 * Copyright (c) 2009, 2010, Oracle and/or its affiliates. All rights reserved.
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     4
 *
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     7
 * published by the Free Software Foundation.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     8
 *
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    13
 * accompanied this code).
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    14
 *
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4891
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4891
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: 4891
diff changeset
    21
 * questions.
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    22
 *
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    23
 */
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    26
#include "compiler/compileLog.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    27
#include "opto/addnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    28
#include "opto/callGenerator.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    29
#include "opto/callnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    30
#include "opto/divnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    31
#include "opto/graphKit.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    32
#include "opto/idealKit.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    33
#include "opto/rootnode.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    34
#include "opto/runtime.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    35
#include "opto/stringopts.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6743
diff changeset
    36
#include "opto/subnode.hpp"
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    37
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    38
#define __ kit.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    39
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    40
class StringConcat : public ResourceObj {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    41
 private:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    42
  PhaseStringOpts*    _stringopts;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    43
  Node*               _string_alloc;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    44
  AllocateNode*       _begin;          // The allocation the begins the pattern
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    45
  CallStaticJavaNode* _end;            // The final call of the pattern.  Will either be
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    46
                                       // SB.toString or or String.<init>(SB.toString)
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    47
  bool                _multiple;       // indicates this is a fusion of two or more
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    48
                                       // separate StringBuilders
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    49
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    50
  Node*               _arguments;      // The list of arguments to be concatenated
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    51
  GrowableArray<int>  _mode;           // into a String along with a mode flag
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    52
                                       // indicating how to treat the value.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    53
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    54
  Node_List           _control;        // List of control nodes that will be deleted
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    55
  Node_List           _uncommon_traps; // Uncommon traps that needs to be rewritten
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    56
                                       // to restart at the initial JVMState.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    57
 public:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    58
  // Mode for converting arguments to Strings
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    59
  enum {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    60
    StringMode,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    61
    IntMode,
7714
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
    62
    CharMode,
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
    63
    StringNullCheckMode
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    64
  };
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    65
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    66
  StringConcat(PhaseStringOpts* stringopts, CallStaticJavaNode* end):
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    67
    _end(end),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    68
    _begin(NULL),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    69
    _multiple(false),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    70
    _string_alloc(NULL),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    71
    _stringopts(stringopts) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    72
    _arguments = new (_stringopts->C, 1) Node(1);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    73
    _arguments->del_req(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    74
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    75
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    76
  bool validate_control_flow();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    77
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    78
  void merge_add() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    79
#if 0
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    80
    // XXX This is place holder code for reusing an existing String
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    81
    // allocation but the logic for checking the state safety is
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    82
    // probably inadequate at the moment.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    83
    CallProjections endprojs;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    84
    sc->end()->extract_projections(&endprojs, false);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    85
    if (endprojs.resproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    86
      for (SimpleDUIterator i(endprojs.resproj); i.has_next(); i.next()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    87
        CallStaticJavaNode *use = i.get()->isa_CallStaticJava();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    88
        if (use != NULL && use->method() != NULL &&
6743
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
    89
            use->method()->intrinsic_id() == vmIntrinsics::_String_String &&
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    90
            use->in(TypeFunc::Parms + 1) == endprojs.resproj) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    91
          // Found useless new String(sb.toString()) so reuse the newly allocated String
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    92
          // when creating the result instead of allocating a new one.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    93
          sc->set_string_alloc(use->in(TypeFunc::Parms));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    94
          sc->set_end(use);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    95
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    96
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    97
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    98
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
    99
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   100
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   101
  StringConcat* merge(StringConcat* other, Node* arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   102
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   103
  void set_allocation(AllocateNode* alloc) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   104
    _begin = alloc;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   105
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   106
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   107
  void append(Node* value, int mode) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   108
    _arguments->add_req(value);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   109
    _mode.append(mode);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   110
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   111
  void push(Node* value, int mode) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   112
    _arguments->ins_req(0, value);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   113
    _mode.insert_before(0, mode);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   114
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   115
  void push_string(Node* value) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   116
    push(value, StringMode);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   117
  }
7714
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   118
  void push_string_null_check(Node* value) {
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   119
    push(value, StringNullCheckMode);
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   120
  }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   121
  void push_int(Node* value) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   122
    push(value, IntMode);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   123
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   124
  void push_char(Node* value) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   125
    push(value, CharMode);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   126
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   127
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   128
  Node* argument(int i) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   129
    return _arguments->in(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   130
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   131
  void set_argument(int i, Node* value) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   132
    _arguments->set_req(i, value);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   133
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   134
  int num_arguments() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   135
    return _mode.length();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   136
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   137
  int mode(int i) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   138
    return _mode.at(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   139
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   140
  void add_control(Node* ctrl) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   141
    assert(!_control.contains(ctrl), "only push once");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   142
    _control.push(ctrl);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   143
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   144
  CallStaticJavaNode* end() { return _end; }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   145
  AllocateNode* begin() { return _begin; }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   146
  Node* string_alloc() { return _string_alloc; }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   147
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   148
  void eliminate_unneeded_control();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   149
  void eliminate_initialize(InitializeNode* init);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   150
  void eliminate_call(CallNode* call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   151
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   152
  void maybe_log_transform() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   153
    CompileLog* log = _stringopts->C->log();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   154
    if (log != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   155
      log->head("replace_string_concat arguments='%d' string_alloc='%d' multiple='%d'",
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   156
                num_arguments(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   157
                _string_alloc != NULL,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   158
                _multiple);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   159
      JVMState* p = _begin->jvms();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   160
      while (p != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   161
        log->elem("jvms bci='%d' method='%d'", p->bci(), log->identify(p->method()));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   162
        p = p->caller();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   163
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   164
      log->tail("replace_string_concat");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   165
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   166
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   167
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   168
  void convert_uncommon_traps(GraphKit& kit, const JVMState* jvms) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   169
    for (uint u = 0; u < _uncommon_traps.size(); u++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   170
      Node* uct = _uncommon_traps.at(u);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   171
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   172
      // Build a new call using the jvms state of the allocate
6418
6671edbd230e 6978355: renaming for 6961697
twisti
parents: 5547
diff changeset
   173
      address call_addr = SharedRuntime::uncommon_trap_blob()->entry_point();
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   174
      const TypeFunc* call_type = OptoRuntime::uncommon_trap_Type();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   175
      int size = call_type->domain()->cnt();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   176
      const TypePtr* no_memory_effects = NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   177
      Compile* C = _stringopts->C;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   178
      CallStaticJavaNode* call = new (C, size) CallStaticJavaNode(call_type, call_addr, "uncommon_trap",
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   179
                                                                  jvms->bci(), no_memory_effects);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   180
      for (int e = 0; e < TypeFunc::Parms; e++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   181
        call->init_req(e, uct->in(e));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   182
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   183
      // Set the trap request to record intrinsic failure if this trap
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   184
      // is taken too many times.  Ideally we would handle then traps by
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   185
      // doing the original bookkeeping in the MDO so that if it caused
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   186
      // the code to be thrown out we could still recompile and use the
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   187
      // optimization.  Failing the uncommon traps doesn't really mean
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   188
      // that the optimization is a bad idea but there's no other way to
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   189
      // do the MDO updates currently.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   190
      int trap_request = Deoptimization::make_trap_request(Deoptimization::Reason_intrinsic,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   191
                                                           Deoptimization::Action_make_not_entrant);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   192
      call->init_req(TypeFunc::Parms, __ intcon(trap_request));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   193
      kit.add_safepoint_edges(call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   194
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   195
      _stringopts->gvn()->transform(call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   196
      C->gvn_replace_by(uct, call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   197
      uct->disconnect_inputs(NULL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   198
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   199
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   200
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   201
  void cleanup() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   202
    // disconnect the hook node
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   203
    _arguments->disconnect_inputs(NULL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   204
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   205
};
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   206
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   207
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   208
void StringConcat::eliminate_unneeded_control() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   209
  eliminate_initialize(begin()->initialization());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   210
  for (uint i = 0; i < _control.size(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   211
    Node* n = _control.at(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   212
    if (n->is_Call()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   213
      if (n != _end) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   214
        eliminate_call(n->as_Call());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   215
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   216
    } else if (n->is_IfTrue()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   217
      Compile* C = _stringopts->C;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   218
      C->gvn_replace_by(n, n->in(0)->in(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   219
      C->gvn_replace_by(n->in(0), C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   220
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   221
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   222
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   223
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   224
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   225
StringConcat* StringConcat::merge(StringConcat* other, Node* arg) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   226
  StringConcat* result = new StringConcat(_stringopts, _end);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   227
  for (uint x = 0; x < _control.size(); x++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   228
    Node* n = _control.at(x);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   229
    if (n->is_Call()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   230
      result->_control.push(n);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   231
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   232
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   233
  for (uint x = 0; x < other->_control.size(); x++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   234
    Node* n = other->_control.at(x);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   235
    if (n->is_Call()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   236
      result->_control.push(n);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   237
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   238
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   239
  assert(result->_control.contains(other->_end), "what?");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   240
  assert(result->_control.contains(_begin), "what?");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   241
  for (int x = 0; x < num_arguments(); x++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   242
    if (argument(x) == arg) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   243
      // replace the toString result with the all the arguments that
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   244
      // made up the other StringConcat
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   245
      for (int y = 0; y < other->num_arguments(); y++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   246
        result->append(other->argument(y), other->mode(y));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   247
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   248
    } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   249
      result->append(argument(x), mode(x));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   250
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   251
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   252
  result->set_allocation(other->_begin);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   253
  result->_multiple = true;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   254
  return result;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   255
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   256
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   257
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   258
void StringConcat::eliminate_call(CallNode* call) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   259
  Compile* C = _stringopts->C;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   260
  CallProjections projs;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   261
  call->extract_projections(&projs, false);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   262
  if (projs.fallthrough_catchproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   263
    C->gvn_replace_by(projs.fallthrough_catchproj, call->in(TypeFunc::Control));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   264
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   265
  if (projs.fallthrough_memproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   266
    C->gvn_replace_by(projs.fallthrough_memproj, call->in(TypeFunc::Memory));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   267
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   268
  if (projs.catchall_memproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   269
    C->gvn_replace_by(projs.catchall_memproj, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   270
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   271
  if (projs.fallthrough_ioproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   272
    C->gvn_replace_by(projs.fallthrough_ioproj, call->in(TypeFunc::I_O));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   273
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   274
  if (projs.catchall_ioproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   275
    C->gvn_replace_by(projs.catchall_ioproj, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   276
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   277
  if (projs.catchall_catchproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   278
    // EA can't cope with the partially collapsed graph this
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   279
    // creates so put it on the worklist to be collapsed later.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   280
    for (SimpleDUIterator i(projs.catchall_catchproj); i.has_next(); i.next()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   281
      Node *use = i.get();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   282
      int opc = use->Opcode();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   283
      if (opc == Op_CreateEx || opc == Op_Region) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   284
        _stringopts->record_dead_node(use);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   285
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   286
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   287
    C->gvn_replace_by(projs.catchall_catchproj, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   288
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   289
  if (projs.resproj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   290
    C->gvn_replace_by(projs.resproj, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   291
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   292
  C->gvn_replace_by(call, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   293
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   294
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   295
void StringConcat::eliminate_initialize(InitializeNode* init) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   296
  Compile* C = _stringopts->C;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   297
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   298
  // Eliminate Initialize node.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   299
  assert(init->outcnt() <= 2, "only a control and memory projection expected");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   300
  assert(init->req() <= InitializeNode::RawStores, "no pending inits");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   301
  Node *ctrl_proj = init->proj_out(TypeFunc::Control);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   302
  if (ctrl_proj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   303
    C->gvn_replace_by(ctrl_proj, init->in(TypeFunc::Control));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   304
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   305
  Node *mem_proj = init->proj_out(TypeFunc::Memory);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   306
  if (mem_proj != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   307
    Node *mem = init->in(TypeFunc::Memory);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   308
    C->gvn_replace_by(mem_proj, mem);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   309
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   310
  C->gvn_replace_by(init, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   311
  init->disconnect_inputs(NULL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   312
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   313
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   314
Node_List PhaseStringOpts::collect_toString_calls() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   315
  Node_List string_calls;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   316
  Node_List worklist;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   317
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   318
  _visited.Clear();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   319
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   320
  // Prime the worklist
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   321
  for (uint i = 1; i < C->root()->len(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   322
    Node* n = C->root()->in(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   323
    if (n != NULL && !_visited.test_set(n->_idx)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   324
      worklist.push(n);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   325
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   326
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   327
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   328
  while (worklist.size() > 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   329
    Node* ctrl = worklist.pop();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   330
    if (ctrl->is_CallStaticJava()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   331
      CallStaticJavaNode* csj = ctrl->as_CallStaticJava();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   332
      ciMethod* m = csj->method();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   333
      if (m != NULL &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   334
          (m->intrinsic_id() == vmIntrinsics::_StringBuffer_toString ||
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   335
           m->intrinsic_id() == vmIntrinsics::_StringBuilder_toString)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   336
        string_calls.push(csj);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   337
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   338
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   339
    if (ctrl->in(0) != NULL && !_visited.test_set(ctrl->in(0)->_idx)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   340
      worklist.push(ctrl->in(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   341
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   342
    if (ctrl->is_Region()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   343
      for (uint i = 1; i < ctrl->len(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   344
        if (ctrl->in(i) != NULL && !_visited.test_set(ctrl->in(i)->_idx)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   345
          worklist.push(ctrl->in(i));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   346
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   347
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   348
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   349
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   350
  return string_calls;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   351
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   352
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   353
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   354
StringConcat* PhaseStringOpts::build_candidate(CallStaticJavaNode* call) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   355
  ciMethod* m = call->method();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   356
  ciSymbol* string_sig;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   357
  ciSymbol* int_sig;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   358
  ciSymbol* char_sig;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   359
  if (m->holder() == C->env()->StringBuilder_klass()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   360
    string_sig = ciSymbol::String_StringBuilder_signature();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   361
    int_sig = ciSymbol::int_StringBuilder_signature();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   362
    char_sig = ciSymbol::char_StringBuilder_signature();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   363
  } else if (m->holder() == C->env()->StringBuffer_klass()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   364
    string_sig = ciSymbol::String_StringBuffer_signature();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   365
    int_sig = ciSymbol::int_StringBuffer_signature();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   366
    char_sig = ciSymbol::char_StringBuffer_signature();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   367
  } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   368
    return NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   369
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   370
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   371
  if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   372
    tty->print("considering toString call in ");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   373
    call->jvms()->dump_spec(tty); tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   374
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   375
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   376
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   377
  StringConcat* sc = new StringConcat(this, call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   378
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   379
  AllocateNode* alloc = NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   380
  InitializeNode* init = NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   381
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   382
  // possible opportunity for StringBuilder fusion
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   383
  CallStaticJavaNode* cnode = call;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   384
  while (cnode) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   385
    Node* recv = cnode->in(TypeFunc::Parms)->uncast();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   386
    if (recv->is_Proj()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   387
      recv = recv->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   388
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   389
    cnode = recv->isa_CallStaticJava();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   390
    if (cnode == NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   391
      alloc = recv->isa_Allocate();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   392
      if (alloc == NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   393
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   394
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   395
      // Find the constructor call
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   396
      Node* result = alloc->result_cast();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   397
      if (result == NULL || !result->is_CheckCastPP()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   398
        // strange looking allocation
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   399
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   400
        if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   401
          tty->print("giving up because allocation looks strange ");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   402
          alloc->jvms()->dump_spec(tty); tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   403
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   404
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   405
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   406
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   407
      Node* constructor = NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   408
      for (SimpleDUIterator i(result); i.has_next(); i.next()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   409
        CallStaticJavaNode *use = i.get()->isa_CallStaticJava();
6743
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   410
        if (use != NULL &&
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   411
            use->method() != NULL &&
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   412
            !use->method()->is_static() &&
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   413
            use->method()->name() == ciSymbol::object_initializer_name() &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   414
            use->method()->holder() == m->holder()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   415
          // Matched the constructor.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   416
          ciSymbol* sig = use->method()->signature()->as_symbol();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   417
          if (sig == ciSymbol::void_method_signature() ||
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   418
              sig == ciSymbol::int_void_signature() ||
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   419
              sig == ciSymbol::string_void_signature()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   420
            if (sig == ciSymbol::string_void_signature()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   421
              // StringBuilder(String) so pick this up as the first argument
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   422
              assert(use->in(TypeFunc::Parms + 1) != NULL, "what?");
7714
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   423
              const Type* type = _gvn->type(use->in(TypeFunc::Parms + 1));
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   424
              if (type == TypePtr::NULL_PTR) {
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   425
                // StringBuilder(null) throws exception.
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   426
#ifndef PRODUCT
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   427
                if (PrintOptimizeStringConcat) {
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   428
                  tty->print("giving up because StringBuilder(null) throws exception");
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   429
                  alloc->jvms()->dump_spec(tty); tty->cr();
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   430
                }
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   431
#endif
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   432
                return NULL;
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   433
              }
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   434
              // StringBuilder(str) argument needs null check.
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   435
              sc->push_string_null_check(use->in(TypeFunc::Parms + 1));
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   436
            }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   437
            // The int variant takes an initial size for the backing
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   438
            // array so just treat it like the void version.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   439
            constructor = use;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   440
          } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   441
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   442
            if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   443
              tty->print("unexpected constructor signature: %s", sig->as_utf8());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   444
            }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   445
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   446
          }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   447
          break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   448
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   449
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   450
      if (constructor == NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   451
        // couldn't find constructor
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   452
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   453
        if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   454
          tty->print("giving up because couldn't find constructor ");
7714
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
   455
          alloc->jvms()->dump_spec(tty); tty->cr();
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   456
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   457
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   458
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   459
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   460
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   461
      // Walked all the way back and found the constructor call so see
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   462
      // if this call converted into a direct string concatenation.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   463
      sc->add_control(call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   464
      sc->add_control(constructor);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   465
      sc->add_control(alloc);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   466
      sc->set_allocation(alloc);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   467
      if (sc->validate_control_flow()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   468
        return sc;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   469
      } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   470
        return NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   471
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   472
    } else if (cnode->method() == NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   473
      break;
6743
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   474
    } else if (!cnode->method()->is_static() &&
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   475
               cnode->method()->holder() == m->holder() &&
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   476
               cnode->method()->name() == ciSymbol::append_name() &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   477
               (cnode->method()->signature()->as_symbol() == string_sig ||
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   478
                cnode->method()->signature()->as_symbol() == char_sig ||
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   479
                cnode->method()->signature()->as_symbol() == int_sig)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   480
      sc->add_control(cnode);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   481
      Node* arg = cnode->in(TypeFunc::Parms + 1);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   482
      if (cnode->method()->signature()->as_symbol() == int_sig) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   483
        sc->push_int(arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   484
      } else if (cnode->method()->signature()->as_symbol() == char_sig) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   485
        sc->push_char(arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   486
      } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   487
        if (arg->is_Proj() && arg->in(0)->is_CallStaticJava()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   488
          CallStaticJavaNode* csj = arg->in(0)->as_CallStaticJava();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   489
          if (csj->method() != NULL &&
6743
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   490
              csj->method()->intrinsic_id() == vmIntrinsics::_Integer_toString) {
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   491
            sc->add_control(csj);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   492
            sc->push_int(csj->in(TypeFunc::Parms));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   493
            continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   494
          }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   495
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   496
        sc->push_string(arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   497
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   498
      continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   499
    } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   500
      // some unhandled signature
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   501
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   502
      if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   503
        tty->print("giving up because encountered unexpected signature ");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   504
        cnode->tf()->dump(); tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   505
        cnode->in(TypeFunc::Parms + 1)->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   506
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   507
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   508
      break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   509
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   510
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   511
  return NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   512
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   513
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   514
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   515
PhaseStringOpts::PhaseStringOpts(PhaseGVN* gvn, Unique_Node_List*):
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   516
  Phase(StringOpts),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   517
  _gvn(gvn),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   518
  _visited(Thread::current()->resource_area()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   519
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   520
  assert(OptimizeStringConcat, "shouldn't be here");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   521
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   522
  size_table_field = C->env()->Integer_klass()->get_field_by_name(ciSymbol::make("sizeTable"),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   523
                                                                  ciSymbol::make("[I"), true);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   524
  if (size_table_field == NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   525
    // Something wrong so give up.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   526
    assert(false, "why can't we find Integer.sizeTable?");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   527
    return;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   528
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   529
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   530
  // Collect the types needed to talk about the various slices of memory
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   531
  const TypeInstPtr* string_type = TypeInstPtr::make(TypePtr::NotNull, C->env()->String_klass(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   532
                                                     false, NULL, 0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   533
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   534
  const TypePtr* value_field_type = string_type->add_offset(java_lang_String::value_offset_in_bytes());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   535
  const TypePtr* offset_field_type = string_type->add_offset(java_lang_String::offset_offset_in_bytes());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   536
  const TypePtr* count_field_type = string_type->add_offset(java_lang_String::count_offset_in_bytes());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   537
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   538
  value_field_idx = C->get_alias_index(value_field_type);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   539
  count_field_idx = C->get_alias_index(count_field_type);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   540
  offset_field_idx = C->get_alias_index(offset_field_type);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   541
  char_adr_idx = C->get_alias_index(TypeAryPtr::CHARS);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   542
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   543
  // For each locally allocated StringBuffer see if the usages can be
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   544
  // collapsed into a single String construction.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   545
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   546
  // Run through the list of allocation looking for SB.toString to see
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   547
  // if it's possible to fuse the usage of the SB into a single String
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   548
  // construction.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   549
  GrowableArray<StringConcat*> concats;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   550
  Node_List toStrings = collect_toString_calls();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   551
  while (toStrings.size() > 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   552
    StringConcat* sc = build_candidate(toStrings.pop()->as_CallStaticJava());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   553
    if (sc != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   554
      concats.push(sc);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   555
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   556
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   557
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   558
  // try to coalesce separate concats
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   559
 restart:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   560
  for (int c = 0; c < concats.length(); c++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   561
    StringConcat* sc = concats.at(c);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   562
    for (int i = 0; i < sc->num_arguments(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   563
      Node* arg = sc->argument(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   564
      if (arg->is_Proj() && arg->in(0)->is_CallStaticJava()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   565
        CallStaticJavaNode* csj = arg->in(0)->as_CallStaticJava();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   566
        if (csj->method() != NULL &&
6743
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   567
            (csj->method()->intrinsic_id() == vmIntrinsics::_StringBuilder_toString ||
ef1795cd50a7 6986028: assert(_base == Int) failed: Not an Int in CmpINode::sub
never
parents: 6418
diff changeset
   568
             csj->method()->intrinsic_id() == vmIntrinsics::_StringBuffer_toString)) {
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   569
          for (int o = 0; o < concats.length(); o++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   570
            if (c == o) continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   571
            StringConcat* other = concats.at(o);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   572
            if (other->end() == csj) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   573
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   574
              if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   575
                tty->print_cr("considering stacked concats");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   576
              }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   577
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   578
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   579
              StringConcat* merged = sc->merge(other, arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   580
              if (merged->validate_control_flow()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   581
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   582
                if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   583
                  tty->print_cr("stacking would succeed");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   584
                }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   585
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   586
                if (c < o) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   587
                  concats.remove_at(o);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   588
                  concats.at_put(c, merged);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   589
                } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   590
                  concats.remove_at(c);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   591
                  concats.at_put(o, merged);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   592
                }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   593
                goto restart;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   594
              } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   595
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   596
                if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   597
                  tty->print_cr("stacking would fail");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   598
                }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   599
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   600
              }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   601
            }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   602
          }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   603
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   604
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   605
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   606
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   607
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   608
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   609
  for (int c = 0; c < concats.length(); c++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   610
    StringConcat* sc = concats.at(c);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   611
    replace_string_concat(sc);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   612
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   613
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   614
  remove_dead_nodes();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   615
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   616
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   617
void PhaseStringOpts::record_dead_node(Node* dead) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   618
  dead_worklist.push(dead);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   619
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   620
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   621
void PhaseStringOpts::remove_dead_nodes() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   622
  // Delete any dead nodes to make things clean enough that escape
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   623
  // analysis doesn't get unhappy.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   624
  while (dead_worklist.size() > 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   625
    Node* use = dead_worklist.pop();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   626
    int opc = use->Opcode();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   627
    switch (opc) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   628
      case Op_Region: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   629
        uint i = 1;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   630
        for (i = 1; i < use->req(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   631
          if (use->in(i) != C->top()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   632
            break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   633
          }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   634
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   635
        if (i >= use->req()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   636
          for (SimpleDUIterator i(use); i.has_next(); i.next()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   637
            Node* m = i.get();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   638
            if (m->is_Phi()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   639
              dead_worklist.push(m);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   640
            }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   641
          }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   642
          C->gvn_replace_by(use, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   643
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   644
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   645
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   646
      case Op_AddP:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   647
      case Op_CreateEx: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   648
        // Recurisvely clean up references to CreateEx so EA doesn't
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   649
        // get unhappy about the partially collapsed graph.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   650
        for (SimpleDUIterator i(use); i.has_next(); i.next()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   651
          Node* m = i.get();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   652
          if (m->is_AddP()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   653
            dead_worklist.push(m);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   654
          }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   655
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   656
        C->gvn_replace_by(use, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   657
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   658
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   659
      case Op_Phi:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   660
        if (use->in(0) == C->top()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   661
          C->gvn_replace_by(use, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   662
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   663
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   664
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   665
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   666
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   667
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   668
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   669
bool StringConcat::validate_control_flow() {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   670
  // We found all the calls and arguments now lets see if it's
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   671
  // safe to transform the graph as we would expect.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   672
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   673
  // Check to see if this resulted in too many uncommon traps previously
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   674
  if (Compile::current()->too_many_traps(_begin->jvms()->method(), _begin->jvms()->bci(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   675
                        Deoptimization::Reason_intrinsic)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   676
    return false;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   677
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   678
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   679
  // Walk backwards over the control flow from toString to the
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   680
  // allocation and make sure all the control flow is ok.  This
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   681
  // means it's either going to be eliminated once the calls are
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   682
  // removed or it can safely be transformed into an uncommon
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   683
  // trap.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   684
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   685
  int null_check_count = 0;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   686
  Unique_Node_List ctrl_path;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   687
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   688
  assert(_control.contains(_begin), "missing");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   689
  assert(_control.contains(_end), "missing");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   690
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   691
  // Collect the nodes that we know about and will eliminate into ctrl_path
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   692
  for (uint i = 0; i < _control.size(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   693
    // Push the call and it's control projection
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   694
    Node* n = _control.at(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   695
    if (n->is_Allocate()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   696
      AllocateNode* an = n->as_Allocate();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   697
      InitializeNode* init = an->initialization();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   698
      ctrl_path.push(init);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   699
      ctrl_path.push(init->as_Multi()->proj_out(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   700
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   701
    if (n->is_Call()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   702
      CallNode* cn = n->as_Call();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   703
      ctrl_path.push(cn);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   704
      ctrl_path.push(cn->proj_out(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   705
      ctrl_path.push(cn->proj_out(0)->unique_out());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   706
      ctrl_path.push(cn->proj_out(0)->unique_out()->as_Catch()->proj_out(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   707
    } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   708
      ShouldNotReachHere();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   709
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   710
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   711
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   712
  // Skip backwards through the control checking for unexpected contro flow
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   713
  Node* ptr = _end;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   714
  bool fail = false;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   715
  while (ptr != _begin) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   716
    if (ptr->is_Call() && ctrl_path.member(ptr)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   717
      ptr = ptr->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   718
    } else if (ptr->is_CatchProj() && ctrl_path.member(ptr)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   719
      ptr = ptr->in(0)->in(0)->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   720
      assert(ctrl_path.member(ptr), "should be a known piece of control");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   721
    } else if (ptr->is_IfTrue()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   722
      IfNode* iff = ptr->in(0)->as_If();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   723
      BoolNode* b = iff->in(1)->isa_Bool();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   724
      Node* cmp = b->in(1);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   725
      Node* v1 = cmp->in(1);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   726
      Node* v2 = cmp->in(2);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   727
      Node* otherproj = iff->proj_out(1 - ptr->as_Proj()->_con);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   728
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   729
      // Null check of the return of append which can simply be eliminated
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   730
      if (b->_test._test == BoolTest::ne &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   731
          v2->bottom_type() == TypePtr::NULL_PTR &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   732
          v1->is_Proj() && ctrl_path.member(v1->in(0))) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   733
        // NULL check of the return value of the append
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   734
        null_check_count++;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   735
        if (otherproj->outcnt() == 1) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   736
          CallStaticJavaNode* call = otherproj->unique_out()->isa_CallStaticJava();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   737
          if (call != NULL && call->_name != NULL && strcmp(call->_name, "uncommon_trap") == 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   738
            ctrl_path.push(call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   739
          }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   740
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   741
        _control.push(ptr);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   742
        ptr = ptr->in(0)->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   743
        continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   744
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   745
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   746
      // A test which leads to an uncommon trap which should be safe.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   747
      // Later this trap will be converted into a trap that restarts
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   748
      // at the beginning.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   749
      if (otherproj->outcnt() == 1) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   750
        CallStaticJavaNode* call = otherproj->unique_out()->isa_CallStaticJava();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   751
        if (call != NULL && call->_name != NULL && strcmp(call->_name, "uncommon_trap") == 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   752
          // control flow leads to uct so should be ok
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   753
          _uncommon_traps.push(call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   754
          ctrl_path.push(call);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   755
          ptr = ptr->in(0)->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   756
          continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   757
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   758
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   759
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   760
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   761
      // Some unexpected control flow we don't know how to handle.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   762
      if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   763
        tty->print_cr("failing with unknown test");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   764
        b->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   765
        cmp->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   766
        v1->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   767
        v2->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   768
        tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   769
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   770
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   771
      break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   772
    } else if (ptr->is_Proj() && ptr->in(0)->is_Initialize()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   773
      ptr = ptr->in(0)->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   774
    } else if (ptr->is_Region()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   775
      Node* copy = ptr->as_Region()->is_copy();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   776
      if (copy != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   777
        ptr = copy;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   778
        continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   779
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   780
      if (ptr->req() == 3 &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   781
          ptr->in(1) != NULL && ptr->in(1)->is_Proj() &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   782
          ptr->in(2) != NULL && ptr->in(2)->is_Proj() &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   783
          ptr->in(1)->in(0) == ptr->in(2)->in(0) &&
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   784
          ptr->in(1)->in(0) != NULL && ptr->in(1)->in(0)->is_If()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   785
        // Simple diamond.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   786
        // XXX should check for possibly merging stores.  simple data merges are ok.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   787
        ptr = ptr->in(1)->in(0)->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   788
        continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   789
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   790
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   791
      if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   792
        tty->print_cr("fusion would fail for region");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   793
        _begin->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   794
        ptr->dump(2);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   795
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   796
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   797
      fail = true;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   798
      break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   799
    } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   800
      // other unknown control
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   801
      if (!fail) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   802
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   803
        if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   804
          tty->print_cr("fusion would fail for");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   805
          _begin->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   806
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   807
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   808
        fail = true;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   809
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   810
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   811
      if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   812
        ptr->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   813
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   814
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   815
      ptr = ptr->in(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   816
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   817
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   818
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   819
  if (PrintOptimizeStringConcat && fail) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   820
    tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   821
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   822
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   823
  if (fail) return !fail;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   824
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   825
  // Validate that all these results produced are contained within
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   826
  // this cluster of objects.  First collect all the results produced
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   827
  // by calls in the region.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   828
  _stringopts->_visited.Clear();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   829
  Node_List worklist;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   830
  Node* final_result = _end->proj_out(TypeFunc::Parms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   831
  for (uint i = 0; i < _control.size(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   832
    CallNode* cnode = _control.at(i)->isa_Call();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   833
    if (cnode != NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   834
      _stringopts->_visited.test_set(cnode->_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   835
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   836
    Node* result = cnode != NULL ? cnode->proj_out(TypeFunc::Parms) : NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   837
    if (result != NULL && result != final_result) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   838
      worklist.push(result);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   839
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   840
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   841
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   842
  Node* last_result = NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   843
  while (worklist.size() > 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   844
    Node* result = worklist.pop();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   845
    if (_stringopts->_visited.test_set(result->_idx))
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   846
      continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   847
    for (SimpleDUIterator i(result); i.has_next(); i.next()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   848
      Node *use = i.get();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   849
      if (ctrl_path.member(use)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   850
        // already checked this
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   851
        continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   852
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   853
      int opc = use->Opcode();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   854
      if (opc == Op_CmpP || opc == Op_Node) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   855
        ctrl_path.push(use);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   856
        continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   857
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   858
      if (opc == Op_CastPP || opc == Op_CheckCastPP) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   859
        for (SimpleDUIterator j(use); j.has_next(); j.next()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   860
          worklist.push(j.get());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   861
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   862
        worklist.push(use->in(1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   863
        ctrl_path.push(use);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   864
        continue;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   865
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   866
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   867
      if (PrintOptimizeStringConcat) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   868
        if (result != last_result) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   869
          last_result = result;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   870
          tty->print_cr("extra uses for result:");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   871
          last_result->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   872
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   873
        use->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   874
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   875
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   876
      fail = true;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   877
      break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   878
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   879
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   880
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   881
#ifndef PRODUCT
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   882
  if (PrintOptimizeStringConcat && !fail) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   883
    ttyLocker ttyl;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   884
    tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   885
    tty->print("fusion would succeed (%d %d) for ", null_check_count, _uncommon_traps.size());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   886
    _begin->jvms()->dump_spec(tty); tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   887
    for (int i = 0; i < num_arguments(); i++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   888
      argument(i)->dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   889
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   890
    _control.dump();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   891
    tty->cr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   892
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   893
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   894
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   895
  return !fail;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   896
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   897
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   898
Node* PhaseStringOpts::fetch_static_field(GraphKit& kit, ciField* field) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   899
  const TypeKlassPtr* klass_type = TypeKlassPtr::make(field->holder());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   900
  Node* klass_node = __ makecon(klass_type);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   901
  BasicType bt = field->layout_type();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   902
  ciType* field_klass = field->type();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   903
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   904
  const Type *type;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   905
  if( bt == T_OBJECT ) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   906
    if (!field->type()->is_loaded()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   907
      type = TypeInstPtr::BOTTOM;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   908
    } else if (field->is_constant()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   909
      // This can happen if the constant oop is non-perm.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   910
      ciObject* con = field->constant_value().as_object();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   911
      // Do not "join" in the previous type; it doesn't add value,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   912
      // and may yield a vacuous result if the field is of interface type.
8728
3f1bcd33068e 6962931: move interned strings out of the perm gen
jcoomes
parents: 7714
diff changeset
   913
      type = TypeOopPtr::make_from_constant(con, true)->isa_oopptr();
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   914
      assert(type != NULL, "field singleton type must be consistent");
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   915
    } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   916
      type = TypeOopPtr::make_from_klass(field_klass->as_klass());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   917
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   918
  } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   919
    type = Type::get_const_basic_type(bt);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   920
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   921
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   922
  return kit.make_load(NULL, kit.basic_plus_adr(klass_node, field->offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   923
                       type, T_OBJECT,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   924
                       C->get_alias_index(klass_type->add_offset(field->offset_in_bytes())));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   925
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   926
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   927
Node* PhaseStringOpts::int_stringSize(GraphKit& kit, Node* arg) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   928
  RegionNode *final_merge = new (C, 3) RegionNode(3);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   929
  kit.gvn().set_type(final_merge, Type::CONTROL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   930
  Node* final_size = new (C, 3) PhiNode(final_merge, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   931
  kit.gvn().set_type(final_size, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   932
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   933
  IfNode* iff = kit.create_and_map_if(kit.control(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   934
                                      __ Bool(__ CmpI(arg, __ intcon(0x80000000)), BoolTest::ne),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   935
                                      PROB_FAIR, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   936
  Node* is_min = __ IfFalse(iff);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   937
  final_merge->init_req(1, is_min);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   938
  final_size->init_req(1, __ intcon(11));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   939
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   940
  kit.set_control(__ IfTrue(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   941
  if (kit.stopped()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   942
    final_merge->init_req(2, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   943
    final_size->init_req(2, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   944
  } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   945
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   946
    // int size = (i < 0) ? stringSize(-i) + 1 : stringSize(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   947
    RegionNode *r = new (C, 3) RegionNode(3);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   948
    kit.gvn().set_type(r, Type::CONTROL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   949
    Node *phi = new (C, 3) PhiNode(r, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   950
    kit.gvn().set_type(phi, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   951
    Node *size = new (C, 3) PhiNode(r, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   952
    kit.gvn().set_type(size, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   953
    Node* chk = __ CmpI(arg, __ intcon(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   954
    Node* p = __ Bool(chk, BoolTest::lt);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   955
    IfNode* iff = kit.create_and_map_if(kit.control(), p, PROB_FAIR, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   956
    Node* lessthan = __ IfTrue(iff);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   957
    Node* greaterequal = __ IfFalse(iff);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   958
    r->init_req(1, lessthan);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   959
    phi->init_req(1, __ SubI(__ intcon(0), arg));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   960
    size->init_req(1, __ intcon(1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   961
    r->init_req(2, greaterequal);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   962
    phi->init_req(2, arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   963
    size->init_req(2, __ intcon(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   964
    kit.set_control(r);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   965
    C->record_for_igvn(r);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   966
    C->record_for_igvn(phi);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   967
    C->record_for_igvn(size);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   968
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   969
    // for (int i=0; ; i++)
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   970
    //   if (x <= sizeTable[i])
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   971
    //     return i+1;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   972
    RegionNode *loop = new (C, 3) RegionNode(3);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   973
    loop->init_req(1, kit.control());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   974
    kit.gvn().set_type(loop, Type::CONTROL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   975
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   976
    Node *index = new (C, 3) PhiNode(loop, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   977
    index->init_req(1, __ intcon(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   978
    kit.gvn().set_type(index, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   979
    kit.set_control(loop);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   980
    Node* sizeTable = fetch_static_field(kit, size_table_field);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   981
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   982
    Node* value = kit.load_array_element(NULL, sizeTable, index, TypeAryPtr::INTS);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   983
    C->record_for_igvn(value);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   984
    Node* limit = __ CmpI(phi, value);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   985
    Node* limitb = __ Bool(limit, BoolTest::le);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   986
    IfNode* iff2 = kit.create_and_map_if(kit.control(), limitb, PROB_MIN, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   987
    Node* lessEqual = __ IfTrue(iff2);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   988
    Node* greater = __ IfFalse(iff2);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   989
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   990
    loop->init_req(2, greater);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   991
    index->init_req(2, __ AddI(index, __ intcon(1)));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   992
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   993
    kit.set_control(lessEqual);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   994
    C->record_for_igvn(loop);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   995
    C->record_for_igvn(index);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   996
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   997
    final_merge->init_req(2, kit.control());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   998
    final_size->init_req(2, __ AddI(__ AddI(index, size), __ intcon(1)));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
   999
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1000
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1001
  kit.set_control(final_merge);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1002
  C->record_for_igvn(final_merge);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1003
  C->record_for_igvn(final_size);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1004
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1005
  return final_size;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1006
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1007
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1008
void PhaseStringOpts::int_getChars(GraphKit& kit, Node* arg, Node* char_array, Node* start, Node* end) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1009
  RegionNode *final_merge = new (C, 4) RegionNode(4);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1010
  kit.gvn().set_type(final_merge, Type::CONTROL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1011
  Node *final_mem = PhiNode::make(final_merge, kit.memory(char_adr_idx), Type::MEMORY, TypeAryPtr::CHARS);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1012
  kit.gvn().set_type(final_mem, Type::MEMORY);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1013
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1014
  // need to handle Integer.MIN_VALUE specially because negating doesn't make it positive
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1015
  {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1016
    // i == MIN_VALUE
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1017
    IfNode* iff = kit.create_and_map_if(kit.control(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1018
                                        __ Bool(__ CmpI(arg, __ intcon(0x80000000)), BoolTest::ne),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1019
                                        PROB_FAIR, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1020
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1021
    Node* old_mem = kit.memory(char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1022
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1023
    kit.set_control(__ IfFalse(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1024
    if (kit.stopped()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1025
      // Statically not equal to MIN_VALUE so this path is dead
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1026
      final_merge->init_req(3, kit.control());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1027
    } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1028
      copy_string(kit, __ makecon(TypeInstPtr::make(C->env()->the_min_jint_string())),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1029
                  char_array, start);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1030
      final_merge->init_req(3, kit.control());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1031
      final_mem->init_req(3, kit.memory(char_adr_idx));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1032
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1033
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1034
    kit.set_control(__ IfTrue(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1035
    kit.set_memory(old_mem, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1036
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1037
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1038
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1039
  // Simplified version of Integer.getChars
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1040
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1041
  // int q, r;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1042
  // int charPos = index;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1043
  Node* charPos = end;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1044
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1045
  // char sign = 0;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1046
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1047
  Node* i = arg;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1048
  Node* sign = __ intcon(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1049
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1050
  // if (i < 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1051
  //     sign = '-';
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1052
  //     i = -i;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1053
  // }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1054
  {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1055
    IfNode* iff = kit.create_and_map_if(kit.control(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1056
                                        __ Bool(__ CmpI(arg, __ intcon(0)), BoolTest::lt),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1057
                                        PROB_FAIR, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1058
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1059
    RegionNode *merge = new (C, 3) RegionNode(3);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1060
    kit.gvn().set_type(merge, Type::CONTROL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1061
    i = new (C, 3) PhiNode(merge, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1062
    kit.gvn().set_type(i, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1063
    sign = new (C, 3) PhiNode(merge, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1064
    kit.gvn().set_type(sign, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1065
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1066
    merge->init_req(1, __ IfTrue(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1067
    i->init_req(1, __ SubI(__ intcon(0), arg));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1068
    sign->init_req(1, __ intcon('-'));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1069
    merge->init_req(2, __ IfFalse(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1070
    i->init_req(2, arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1071
    sign->init_req(2, __ intcon(0));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1072
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1073
    kit.set_control(merge);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1074
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1075
    C->record_for_igvn(merge);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1076
    C->record_for_igvn(i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1077
    C->record_for_igvn(sign);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1078
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1079
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1080
  // for (;;) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1081
  //     q = i / 10;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1082
  //     r = i - ((q << 3) + (q << 1));  // r = i-(q*10) ...
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1083
  //     buf [--charPos] = digits [r];
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1084
  //     i = q;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1085
  //     if (i == 0) break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1086
  // }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1087
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1088
  {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1089
    RegionNode *head = new (C, 3) RegionNode(3);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1090
    head->init_req(1, kit.control());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1091
    kit.gvn().set_type(head, Type::CONTROL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1092
    Node *i_phi = new (C, 3) PhiNode(head, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1093
    i_phi->init_req(1, i);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1094
    kit.gvn().set_type(i_phi, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1095
    charPos = PhiNode::make(head, charPos);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1096
    kit.gvn().set_type(charPos, TypeInt::INT);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1097
    Node *mem = PhiNode::make(head, kit.memory(char_adr_idx), Type::MEMORY, TypeAryPtr::CHARS);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1098
    kit.gvn().set_type(mem, Type::MEMORY);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1099
    kit.set_control(head);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1100
    kit.set_memory(mem, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1101
4891
7c8755dd5bb2 6920293: OptimizeStringConcat causing core dumps
never
parents: 4450
diff changeset
  1102
    Node* q = __ DivI(NULL, i_phi, __ intcon(10));
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1103
    Node* r = __ SubI(i_phi, __ AddI(__ LShiftI(q, __ intcon(3)),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1104
                                     __ LShiftI(q, __ intcon(1))));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1105
    Node* m1 = __ SubI(charPos, __ intcon(1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1106
    Node* ch = __ AddI(r, __ intcon('0'));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1107
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1108
    Node* st = __ store_to_memory(kit.control(), kit.array_element_address(char_array, m1, T_CHAR),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1109
                                  ch, T_CHAR, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1110
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1111
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1112
    IfNode* iff = kit.create_and_map_if(head, __ Bool(__ CmpI(q, __ intcon(0)), BoolTest::ne),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1113
                                        PROB_FAIR, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1114
    Node* ne = __ IfTrue(iff);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1115
    Node* eq = __ IfFalse(iff);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1116
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1117
    head->init_req(2, ne);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1118
    mem->init_req(2, st);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1119
    i_phi->init_req(2, q);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1120
    charPos->init_req(2, m1);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1121
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1122
    charPos = m1;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1123
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1124
    kit.set_control(eq);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1125
    kit.set_memory(st, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1126
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1127
    C->record_for_igvn(head);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1128
    C->record_for_igvn(mem);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1129
    C->record_for_igvn(i_phi);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1130
    C->record_for_igvn(charPos);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1131
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1132
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1133
  {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1134
    // if (sign != 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1135
    //     buf [--charPos] = sign;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1136
    // }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1137
    IfNode* iff = kit.create_and_map_if(kit.control(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1138
                                        __ Bool(__ CmpI(sign, __ intcon(0)), BoolTest::ne),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1139
                                        PROB_FAIR, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1140
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1141
    final_merge->init_req(2, __ IfFalse(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1142
    final_mem->init_req(2, kit.memory(char_adr_idx));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1143
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1144
    kit.set_control(__ IfTrue(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1145
    if (kit.stopped()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1146
      final_merge->init_req(1, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1147
      final_mem->init_req(1, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1148
    } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1149
      Node* m1 = __ SubI(charPos, __ intcon(1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1150
      Node* st = __ store_to_memory(kit.control(), kit.array_element_address(char_array, m1, T_CHAR),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1151
                                    sign, T_CHAR, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1152
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1153
      final_merge->init_req(1, kit.control());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1154
      final_mem->init_req(1, st);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1155
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1156
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1157
    kit.set_control(final_merge);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1158
    kit.set_memory(final_mem, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1159
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1160
    C->record_for_igvn(final_merge);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1161
    C->record_for_igvn(final_mem);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1162
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1163
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1164
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1165
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1166
Node* PhaseStringOpts::copy_string(GraphKit& kit, Node* str, Node* char_array, Node* start) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1167
  Node* string = str;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1168
  Node* offset = kit.make_load(NULL,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1169
                               kit.basic_plus_adr(string, string, java_lang_String::offset_offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1170
                               TypeInt::INT, T_INT, offset_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1171
  Node* count = kit.make_load(NULL,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1172
                              kit.basic_plus_adr(string, string, java_lang_String::count_offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1173
                              TypeInt::INT, T_INT, count_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1174
  const TypeAryPtr*  value_type = TypeAryPtr::make(TypePtr::NotNull,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1175
                                                   TypeAry::make(TypeInt::CHAR,TypeInt::POS),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1176
                                                   ciTypeArrayKlass::make(T_CHAR), true, 0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1177
  Node* value = kit.make_load(NULL,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1178
                              kit.basic_plus_adr(string, string, java_lang_String::value_offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1179
                              value_type, T_OBJECT, value_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1180
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1181
  // copy the contents
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1182
  if (offset->is_Con() && count->is_Con() && value->is_Con() && count->get_int() < unroll_string_copy_length) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1183
    // For small constant strings just emit individual stores.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1184
    // A length of 6 seems like a good space/speed tradeof.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1185
    int c = count->get_int();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1186
    int o = offset->get_int();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1187
    const TypeOopPtr* t = kit.gvn().type(value)->isa_oopptr();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1188
    ciTypeArray* value_array = t->const_oop()->as_type_array();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1189
    for (int e = 0; e < c; e++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1190
      __ store_to_memory(kit.control(), kit.array_element_address(char_array, start, T_CHAR),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1191
                         __ intcon(value_array->char_at(o + e)), T_CHAR, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1192
      start = __ AddI(start, __ intcon(1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1193
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1194
  } else {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1195
    Node* src_ptr = kit.array_element_address(value, offset, T_CHAR);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1196
    Node* dst_ptr = kit.array_element_address(char_array, start, T_CHAR);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1197
    Node* c = count;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1198
    Node* extra = NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1199
#ifdef _LP64
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1200
    c = __ ConvI2L(c);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1201
    extra = C->top();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1202
#endif
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1203
    Node* call = kit.make_runtime_call(GraphKit::RC_LEAF|GraphKit::RC_NO_FP,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1204
                                       OptoRuntime::fast_arraycopy_Type(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1205
                                       CAST_FROM_FN_PTR(address, StubRoutines::jshort_disjoint_arraycopy()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1206
                                       "jshort_disjoint_arraycopy", TypeAryPtr::CHARS,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1207
                                       src_ptr, dst_ptr, c, extra);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1208
    start = __ AddI(start, count);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1209
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1210
  return start;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1211
}
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1212
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1213
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1214
void PhaseStringOpts::replace_string_concat(StringConcat* sc) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1215
  // Log a little info about the transformation
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1216
  sc->maybe_log_transform();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1217
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1218
  // pull the JVMState of the allocation into a SafePointNode to serve as
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1219
  // as a shim for the insertion of the new code.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1220
  JVMState* jvms     = sc->begin()->jvms()->clone_shallow(C);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1221
  uint size = sc->begin()->req();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1222
  SafePointNode* map = new (C, size) SafePointNode(size, jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1223
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1224
  // copy the control and memory state from the final call into our
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1225
  // new starting state.  This allows any preceeding tests to feed
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1226
  // into the new section of code.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1227
  for (uint i1 = 0; i1 < TypeFunc::Parms; i1++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1228
    map->init_req(i1, sc->end()->in(i1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1229
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1230
  // blow away old allocation arguments
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1231
  for (uint i1 = TypeFunc::Parms; i1 < jvms->debug_start(); i1++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1232
    map->init_req(i1, C->top());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1233
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1234
  // Copy the rest of the inputs for the JVMState
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1235
  for (uint i1 = jvms->debug_start(); i1 < sc->begin()->req(); i1++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1236
    map->init_req(i1, sc->begin()->in(i1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1237
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1238
  // Make sure the memory state is a MergeMem for parsing.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1239
  if (!map->in(TypeFunc::Memory)->is_MergeMem()) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1240
    map->set_req(TypeFunc::Memory, MergeMemNode::make(C, map->in(TypeFunc::Memory)));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1241
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1242
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1243
  jvms->set_map(map);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1244
  map->ensure_stack(jvms, jvms->method()->max_stack());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1245
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1246
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1247
  // disconnect all the old StringBuilder calls from the graph
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1248
  sc->eliminate_unneeded_control();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1249
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1250
  // At this point all the old work has been completely removed from
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1251
  // the graph and the saved JVMState exists at the point where the
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1252
  // final toString call used to be.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1253
  GraphKit kit(jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1254
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1255
  // There may be uncommon traps which are still using the
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1256
  // intermediate states and these need to be rewritten to point at
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1257
  // the JVMState at the beginning of the transformation.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1258
  sc->convert_uncommon_traps(kit, jvms);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1259
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1260
  // Now insert the logic to compute the size of the string followed
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1261
  // by all the logic to construct array and resulting string.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1262
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1263
  Node* null_string = __ makecon(TypeInstPtr::make(C->env()->the_null_string()));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1264
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1265
  // Create a region for the overflow checks to merge into.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1266
  int args = MAX2(sc->num_arguments(), 1);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1267
  RegionNode* overflow = new (C, args) RegionNode(args);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1268
  kit.gvn().set_type(overflow, Type::CONTROL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1269
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1270
  // Create a hook node to hold onto the individual sizes since they
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1271
  // are need for the copying phase.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1272
  Node* string_sizes = new (C, args) Node(args);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1273
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1274
  Node* length = __ intcon(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1275
  for (int argi = 0; argi < sc->num_arguments(); argi++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1276
    Node* arg = sc->argument(argi);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1277
    switch (sc->mode(argi)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1278
      case StringConcat::IntMode: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1279
        Node* string_size = int_stringSize(kit, arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1280
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1281
        // accumulate total
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1282
        length = __ AddI(length, string_size);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1283
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1284
        // Cache this value for the use by int_toString
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1285
        string_sizes->init_req(argi, string_size);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1286
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1287
      }
7714
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1288
      case StringConcat::StringNullCheckMode: {
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1289
        const Type* type = kit.gvn().type(arg);
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1290
        assert(type != TypePtr::NULL_PTR, "missing check");
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1291
        if (!type->higher_equal(TypeInstPtr::NOTNULL)) {
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1292
          // Null check with uncommont trap since
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1293
          // StringBuilder(null) throws exception.
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1294
          // Use special uncommon trap instead of
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1295
          // calling normal do_null_check().
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1296
          Node* p = __ Bool(__ CmpP(arg, kit.null()), BoolTest::ne);
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1297
          IfNode* iff = kit.create_and_map_if(kit.control(), p, PROB_MIN, COUNT_UNKNOWN);
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1298
          overflow->add_req(__ IfFalse(iff));
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1299
          Node* notnull = __ IfTrue(iff);
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1300
          kit.set_control(notnull); // set control for the cast_not_null
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1301
          arg = kit.cast_not_null(arg, false);
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1302
          sc->set_argument(argi, arg);
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1303
        }
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1304
        assert(kit.gvn().type(arg)->higher_equal(TypeInstPtr::NOTNULL), "sanity");
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1305
        // Fallthrough to add string length.
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1306
      }
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1307
      case StringConcat::StringMode: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1308
        const Type* type = kit.gvn().type(arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1309
        if (type == TypePtr::NULL_PTR) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1310
          // replace the argument with the null checked version
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1311
          arg = null_string;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1312
          sc->set_argument(argi, arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1313
        } else if (!type->higher_equal(TypeInstPtr::NOTNULL)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1314
          // s = s != null ? s : "null";
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1315
          // length = length + (s.count - s.offset);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1316
          RegionNode *r = new (C, 3) RegionNode(3);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1317
          kit.gvn().set_type(r, Type::CONTROL);
4891
7c8755dd5bb2 6920293: OptimizeStringConcat causing core dumps
never
parents: 4450
diff changeset
  1318
          Node *phi = new (C, 3) PhiNode(r, type);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1319
          kit.gvn().set_type(phi, phi->bottom_type());
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1320
          Node* p = __ Bool(__ CmpP(arg, kit.null()), BoolTest::ne);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1321
          IfNode* iff = kit.create_and_map_if(kit.control(), p, PROB_MIN, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1322
          Node* notnull = __ IfTrue(iff);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1323
          Node* isnull =  __ IfFalse(iff);
4891
7c8755dd5bb2 6920293: OptimizeStringConcat causing core dumps
never
parents: 4450
diff changeset
  1324
          kit.set_control(notnull); // set control for the cast_not_null
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1325
          r->init_req(1, notnull);
4891
7c8755dd5bb2 6920293: OptimizeStringConcat causing core dumps
never
parents: 4450
diff changeset
  1326
          phi->init_req(1, kit.cast_not_null(arg, false));
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1327
          r->init_req(2, isnull);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1328
          phi->init_req(2, null_string);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1329
          kit.set_control(r);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1330
          C->record_for_igvn(r);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1331
          C->record_for_igvn(phi);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1332
          // replace the argument with the null checked version
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1333
          arg = phi;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1334
          sc->set_argument(argi, arg);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1335
        }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1336
        //         Node* offset = kit.make_load(NULL, kit.basic_plus_adr(arg, arg, offset_offset),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1337
        //                                      TypeInt::INT, T_INT, offset_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1338
        Node* count = kit.make_load(NULL, kit.basic_plus_adr(arg, arg, java_lang_String::count_offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1339
                                    TypeInt::INT, T_INT, count_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1340
        length = __ AddI(length, count);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1341
        string_sizes->init_req(argi, NULL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1342
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1343
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1344
      case StringConcat::CharMode: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1345
        // one character only
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1346
        length = __ AddI(length, __ intcon(1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1347
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1348
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1349
      default:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1350
        ShouldNotReachHere();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1351
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1352
    if (argi > 0) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1353
      // Check that the sum hasn't overflowed
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1354
      IfNode* iff = kit.create_and_map_if(kit.control(),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1355
                                          __ Bool(__ CmpI(length, __ intcon(0)), BoolTest::lt),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1356
                                          PROB_MIN, COUNT_UNKNOWN);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1357
      kit.set_control(__ IfFalse(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1358
      overflow->set_req(argi, __ IfTrue(iff));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1359
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1360
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1361
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1362
  {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1363
    // Hook
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1364
    PreserveJVMState pjvms(&kit);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1365
    kit.set_control(overflow);
7714
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1366
    C->record_for_igvn(overflow);
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1367
    kit.uncommon_trap(Deoptimization::Reason_intrinsic,
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1368
                      Deoptimization::Action_make_not_entrant);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1369
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1370
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1371
  // length now contains the number of characters needed for the
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1372
  // char[] so create a new AllocateArray for the char[]
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1373
  Node* char_array = NULL;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1374
  {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1375
    PreserveReexecuteState preexecs(&kit);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1376
    // The original jvms is for an allocation of either a String or
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1377
    // StringBuffer so no stack adjustment is necessary for proper
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1378
    // reexecution.  If we deoptimize in the slow path the bytecode
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1379
    // will be reexecuted and the char[] allocation will be thrown away.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1380
    kit.jvms()->set_should_reexecute(true);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1381
    char_array = kit.new_array(__ makecon(TypeKlassPtr::make(ciTypeArrayKlass::make(T_CHAR))),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1382
                               length, 1);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1383
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1384
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1385
  // Mark the allocation so that zeroing is skipped since the code
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1386
  // below will overwrite the entire array
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1387
  AllocateArrayNode* char_alloc = AllocateArrayNode::Ideal_array_allocation(char_array, _gvn);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1388
  char_alloc->maybe_set_complete(_gvn);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1389
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1390
  // Now copy the string representations into the final char[]
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1391
  Node* start = __ intcon(0);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1392
  for (int argi = 0; argi < sc->num_arguments(); argi++) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1393
    Node* arg = sc->argument(argi);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1394
    switch (sc->mode(argi)) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1395
      case StringConcat::IntMode: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1396
        Node* end = __ AddI(start, string_sizes->in(argi));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1397
        // getChars words backwards so pass the ending point as well as the start
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1398
        int_getChars(kit, arg, char_array, start, end);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1399
        start = end;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1400
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1401
      }
7714
f078cdefa674 7009359: HS with -XX:+AggressiveOpts optimize new StringBuffer(null) so it does not throw NPE as expected
kvn
parents: 7397
diff changeset
  1402
      case StringConcat::StringNullCheckMode:
4450
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1403
      case StringConcat::StringMode: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1404
        start = copy_string(kit, arg, char_array, start);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1405
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1406
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1407
      case StringConcat::CharMode: {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1408
        __ store_to_memory(kit.control(), kit.array_element_address(char_array, start, T_CHAR),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1409
                           arg, T_CHAR, char_adr_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1410
        start = __ AddI(start, __ intcon(1));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1411
        break;
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1412
      }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1413
      default:
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1414
        ShouldNotReachHere();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1415
    }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1416
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1417
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1418
  // If we're not reusing an existing String allocation then allocate one here.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1419
  Node* result = sc->string_alloc();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1420
  if (result == NULL) {
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1421
    PreserveReexecuteState preexecs(&kit);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1422
    // The original jvms is for an allocation of either a String or
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1423
    // StringBuffer so no stack adjustment is necessary for proper
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1424
    // reexecution.
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1425
    kit.jvms()->set_should_reexecute(true);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1426
    result = kit.new_instance(__ makecon(TypeKlassPtr::make(C->env()->String_klass())));
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1427
  }
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1428
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1429
  // Intialize the string
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1430
  kit.store_to_memory(kit.control(), kit.basic_plus_adr(result, java_lang_String::offset_offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1431
                      __ intcon(0), T_INT, offset_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1432
  kit.store_to_memory(kit.control(), kit.basic_plus_adr(result, java_lang_String::count_offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1433
                      length, T_INT, count_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1434
  kit.store_to_memory(kit.control(), kit.basic_plus_adr(result, java_lang_String::value_offset_in_bytes()),
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1435
                      char_array, T_OBJECT, value_field_idx);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1436
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1437
  // hook up the outgoing control and result
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1438
  kit.replace_call(sc->end(), result);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1439
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1440
  // Unhook any hook nodes
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1441
  string_sizes->disconnect_inputs(NULL);
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1442
  sc->cleanup();
6d700b859b3e 6892658: C2 should optimize some stringbuilder patterns
never
parents:
diff changeset
  1443
}