src/java.base/share/classes/java/lang/invoke/StringConcatFactory.java
author redestad
Wed, 17 Apr 2019 00:06:38 +0200
changeset 54550 5fa7fbddfe9d
parent 52497 34510f65fb58
child 54652 1cb0306f16d1
permissions -rw-r--r--
8222484: Specialize generation of simple String concatenation expressions Reviewed-by: jrose, jlaskey
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     1
/*
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
     2
 * Copyright (c) 2015, 2019, Oracle and/or its affiliates. All rights reserved.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     4
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    10
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    15
 * accompanied this code).
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    16
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    20
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    23
 * questions.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    24
 */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    25
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    26
package java.lang.invoke;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    27
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    28
import jdk.internal.misc.Unsafe;
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
    29
import jdk.internal.misc.VM;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    30
import jdk.internal.org.objectweb.asm.ClassWriter;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    31
import jdk.internal.org.objectweb.asm.Label;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    32
import jdk.internal.org.objectweb.asm.MethodVisitor;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    33
import jdk.internal.org.objectweb.asm.Opcodes;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    34
import jdk.internal.vm.annotation.ForceInline;
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    35
import sun.invoke.util.Wrapper;
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    36
import sun.security.action.GetPropertyAction;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    37
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    38
import java.lang.invoke.MethodHandles.Lookup;
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    39
import java.util.ArrayList;
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    40
import java.util.Arrays;
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    41
import java.util.List;
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    42
import java.util.Objects;
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
    43
import java.util.Properties;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    44
import java.util.concurrent.ConcurrentHashMap;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    45
import java.util.concurrent.ConcurrentMap;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    46
import java.util.function.Function;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    47
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    48
import static jdk.internal.org.objectweb.asm.Opcodes.*;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    49
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    50
/**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    51
 * <p>Methods to facilitate the creation of String concatenation methods, that
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    52
 * can be used to efficiently concatenate a known number of arguments of known
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    53
 * types, possibly after type adaptation and partial evaluation of arguments.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    54
 * These methods are typically used as <em>bootstrap methods</em> for {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    55
 * invokedynamic} call sites, to support the <em>string concatenation</em>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    56
 * feature of the Java Programming Language.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    57
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    58
 * <p>Indirect access to the behavior specified by the provided {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    59
 * MethodHandle} proceeds in order through two phases:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    60
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    61
 * <ol>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    62
 *     <li><em>Linkage</em> occurs when the methods in this class are invoked.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    63
 * They take as arguments a method type describing the concatenated arguments
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    64
 * count and types, and optionally the String <em>recipe</em>, plus the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    65
 * constants that participate in the String concatenation. The details on
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    66
 * accepted recipe shapes are described further below. Linkage may involve
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    67
 * dynamically loading a new class that implements the expected concatenation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    68
 * behavior. The {@code CallSite} holds the {@code MethodHandle} pointing to the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    69
 * exact concatenation method. The concatenation methods may be shared among
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    70
 * different {@code CallSite}s, e.g. if linkage methods produce them as pure
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    71
 * functions.</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    72
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    73
 * <li><em>Invocation</em> occurs when a generated concatenation method is
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    74
 * invoked with the exact dynamic arguments. This may occur many times for a
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    75
 * single concatenation method. The method referenced by the behavior {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    76
 * MethodHandle} is invoked with the static arguments and any additional dynamic
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    77
 * arguments provided on invocation, as if by {@link MethodHandle#invoke(Object...)}.</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    78
 * </ol>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    79
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    80
 * <p> This class provides two forms of linkage methods: a simple version
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    81
 * ({@link #makeConcat(java.lang.invoke.MethodHandles.Lookup, String,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    82
 * MethodType)}) using only the dynamic arguments, and an advanced version
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    83
 * ({@link #makeConcatWithConstants(java.lang.invoke.MethodHandles.Lookup,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    84
 * String, MethodType, String, Object...)} using the advanced forms of capturing
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    85
 * the constant arguments. The advanced strategy can produce marginally better
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    86
 * invocation bytecode, at the expense of exploding the number of shapes of
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    87
 * string concatenation methods present at runtime, because those shapes would
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    88
 * include constant static arguments as well.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    89
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    90
 * @author Aleksey Shipilev
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    91
 * @author Remi Forax
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    92
 * @author Peter Levart
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    93
 *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    94
 * @apiNote
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    95
 * <p>There is a JVM limit (classfile structural constraint): no method
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    96
 * can call with more than 255 slots. This limits the number of static and
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    97
 * dynamic arguments one can pass to bootstrap method. Since there are potential
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
    98
 * concatenation strategies that use {@code MethodHandle} combinators, we need
36441
f40a48b83f57 8151063: Typo in java.lang.invoke.StringConcatFactory javadoc
rriggs
parents: 36001
diff changeset
    99
 * to reserve a few empty slots on the parameter lists to capture the
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   100
 * temporal results. This is why bootstrap methods in this factory do not accept
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   101
 * more than 200 argument slots. Users requiring more than 200 argument slots in
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   102
 * concatenation are expected to split the large concatenation in smaller
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   103
 * expressions.
35401
f57a51dbcc50 8148730: Add @since tags in new String concat APIs
shade
parents: 35388
diff changeset
   104
 *
f57a51dbcc50 8148730: Add @since tags in new String concat APIs
shade
parents: 35388
diff changeset
   105
 * @since 9
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   106
 */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   107
public final class StringConcatFactory {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   108
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   109
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   110
     * Tag used to demarcate an ordinary argument.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   111
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   112
    private static final char TAG_ARG = '\u0001';
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   113
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   114
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   115
     * Tag used to demarcate a constant.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   116
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   117
    private static final char TAG_CONST = '\u0002';
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   118
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   119
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   120
     * Maximum number of argument slots in String Concat call.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   121
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   122
     * While the maximum number of argument slots that indy call can handle is 253,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   123
     * we do not use all those slots, to let the strategies with MethodHandle
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   124
     * combinators to use some arguments.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   125
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   126
    private static final int MAX_INDY_CONCAT_ARG_SLOTS = 200;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   127
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   128
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   129
     * Concatenation strategy to use. See {@link Strategy} for possible options.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   130
     * This option is controllable with -Djava.lang.invoke.stringConcat JDK option.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   131
     */
37684
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   132
    private static Strategy STRATEGY;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   133
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   134
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   135
     * Default strategy to use for concatenation.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   136
     */
38428
0079e7d659db 8148604: JEP 280, Switch to more optimal concatenation strategy
shade
parents: 37781
diff changeset
   137
    private static final Strategy DEFAULT_STRATEGY = Strategy.MH_INLINE_SIZED_EXACT;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   138
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   139
    private enum Strategy {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   140
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   141
         * Bytecode generator, calling into {@link java.lang.StringBuilder}.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   142
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   143
        BC_SB,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   144
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   145
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   146
         * Bytecode generator, calling into {@link java.lang.StringBuilder};
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   147
         * but trying to estimate the required storage.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   148
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   149
        BC_SB_SIZED,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   150
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   151
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   152
         * Bytecode generator, calling into {@link java.lang.StringBuilder};
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   153
         * but computing the required storage exactly.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   154
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   155
        BC_SB_SIZED_EXACT,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   156
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   157
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   158
         * MethodHandle-based generator, that in the end calls into {@link java.lang.StringBuilder}.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   159
         * This strategy also tries to estimate the required storage.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   160
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   161
        MH_SB_SIZED,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   162
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   163
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   164
         * MethodHandle-based generator, that in the end calls into {@link java.lang.StringBuilder}.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   165
         * This strategy also estimate the required storage exactly.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   166
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   167
        MH_SB_SIZED_EXACT,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   168
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   169
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   170
         * MethodHandle-based generator, that constructs its own byte[] array from
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   171
         * the arguments. It computes the required storage exactly.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   172
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   173
        MH_INLINE_SIZED_EXACT
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   174
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   175
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   176
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   177
     * Enables debugging: this may print debugging messages, perform additional (non-neutral for performance)
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   178
     * checks, etc.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   179
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   180
    private static final boolean DEBUG;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   181
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   182
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   183
     * Enables caching of strategy stubs. This may improve the linkage time by reusing the generated
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   184
     * code, at the expense of contaminating the profiles.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   185
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   186
    private static final boolean CACHE_ENABLE;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   187
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   188
    private static final ConcurrentMap<Key, MethodHandle> CACHE;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   189
35776
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   190
    /**
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   191
     * Dump generated classes to disk, for debugging purposes.
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   192
     */
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   193
    private static final ProxyClassesDumper DUMPER;
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   194
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   195
    private static final Class<?> STRING_HELPER;
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   196
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   197
    static {
37684
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   198
        // In case we need to double-back onto the StringConcatFactory during this
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   199
        // static initialization, make sure we have the reasonable defaults to complete
47478
438e0c9f2f17 8190382: fix small typographic errors in comments
smarks
parents: 47471
diff changeset
   200
        // the static initialization properly. After that, actual users would use
47471
304ef03403b1 8190323: "the the" typos
rriggs
parents: 47461
diff changeset
   201
        // the proper values we have read from the properties.
37684
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   202
        STRATEGY = DEFAULT_STRATEGY;
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   203
        // CACHE_ENABLE = false; // implied
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   204
        // CACHE = null;         // implied
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   205
        // DEBUG = false;        // implied
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   206
        // DUMPER = null;        // implied
38b7cb606a05 8155090: String concatenation fails with a custom SecurityManager that uses concatenation
shade
parents: 37593
diff changeset
   207
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   208
        try {
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   209
            STRING_HELPER = Class.forName("java.lang.StringConcatHelper");
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   210
        } catch (Throwable e) {
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   211
            throw new AssertionError(e);
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   212
        }
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   213
37593
824750ada3d6 8154231: Simplify access to System properties from JDK code
redestad
parents: 37332
diff changeset
   214
        final String strategy =
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   215
                VM.getSavedProperty("java.lang.invoke.stringConcat");
37593
824750ada3d6 8154231: Simplify access to System properties from JDK code
redestad
parents: 37332
diff changeset
   216
        CACHE_ENABLE = Boolean.parseBoolean(
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   217
                VM.getSavedProperty("java.lang.invoke.stringConcat.cache"));
37593
824750ada3d6 8154231: Simplify access to System properties from JDK code
redestad
parents: 37332
diff changeset
   218
        DEBUG = Boolean.parseBoolean(
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   219
                VM.getSavedProperty("java.lang.invoke.stringConcat.debug"));
37593
824750ada3d6 8154231: Simplify access to System properties from JDK code
redestad
parents: 37332
diff changeset
   220
        final String dumpPath =
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
   221
                VM.getSavedProperty("java.lang.invoke.stringConcat.dumpClasses");
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   222
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   223
        STRATEGY = (strategy == null) ? DEFAULT_STRATEGY : Strategy.valueOf(strategy);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   224
        CACHE = CACHE_ENABLE ? new ConcurrentHashMap<>() : null;
35776
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   225
        DUMPER = (dumpPath == null) ? null : ProxyClassesDumper.getInstance(dumpPath);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   226
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   227
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   228
    /**
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   229
     * Cache key is a composite of:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   230
     *   - class name, that lets to disambiguate stubs, to avoid excess sharing
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   231
     *   - method type, describing the dynamic arguments for concatenation
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   232
     *   - concat recipe, describing the constants and concat shape
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   233
     */
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   234
    private static final class Key {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   235
        final String className;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   236
        final MethodType mt;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   237
        final Recipe recipe;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   238
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   239
        public Key(String className, MethodType mt, Recipe recipe) {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   240
            this.className = className;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   241
            this.mt = mt;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   242
            this.recipe = recipe;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   243
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   244
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   245
        @Override
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   246
        public boolean equals(Object o) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   247
            if (this == o) return true;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   248
            if (o == null || getClass() != o.getClass()) return false;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   249
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   250
            Key key = (Key) o;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   251
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   252
            if (!className.equals(key.className)) return false;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   253
            if (!mt.equals(key.mt)) return false;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   254
            if (!recipe.equals(key.recipe)) return false;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   255
            return true;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   256
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   257
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   258
        @Override
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   259
        public int hashCode() {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   260
            int result = className.hashCode();
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   261
            result = 31 * result + mt.hashCode();
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   262
            result = 31 * result + recipe.hashCode();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   263
            return result;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   264
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   265
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   266
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   267
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   268
     * Parses the recipe string, and produces the traversable collection of
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   269
     * {@link java.lang.invoke.StringConcatFactory.RecipeElement}-s for generator
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   270
     * strategies. Notably, this class parses out the constants from the recipe
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   271
     * and from other static arguments.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   272
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   273
    private static final class Recipe {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   274
        private final List<RecipeElement> elements;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   275
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   276
        public Recipe(String src, Object[] constants) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   277
            List<RecipeElement> el = new ArrayList<>();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   278
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   279
            int constC = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   280
            int argC = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   281
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   282
            StringBuilder acc = new StringBuilder();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   283
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   284
            for (int i = 0; i < src.length(); i++) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   285
                char c = src.charAt(i);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   286
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   287
                if (c == TAG_CONST || c == TAG_ARG) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   288
                    // Detected a special tag, flush all accumulated characters
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   289
                    // as a constant first:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   290
                    if (acc.length() > 0) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   291
                        el.add(new RecipeElement(acc.toString()));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   292
                        acc.setLength(0);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   293
                    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   294
                    if (c == TAG_CONST) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   295
                        Object cnst = constants[constC++];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   296
                        el.add(new RecipeElement(cnst));
40256
c5e03eaf7ba2 8163370: Reduce number of classes loaded by common usage of java.lang.invoke
redestad
parents: 38868
diff changeset
   297
                    } else if (c == TAG_ARG) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   298
                        el.add(new RecipeElement(argC++));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   299
                    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   300
                } else {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   301
                    // Not a special character, this is a constant embedded into
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   302
                    // the recipe itself.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   303
                    acc.append(c);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   304
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   305
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   306
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   307
            // Flush the remaining characters as constant:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   308
            if (acc.length() > 0) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   309
                el.add(new RecipeElement(acc.toString()));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   310
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   311
36737
67b2804be5ab 8152951: Avoid calculating the reverse of StringConcatFactory$Recipe elements
redestad
parents: 36668
diff changeset
   312
            elements = el;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   313
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   314
36737
67b2804be5ab 8152951: Avoid calculating the reverse of StringConcatFactory$Recipe elements
redestad
parents: 36668
diff changeset
   315
        public List<RecipeElement> getElements() {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   316
            return elements;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   317
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   318
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   319
        @Override
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   320
        public boolean equals(Object o) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   321
            if (this == o) return true;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   322
            if (o == null || getClass() != o.getClass()) return false;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   323
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   324
            Recipe recipe = (Recipe) o;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   325
            return elements.equals(recipe.elements);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   326
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   327
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   328
        @Override
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   329
        public String toString() {
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   330
            return "Recipe{" +
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   331
                    "elements=" + elements +
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   332
                    '}';
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   333
        }
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   334
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   335
        @Override
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   336
        public int hashCode() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   337
            return elements.hashCode();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   338
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   339
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   340
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   341
    private static final class RecipeElement {
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
   342
        private final String value;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   343
        private final int argPos;
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   344
        private final char tag;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   345
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   346
        public RecipeElement(Object cnst) {
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
   347
            this.value = String.valueOf(Objects.requireNonNull(cnst));
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   348
            this.argPos = -1;
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   349
            this.tag = TAG_CONST;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   350
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   351
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   352
        public RecipeElement(int arg) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   353
            this.value = null;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   354
            this.argPos = arg;
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   355
            this.tag = TAG_ARG;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   356
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   357
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
   358
        public String getValue() {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   359
            assert (tag == TAG_CONST);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   360
            return value;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   361
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   362
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   363
        public int getArgPos() {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   364
            assert (tag == TAG_ARG);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   365
            return argPos;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   366
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   367
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   368
        public char getTag() {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   369
            return tag;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   370
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   371
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   372
        @Override
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   373
        public boolean equals(Object o) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   374
            if (this == o) return true;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   375
            if (o == null || getClass() != o.getClass()) return false;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   376
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   377
            RecipeElement that = (RecipeElement) o;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   378
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   379
            if (this.tag != that.tag) return false;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   380
            if (this.tag == TAG_CONST && (!value.equals(that.value))) return false;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   381
            if (this.tag == TAG_ARG && (argPos != that.argPos)) return false;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   382
            return true;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   383
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   384
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   385
        @Override
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   386
        public String toString() {
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   387
            return "RecipeElement{" +
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   388
                    "value='" + value + '\'' +
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   389
                    ", argPos=" + argPos +
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   390
                    ", tag=" + tag +
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   391
                    '}';
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   392
        }
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   393
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
   394
        @Override
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   395
        public int hashCode() {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   396
            return (int)tag;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   397
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   398
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   399
48930
b1a5b4ad7427 8198523: Refactor BootstrapMethodInvoker to further avoid runtime type checks
redestad
parents: 47985
diff changeset
   400
    // StringConcatFactory bootstrap methods are startup sensitive, and may be
b1a5b4ad7427 8198523: Refactor BootstrapMethodInvoker to further avoid runtime type checks
redestad
parents: 47985
diff changeset
   401
    // special cased in java.lang.invokeBootstrapMethodInvoker to ensure
b1a5b4ad7427 8198523: Refactor BootstrapMethodInvoker to further avoid runtime type checks
redestad
parents: 47985
diff changeset
   402
    // methods are invoked with exact type information to avoid generating
b1a5b4ad7427 8198523: Refactor BootstrapMethodInvoker to further avoid runtime type checks
redestad
parents: 47985
diff changeset
   403
    // code for runtime checks. Take care any changes or additions here are
b1a5b4ad7427 8198523: Refactor BootstrapMethodInvoker to further avoid runtime type checks
redestad
parents: 47985
diff changeset
   404
    // reflected there as appropriate.
b1a5b4ad7427 8198523: Refactor BootstrapMethodInvoker to further avoid runtime type checks
redestad
parents: 47985
diff changeset
   405
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   406
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   407
     * Facilitates the creation of optimized String concatenation methods, that
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   408
     * can be used to efficiently concatenate a known number of arguments of
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   409
     * known types, possibly after type adaptation and partial evaluation of
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   410
     * arguments. Typically used as a <em>bootstrap method</em> for {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   411
     * invokedynamic} call sites, to support the <em>string concatenation</em>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   412
     * feature of the Java Programming Language.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   413
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   414
     * <p>When the target of the {@code CallSite} returned from this method is
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   415
     * invoked, it returns the result of String concatenation, taking all
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   416
     * function arguments passed to the linkage method as inputs for
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   417
     * concatenation. The target signature is given by {@code concatType}.
47985
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   418
     * For a target accepting:
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   419
     * <ul>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   420
     *     <li>zero inputs, concatenation results in an empty string;</li>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   421
     *     <li>one input, concatenation results in the single
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   422
     *     input converted as per JLS 5.1.11 "String Conversion"; otherwise</li>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   423
     *     <li>two or more inputs, the inputs are concatenated as per
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   424
     *     requirements stated in JLS 15.18.1 "String Concatenation Operator +".
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   425
     *     The inputs are converted as per JLS 5.1.11 "String Conversion",
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   426
     *     and combined from left to right.</li>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   427
     * </ul>
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   428
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   429
     * <p>Assume the linkage arguments are as follows:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   430
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   431
     * <ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   432
     *     <li>{@code concatType}, describing the {@code CallSite} signature</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   433
     * </ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   434
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   435
     * <p>Then the following linkage invariants must hold:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   436
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   437
     * <ul>
47461
60a5d9c77c4c 8187089: StringConcatFactory.makeConcat & makeConcatWithConstants should throw StringConcatException if parameter slot count is over 200
mchung
parents: 47216
diff changeset
   438
     *     <li>The number of parameter slots in {@code concatType} is
60a5d9c77c4c 8187089: StringConcatFactory.makeConcat & makeConcatWithConstants should throw StringConcatException if parameter slot count is over 200
mchung
parents: 47216
diff changeset
   439
     *         less than or equal to 200</li>
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   440
     *     <li>The return type in {@code concatType} is assignable from {@link java.lang.String}</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   441
     * </ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   442
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   443
     * @param lookup   Represents a lookup context with the accessibility
47985
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   444
     *                 privileges of the caller. Specifically, the lookup
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   445
     *                 context must have
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   446
     *                 <a href="MethodHandles.Lookup.html#privacc">private access</a>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   447
     *                 privileges.
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   448
     *                 When used with {@code invokedynamic}, this is stacked
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   449
     *                 automatically by the VM.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   450
     * @param name     The name of the method to implement. This name is
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   451
     *                 arbitrary, and has no meaning for this linkage method.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   452
     *                 When used with {@code invokedynamic}, this is provided by
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   453
     *                 the {@code NameAndType} of the {@code InvokeDynamic}
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   454
     *                 structure and is stacked automatically by the VM.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   455
     * @param concatType The expected signature of the {@code CallSite}.  The
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   456
     *                   parameter types represent the types of concatenation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   457
     *                   arguments; the return type is always assignable from {@link
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   458
     *                   java.lang.String}.  When used with {@code invokedynamic},
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   459
     *                   this is provided by the {@code NameAndType} of the {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   460
     *                   InvokeDynamic} structure and is stacked automatically by
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   461
     *                   the VM.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   462
     * @return a CallSite whose target can be used to perform String
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   463
     * concatenation, with dynamic concatenation arguments described by the given
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   464
     * {@code concatType}.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   465
     * @throws StringConcatException If any of the linkage invariants described
47985
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   466
     *                               here are violated, or the lookup context
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   467
     *                               does not have private access privileges.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   468
     * @throws NullPointerException If any of the incoming arguments is null.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   469
     *                              This will never happen when a bootstrap method
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   470
     *                              is called with invokedynamic.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   471
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   472
     * @jls  5.1.11 String Conversion
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   473
     * @jls 15.18.1 String Concatenation Operator +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   474
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   475
    public static CallSite makeConcat(MethodHandles.Lookup lookup,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   476
                                      String name,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   477
                                      MethodType concatType) throws StringConcatException {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   478
        if (DEBUG) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   479
            System.out.println("StringConcatFactory " + STRATEGY + " is here for " + concatType);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   480
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   481
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   482
        return doStringConcat(lookup, name, concatType, true, null);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   483
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   484
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   485
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   486
     * Facilitates the creation of optimized String concatenation methods, that
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   487
     * can be used to efficiently concatenate a known number of arguments of
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   488
     * known types, possibly after type adaptation and partial evaluation of
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   489
     * arguments. Typically used as a <em>bootstrap method</em> for {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   490
     * invokedynamic} call sites, to support the <em>string concatenation</em>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   491
     * feature of the Java Programming Language.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   492
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   493
     * <p>When the target of the {@code CallSite} returned from this method is
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   494
     * invoked, it returns the result of String concatenation, taking all
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   495
     * function arguments and constants passed to the linkage method as inputs for
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   496
     * concatenation. The target signature is given by {@code concatType}, and
47985
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   497
     * does not include constants.
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   498
     * For a target accepting:
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   499
     * <ul>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   500
     *     <li>zero inputs, concatenation results in an empty string;</li>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   501
     *     <li>one input, concatenation results in the single
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   502
     *     input converted as per JLS 5.1.11 "String Conversion"; otherwise</li>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   503
     *     <li>two or more inputs, the inputs are concatenated as per
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   504
     *     requirements stated in JLS 15.18.1 "String Concatenation Operator +".
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   505
     *     The inputs are converted as per JLS 5.1.11 "String Conversion",
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   506
     *     and combined from left to right.</li>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   507
     * </ul>
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   508
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   509
     * <p>The concatenation <em>recipe</em> is a String description for the way to
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   510
     * construct a concatenated String from the arguments and constants. The
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   511
     * recipe is processed from left to right, and each character represents an
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   512
     * input to concatenation. Recipe characters mean:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   513
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   514
     * <ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   515
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   516
     *   <li><em>\1 (Unicode point 0001)</em>: an ordinary argument. This
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   517
     *   input is passed through dynamic argument, and is provided during the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   518
     *   concatenation method invocation. This input can be null.</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   519
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   520
     *   <li><em>\2 (Unicode point 0002):</em> a constant. This input passed
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   521
     *   through static bootstrap argument. This constant can be any value
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   522
     *   representable in constant pool. If necessary, the factory would call
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   523
     *   {@code toString} to perform a one-time String conversion.</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   524
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   525
     *   <li><em>Any other char value:</em> a single character constant.</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   526
     * </ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   527
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   528
     * <p>Assume the linkage arguments are as follows:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   529
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   530
     * <ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   531
     *   <li>{@code concatType}, describing the {@code CallSite} signature</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   532
     *   <li>{@code recipe}, describing the String recipe</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   533
     *   <li>{@code constants}, the vararg array of constants</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   534
     * </ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   535
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   536
     * <p>Then the following linkage invariants must hold:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   537
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   538
     * <ul>
47461
60a5d9c77c4c 8187089: StringConcatFactory.makeConcat & makeConcatWithConstants should throw StringConcatException if parameter slot count is over 200
mchung
parents: 47216
diff changeset
   539
     *   <li>The number of parameter slots in {@code concatType} is less than
60a5d9c77c4c 8187089: StringConcatFactory.makeConcat & makeConcatWithConstants should throw StringConcatException if parameter slot count is over 200
mchung
parents: 47216
diff changeset
   540
     *       or equal to 200</li>
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   541
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   542
     *   <li>The parameter count in {@code concatType} equals to number of \1 tags
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   543
     *   in {@code recipe}</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   544
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   545
     *   <li>The return type in {@code concatType} is assignable
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   546
     *   from {@link java.lang.String}, and matches the return type of the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   547
     *   returned {@link MethodHandle}</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   548
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   549
     *   <li>The number of elements in {@code constants} equals to number of \2
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   550
     *   tags in {@code recipe}</li>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   551
     * </ul>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   552
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   553
     * @param lookup    Represents a lookup context with the accessibility
47985
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   554
     *                  privileges of the caller. Specifically, the lookup
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   555
     *                  context must have
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   556
     *                  <a href="MethodHandles.Lookup.html#privacc">private access</a>
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   557
     *                  privileges.
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   558
     *                  When used with {@code invokedynamic}, this is stacked
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   559
     *                  automatically by the VM.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   560
     * @param name      The name of the method to implement. This name is
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   561
     *                  arbitrary, and has no meaning for this linkage method.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   562
     *                  When used with {@code invokedynamic}, this is provided
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   563
     *                  by the {@code NameAndType} of the {@code InvokeDynamic}
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   564
     *                  structure and is stacked automatically by the VM.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   565
     * @param concatType The expected signature of the {@code CallSite}.  The
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   566
     *                  parameter types represent the types of dynamic concatenation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   567
     *                  arguments; the return type is always assignable from {@link
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   568
     *                  java.lang.String}.  When used with {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   569
     *                  invokedynamic}, this is provided by the {@code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   570
     *                  NameAndType} of the {@code InvokeDynamic} structure and
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   571
     *                  is stacked automatically by the VM.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   572
     * @param recipe    Concatenation recipe, described above.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   573
     * @param constants A vararg parameter representing the constants passed to
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   574
     *                  the linkage method.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   575
     * @return a CallSite whose target can be used to perform String
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   576
     * concatenation, with dynamic concatenation arguments described by the given
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   577
     * {@code concatType}.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   578
     * @throws StringConcatException If any of the linkage invariants described
47985
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   579
     *                               here are violated, or the lookup context
80a058c0f993 8187237: Need to define the behaviour for 0 and 1 argument method type in StringConcatFactory.makeConcat
psandoz
parents: 47478
diff changeset
   580
     *                               does not have private access privileges.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   581
     * @throws NullPointerException If any of the incoming arguments is null, or
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   582
     *                              any constant in {@code recipe} is null.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   583
     *                              This will never happen when a bootstrap method
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   584
     *                              is called with invokedynamic.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   585
     * @apiNote Code generators have three distinct ways to process a constant
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   586
     * string operand S in a string concatenation expression.  First, S can be
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   587
     * materialized as a reference (using ldc) and passed as an ordinary argument
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   588
     * (recipe '\1'). Or, S can be stored in the constant pool and passed as a
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   589
     * constant (recipe '\2') . Finally, if S contains neither of the recipe
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   590
     * tag characters ('\1', '\2') then S can be interpolated into the recipe
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   591
     * itself, causing its characters to be inserted into the result.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   592
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   593
     * @jls  5.1.11 String Conversion
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   594
     * @jls 15.18.1 String Concatenation Operator +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   595
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   596
    public static CallSite makeConcatWithConstants(MethodHandles.Lookup lookup,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   597
                                                   String name,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   598
                                                   MethodType concatType,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   599
                                                   String recipe,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   600
                                                   Object... constants) throws StringConcatException {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   601
        if (DEBUG) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   602
            System.out.println("StringConcatFactory " + STRATEGY + " is here for " + concatType + ", {" + recipe + "}, " + Arrays.toString(constants));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   603
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   604
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   605
        return doStringConcat(lookup, name, concatType, false, recipe, constants);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   606
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   607
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   608
    private static CallSite doStringConcat(MethodHandles.Lookup lookup,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   609
                                           String name,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   610
                                           MethodType concatType,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   611
                                           boolean generateRecipe,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   612
                                           String recipe,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   613
                                           Object... constants) throws StringConcatException {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   614
        Objects.requireNonNull(lookup, "Lookup is null");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   615
        Objects.requireNonNull(name, "Name is null");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   616
        Objects.requireNonNull(concatType, "Concat type is null");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   617
        Objects.requireNonNull(constants, "Constants are null");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   618
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   619
        for (Object o : constants) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   620
            Objects.requireNonNull(o, "Cannot accept null constants");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   621
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   622
35776
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   623
        if ((lookup.lookupModes() & MethodHandles.Lookup.PRIVATE) == 0) {
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
   624
            throw new StringConcatException("Invalid caller: " +
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
   625
                    lookup.lookupClass().getName());
35776
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   626
        }
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
   627
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   628
        int cCount = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   629
        int oCount = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   630
        if (generateRecipe) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   631
            // Mock the recipe to reuse the concat generator code
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   632
            char[] value = new char[concatType.parameterCount()];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   633
            Arrays.fill(value, TAG_ARG);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   634
            recipe = new String(value);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   635
            oCount = concatType.parameterCount();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   636
        } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   637
            Objects.requireNonNull(recipe, "Recipe is null");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   638
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   639
            for (int i = 0; i < recipe.length(); i++) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   640
                char c = recipe.charAt(i);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   641
                if (c == TAG_CONST) cCount++;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   642
                if (c == TAG_ARG)   oCount++;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   643
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   644
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   645
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   646
        if (oCount != concatType.parameterCount()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   647
            throw new StringConcatException(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   648
                    "Mismatched number of concat arguments: recipe wants " +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   649
                            oCount +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   650
                            " arguments, but signature provides " +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   651
                            concatType.parameterCount());
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   652
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   653
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   654
        if (cCount != constants.length) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   655
            throw new StringConcatException(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   656
                    "Mismatched number of concat constants: recipe wants " +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   657
                            cCount +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   658
                            " constants, but only " +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   659
                            constants.length +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   660
                            " are passed");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   661
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   662
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   663
        if (!concatType.returnType().isAssignableFrom(String.class)) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   664
            throw new StringConcatException(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   665
                    "The return type should be compatible with String, but it is " +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   666
                            concatType.returnType());
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   667
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   668
47461
60a5d9c77c4c 8187089: StringConcatFactory.makeConcat & makeConcatWithConstants should throw StringConcatException if parameter slot count is over 200
mchung
parents: 47216
diff changeset
   669
        if (concatType.parameterSlotCount() > MAX_INDY_CONCAT_ARG_SLOTS) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   670
            throw new StringConcatException("Too many concat argument slots: " +
47461
60a5d9c77c4c 8187089: StringConcatFactory.makeConcat & makeConcatWithConstants should throw StringConcatException if parameter slot count is over 200
mchung
parents: 47216
diff changeset
   671
                    concatType.parameterSlotCount() +
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   672
                    ", can only accept " +
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   673
                    MAX_INDY_CONCAT_ARG_SLOTS);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   674
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   675
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   676
        String className = getClassName(lookup.lookupClass());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   677
        MethodType mt = adaptType(concatType);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   678
        Recipe rec = new Recipe(recipe, constants);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   679
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   680
        MethodHandle mh;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   681
        if (CACHE_ENABLE) {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   682
            Key key = new Key(className, mt, rec);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   683
            mh = CACHE.get(key);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   684
            if (mh == null) {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   685
                mh = generate(lookup, className, mt, rec);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   686
                CACHE.put(key, mh);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   687
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   688
        } else {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   689
            mh = generate(lookup, className, mt, rec);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   690
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   691
        return new ConstantCallSite(mh.asType(concatType));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   692
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   693
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   694
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   695
     * Adapt method type to an API we are going to use.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   696
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   697
     * This strips the concrete classes from the signatures, thus preventing
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   698
     * class leakage when we cache the concatenation stubs.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   699
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   700
     * @param args actual argument types
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   701
     * @return argument types the strategy is going to use
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   702
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   703
    private static MethodType adaptType(MethodType args) {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   704
        Class<?>[] ptypes = null;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   705
        for (int i = 0; i < args.parameterCount(); i++) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   706
            Class<?> ptype = args.parameterType(i);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   707
            if (!ptype.isPrimitive() &&
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   708
                    ptype != String.class &&
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   709
                    ptype != Object.class) { // truncate to Object
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   710
                if (ptypes == null) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   711
                    ptypes = args.parameterArray();
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   712
                }
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   713
                ptypes[i] = Object.class;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   714
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   715
            // else other primitives or String or Object (unchanged)
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   716
        }
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   717
        return (ptypes != null)
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   718
                ? MethodType.methodType(args.returnType(), ptypes)
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   719
                : args;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   720
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   721
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   722
    private static String getClassName(Class<?> hostClass) throws StringConcatException {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   723
        /*
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   724
          When cache is enabled, we want to cache as much as we can.
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   725
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   726
          However, there are two peculiarities:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   727
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   728
           a) The generated class should stay within the same package as the
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   729
              host class, to allow Unsafe.defineAnonymousClass access controls
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   730
              to work properly. JDK may choose to fail with IllegalAccessException
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   731
              when accessing a VM anonymous class with non-privileged callers,
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   732
              see JDK-8058575.
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   733
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   734
           b) If we mark the stub with some prefix, say, derived from the package
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   735
              name because of (a), we can technically use that stub in other packages.
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   736
              But the call stack traces would be extremely puzzling to unsuspecting users
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   737
              and profiling tools: whatever stub wins the race, would be linked in all
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   738
              similar callsites.
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   739
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   740
           Therefore, we set the class prefix to match the host class package, and use
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   741
           the prefix as the cache key too. This only affects BC_* strategies, and only when
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   742
           cache is enabled.
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   743
         */
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   744
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   745
        switch (STRATEGY) {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   746
            case BC_SB:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   747
            case BC_SB_SIZED:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   748
            case BC_SB_SIZED_EXACT: {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   749
                if (CACHE_ENABLE) {
36511
9d0388c6b336 8142968: Module System implementation
alanb
parents: 36441
diff changeset
   750
                    String pkgName = hostClass.getPackageName();
9d0388c6b336 8142968: Module System implementation
alanb
parents: 36441
diff changeset
   751
                    return (pkgName != null && !pkgName.isEmpty() ? pkgName.replace('.', '/') + "/" : "") + "Stubs$$StringConcat";
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   752
                } else {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   753
                    return hostClass.getName().replace('.', '/') + "$$StringConcat";
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   754
                }
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   755
            }
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   756
            case MH_SB_SIZED:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   757
            case MH_SB_SIZED_EXACT:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   758
            case MH_INLINE_SIZED_EXACT:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   759
                // MethodHandle strategies do not need a class name.
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   760
                return "";
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   761
            default:
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   762
                throw new StringConcatException("Concatenation strategy " + STRATEGY + " is not implemented");
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   763
        }
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   764
    }
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   765
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   766
    private static MethodHandle generate(Lookup lookup, String className, MethodType mt, Recipe recipe) throws StringConcatException {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   767
        try {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   768
            switch (STRATEGY) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   769
                case BC_SB:
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   770
                    return BytecodeStringBuilderStrategy.generate(lookup, className, mt, recipe, Mode.DEFAULT);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   771
                case BC_SB_SIZED:
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   772
                    return BytecodeStringBuilderStrategy.generate(lookup, className, mt, recipe, Mode.SIZED);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   773
                case BC_SB_SIZED_EXACT:
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   774
                    return BytecodeStringBuilderStrategy.generate(lookup, className, mt, recipe, Mode.SIZED_EXACT);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   775
                case MH_SB_SIZED:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   776
                    return MethodHandleStringBuilderStrategy.generate(mt, recipe, Mode.SIZED);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   777
                case MH_SB_SIZED_EXACT:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   778
                    return MethodHandleStringBuilderStrategy.generate(mt, recipe, Mode.SIZED_EXACT);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   779
                case MH_INLINE_SIZED_EXACT:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   780
                    return MethodHandleInlineCopyStrategy.generate(mt, recipe);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   781
                default:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   782
                    throw new StringConcatException("Concatenation strategy " + STRATEGY + " is not implemented");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   783
            }
42105
d04ede5f10e7 8163553: java.lang.LinkageError from test java/lang/ThreadGroup/Stop.java
psandoz
parents: 40810
diff changeset
   784
        } catch (Error | StringConcatException e) {
d04ede5f10e7 8163553: java.lang.LinkageError from test java/lang/ThreadGroup/Stop.java
psandoz
parents: 40810
diff changeset
   785
            // Pass through any error or existing StringConcatException
d04ede5f10e7 8163553: java.lang.LinkageError from test java/lang/ThreadGroup/Stop.java
psandoz
parents: 40810
diff changeset
   786
            throw e;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   787
        } catch (Throwable t) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   788
            throw new StringConcatException("Generator failed", t);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   789
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   790
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   791
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   792
    private enum Mode {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   793
        DEFAULT(false, false),
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   794
        SIZED(true, false),
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   795
        SIZED_EXACT(true, true);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   796
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   797
        private final boolean sized;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   798
        private final boolean exact;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   799
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   800
        Mode(boolean sized, boolean exact) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   801
            this.sized = sized;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   802
            this.exact = exact;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   803
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   804
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   805
        boolean isSized() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   806
            return sized;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   807
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   808
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   809
        boolean isExact() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   810
            return exact;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   811
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   812
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   813
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   814
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   815
     * Bytecode StringBuilder strategy.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   816
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   817
     * <p>This strategy operates in three modes, gated by {@link Mode}.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   818
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   819
     * <p><b>{@link Strategy#BC_SB}: "bytecode StringBuilder".</b>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   820
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   821
     * <p>This strategy spins up the bytecode that has the same StringBuilder
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   822
     * chain javac would otherwise emit. This strategy uses only the public API,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   823
     * and comes as the baseline for the current JDK behavior. On other words,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   824
     * this strategy moves the javac generated bytecode to runtime. The
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   825
     * generated bytecode is loaded via Unsafe.defineAnonymousClass, but with
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   826
     * the caller class coming from the BSM -- in other words, the protection
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   827
     * guarantees are inherited from the method where invokedynamic was
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   828
     * originally called. This means, among other things, that the bytecode is
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   829
     * verified for all non-JDK uses.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   830
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   831
     * <p><b>{@link Strategy#BC_SB_SIZED}: "bytecode StringBuilder, but
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   832
     * sized".</b>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   833
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   834
     * <p>This strategy acts similarly to {@link Strategy#BC_SB}, but it also
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   835
     * tries to guess the capacity required for StringBuilder to accept all
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   836
     * arguments without resizing. This strategy only makes an educated guess:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   837
     * it only guesses the space required for known types (e.g. primitives and
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   838
     * Strings), but does not otherwise convert arguments. Therefore, the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   839
     * capacity estimate may be wrong, and StringBuilder may have to
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   840
     * transparently resize or trim when doing the actual concatenation. While
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   841
     * this does not constitute a correctness issue (in the end, that what BC_SB
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   842
     * has to do anyway), this does pose a potential performance problem.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   843
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   844
     * <p><b>{@link Strategy#BC_SB_SIZED_EXACT}: "bytecode StringBuilder, but
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   845
     * sized exactly".</b>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   846
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   847
     * <p>This strategy improves on @link Strategy#BC_SB_SIZED}, by first
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   848
     * converting all arguments to String in order to get the exact capacity
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   849
     * StringBuilder should have. The conversion is done via the public
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   850
     * String.valueOf and/or Object.toString methods, and does not touch any
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   851
     * private String API.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   852
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   853
    private static final class BytecodeStringBuilderStrategy {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   854
        static final Unsafe UNSAFE = Unsafe.getUnsafe();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   855
        static final int CLASSFILE_VERSION = 52;
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   856
        static final String METHOD_NAME = "concat";
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   857
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   858
        private BytecodeStringBuilderStrategy() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   859
            // no instantiation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   860
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   861
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   862
        private static MethodHandle generate(Lookup lookup, String className, MethodType args, Recipe recipe, Mode mode) throws Exception {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   863
            ClassWriter cw = new ClassWriter(ClassWriter.COMPUTE_MAXS + ClassWriter.COMPUTE_FRAMES);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   864
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   865
            cw.visit(CLASSFILE_VERSION,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   866
                    ACC_SUPER + ACC_PUBLIC + ACC_FINAL + ACC_SYNTHETIC,
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   867
                    className,  // Unsafe.defineAnonymousClass would append an unique ID
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   868
                    null,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   869
                    "java/lang/Object",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   870
                    null
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   871
            );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   872
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   873
            MethodVisitor mv = cw.visitMethod(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   874
                    ACC_PUBLIC + ACC_STATIC + ACC_FINAL,
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
   875
                    METHOD_NAME,
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   876
                    args.toMethodDescriptorString(),
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   877
                    null,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   878
                    null);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   879
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   880
            mv.visitAnnotation("Ljdk/internal/vm/annotation/ForceInline;", true);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   881
            mv.visitCode();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   882
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   883
            Class<?>[] arr = args.parameterArray();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   884
            boolean[] guaranteedNonNull = new boolean[arr.length];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   885
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   886
            if (mode.isExact()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   887
                /*
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   888
                    In exact mode, we need to convert all arguments to their String representations,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   889
                    as this allows to compute their String sizes exactly. We cannot use private
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   890
                    methods for primitives in here, therefore we need to convert those as well.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   891
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   892
                    We also record what arguments are guaranteed to be non-null as the result
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   893
                    of the conversion. String.valueOf does the null checks for us. The only
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   894
                    corner case to take care of is String.valueOf(Object) returning null itself.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   895
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   896
                    Also, if any conversion happened, then the slot indices in the incoming
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   897
                    arguments are not equal to the final local maps. The only case this may break
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   898
                    is when converting 2-slot long/double argument to 1-slot String. Therefore,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   899
                    we get away with tracking modified offset, since no conversion can overwrite
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   900
                    the upcoming the argument.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   901
                 */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   902
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   903
                int off = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   904
                int modOff = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   905
                for (int c = 0; c < arr.length; c++) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   906
                    Class<?> cl = arr[c];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   907
                    if (cl == String.class) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   908
                        if (off != modOff) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   909
                            mv.visitIntInsn(getLoadOpcode(cl), off);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   910
                            mv.visitIntInsn(ASTORE, modOff);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   911
                        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   912
                    } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   913
                        mv.visitIntInsn(getLoadOpcode(cl), off);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   914
                        mv.visitMethodInsn(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   915
                                INVOKESTATIC,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   916
                                "java/lang/String",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   917
                                "valueOf",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   918
                                getStringValueOfDesc(cl),
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   919
                                false
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   920
                        );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   921
                        mv.visitIntInsn(ASTORE, modOff);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   922
                        arr[c] = String.class;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   923
                        guaranteedNonNull[c] = cl.isPrimitive();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   924
                    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   925
                    off += getParameterSize(cl);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   926
                    modOff += getParameterSize(String.class);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   927
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   928
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   929
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   930
            if (mode.isSized()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   931
                /*
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   932
                    When operating in sized mode (this includes exact mode), it makes sense to make
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   933
                    StringBuilder append chains look familiar to OptimizeStringConcat. For that, we
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   934
                    need to do null-checks early, not make the append chain shape simpler.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   935
                 */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   936
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   937
                int off = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   938
                for (RecipeElement el : recipe.getElements()) {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   939
                    switch (el.getTag()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   940
                        case TAG_CONST:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   941
                            // Guaranteed non-null, no null check required.
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   942
                            break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   943
                        case TAG_ARG:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   944
                            // Null-checks are needed only for String arguments, and when a previous stage
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   945
                            // did not do implicit null-checks. If a String is null, we eagerly replace it
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   946
                            // with "null" constant. Note, we omit Objects here, because we don't call
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   947
                            // .length() on them down below.
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   948
                            int ac = el.getArgPos();
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   949
                            Class<?> cl = arr[ac];
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   950
                            if (cl == String.class && !guaranteedNonNull[ac]) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   951
                                Label l0 = new Label();
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   952
                                mv.visitIntInsn(ALOAD, off);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   953
                                mv.visitJumpInsn(IFNONNULL, l0);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   954
                                mv.visitLdcInsn("null");
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   955
                                mv.visitIntInsn(ASTORE, off);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   956
                                mv.visitLabel(l0);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   957
                            }
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   958
                            off += getParameterSize(cl);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   959
                            break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   960
                        default:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   961
                            throw new StringConcatException("Unhandled tag: " + el.getTag());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   962
                    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   963
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   964
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   965
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   966
            // Prepare StringBuilder instance
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   967
            mv.visitTypeInsn(NEW, "java/lang/StringBuilder");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   968
            mv.visitInsn(DUP);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   969
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   970
            if (mode.isSized()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   971
                /*
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   972
                    Sized mode requires us to walk through the arguments, and estimate the final length.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   973
                    In exact mode, this will operate on Strings only. This code would accumulate the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   974
                    final length on stack.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   975
                 */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   976
                int len = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   977
                int off = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   978
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   979
                mv.visitInsn(ICONST_0);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   980
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
   981
                for (RecipeElement el : recipe.getElements()) {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   982
                    switch (el.getTag()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   983
                        case TAG_CONST:
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
   984
                            len += el.getValue().length();
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   985
                            break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   986
                        case TAG_ARG:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   987
                            /*
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   988
                                If an argument is String, then we can call .length() on it. Sized/Exact modes have
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   989
                                converted arguments for us. If an argument is primitive, we can provide a guess
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   990
                                for its String representation size.
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   991
                            */
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   992
                            Class<?> cl = arr[el.getArgPos()];
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   993
                            if (cl == String.class) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   994
                                mv.visitIntInsn(ALOAD, off);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   995
                                mv.visitMethodInsn(
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   996
                                        INVOKEVIRTUAL,
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   997
                                        "java/lang/String",
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   998
                                        "length",
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
   999
                                        "()I",
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1000
                                        false
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1001
                                );
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1002
                                mv.visitInsn(IADD);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1003
                            } else if (cl.isPrimitive()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1004
                                len += estimateSize(cl);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1005
                            }
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1006
                            off += getParameterSize(cl);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1007
                            break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1008
                        default:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1009
                            throw new StringConcatException("Unhandled tag: " + el.getTag());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1010
                    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1011
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1012
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1013
                // Constants have non-zero length, mix in
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1014
                if (len > 0) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1015
                    iconst(mv, len);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1016
                    mv.visitInsn(IADD);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1017
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1018
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1019
                mv.visitMethodInsn(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1020
                        INVOKESPECIAL,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1021
                        "java/lang/StringBuilder",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1022
                        "<init>",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1023
                        "(I)V",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1024
                        false
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1025
                );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1026
            } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1027
                mv.visitMethodInsn(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1028
                        INVOKESPECIAL,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1029
                        "java/lang/StringBuilder",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1030
                        "<init>",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1031
                        "()V",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1032
                        false
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1033
                );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1034
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1035
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1036
            // At this point, we have a blank StringBuilder on stack, fill it in with .append calls.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1037
            {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1038
                int off = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1039
                for (RecipeElement el : recipe.getElements()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1040
                    String desc;
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1041
                    switch (el.getTag()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1042
                        case TAG_CONST:
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1043
                            mv.visitLdcInsn(el.getValue());
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1044
                            desc = getSBAppendDesc(String.class);
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1045
                            break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1046
                        case TAG_ARG:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1047
                            Class<?> cl = arr[el.getArgPos()];
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1048
                            mv.visitVarInsn(getLoadOpcode(cl), off);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1049
                            off += getParameterSize(cl);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1050
                            desc = getSBAppendDesc(cl);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1051
                            break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1052
                        default:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1053
                            throw new StringConcatException("Unhandled tag: " + el.getTag());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1054
                    }
40256
c5e03eaf7ba2 8163370: Reduce number of classes loaded by common usage of java.lang.invoke
redestad
parents: 38868
diff changeset
  1055
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1056
                    mv.visitMethodInsn(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1057
                            INVOKEVIRTUAL,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1058
                            "java/lang/StringBuilder",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1059
                            "append",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1060
                            desc,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1061
                            false
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1062
                    );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1063
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1064
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1065
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1066
            if (DEBUG && mode.isExact()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1067
                /*
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1068
                    Exactness checks compare the final StringBuilder.capacity() with a resulting
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1069
                    String.length(). If these values disagree, that means StringBuilder had to perform
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1070
                    storage trimming, which defeats the purpose of exact strategies.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1071
                 */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1072
35639
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1073
                /*
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1074
                   The logic for this check is as follows:
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1075
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1076
                     Stack before:     Op:
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1077
                      (SB)              dup, dup
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1078
                      (SB, SB, SB)      capacity()
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1079
                      (int, SB, SB)     swap
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1080
                      (SB, int, SB)     toString()
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1081
                      (S, int, SB)      length()
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1082
                      (int, int, SB)    if_icmpeq
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1083
                      (SB)              <end>
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1084
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1085
                   Note that it leaves the same StringBuilder on exit, like the one on enter.
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1086
                 */
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1087
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1088
                mv.visitInsn(DUP);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1089
                mv.visitInsn(DUP);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1090
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1091
                mv.visitMethodInsn(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1092
                        INVOKEVIRTUAL,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1093
                        "java/lang/StringBuilder",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1094
                        "capacity",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1095
                        "()I",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1096
                        false
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1097
                );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1098
35639
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1099
                mv.visitInsn(SWAP);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1100
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1101
                mv.visitMethodInsn(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1102
                        INVOKEVIRTUAL,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1103
                        "java/lang/StringBuilder",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1104
                        "toString",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1105
                        "()Ljava/lang/String;",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1106
                        false
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1107
                );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1108
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1109
                mv.visitMethodInsn(
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1110
                        INVOKEVIRTUAL,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1111
                        "java/lang/String",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1112
                        "length",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1113
                        "()I",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1114
                        false
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1115
                );
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1116
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1117
                Label l0 = new Label();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1118
                mv.visitJumpInsn(IF_ICMPEQ, l0);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1119
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1120
                mv.visitTypeInsn(NEW, "java/lang/AssertionError");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1121
                mv.visitInsn(DUP);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1122
                mv.visitLdcInsn("Failed exactness check");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1123
                mv.visitMethodInsn(INVOKESPECIAL,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1124
                        "java/lang/AssertionError",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1125
                        "<init>",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1126
                        "(Ljava/lang/Object;)V",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1127
                        false);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1128
                mv.visitInsn(ATHROW);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1129
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1130
                mv.visitLabel(l0);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1131
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1132
35639
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1133
            mv.visitMethodInsn(
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1134
                    INVOKEVIRTUAL,
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1135
                    "java/lang/StringBuilder",
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1136
                    "toString",
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1137
                    "()Ljava/lang/String;",
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1138
                    false
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1139
            );
f34e7e8b4eac 8148787: StringConcatFactory exactness check produces bad bytecode when a non-arg concat is requested
shade
parents: 35401
diff changeset
  1140
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1141
            mv.visitInsn(ARETURN);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1142
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1143
            mv.visitMaxs(-1, -1);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1144
            mv.visitEnd();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1145
            cw.visitEnd();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1146
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1147
            byte[] classBytes = cw.toByteArray();
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1148
            try {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1149
                Class<?> hostClass = lookup.lookupClass();
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1150
                Class<?> innerClass = UNSAFE.defineAnonymousClass(hostClass, classBytes, null);
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1151
                UNSAFE.ensureClassInitialized(innerClass);
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1152
                dumpIfEnabled(innerClass.getName(), classBytes);
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1153
                return Lookup.IMPL_LOOKUP.findStatic(innerClass, METHOD_NAME, args);
42105
d04ede5f10e7 8163553: java.lang.LinkageError from test java/lang/ThreadGroup/Stop.java
psandoz
parents: 40810
diff changeset
  1154
            } catch (Exception e) {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1155
                dumpIfEnabled(className + "$$FAILED", classBytes);
42105
d04ede5f10e7 8163553: java.lang.LinkageError from test java/lang/ThreadGroup/Stop.java
psandoz
parents: 40810
diff changeset
  1156
                throw new StringConcatException("Exception while spinning the class", e);
35776
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
  1157
            }
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1158
        }
35776
dd5df83fcc0d 8149459: StringConcatFactory should be synced up with LambdaMetafactory
shade
parents: 35700
diff changeset
  1159
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1160
        private static void dumpIfEnabled(String name, byte[] bytes) {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1161
            if (DUMPER != null) {
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1162
                DUMPER.dumpClass(name, bytes);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1163
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1164
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1165
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1166
        private static String getSBAppendDesc(Class<?> cl) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1167
            if (cl.isPrimitive()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1168
                if (cl == Integer.TYPE || cl == Byte.TYPE || cl == Short.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1169
                    return "(I)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1170
                } else if (cl == Boolean.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1171
                    return "(Z)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1172
                } else if (cl == Character.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1173
                    return "(C)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1174
                } else if (cl == Double.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1175
                    return "(D)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1176
                } else if (cl == Float.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1177
                    return "(F)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1178
                } else if (cl == Long.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1179
                    return "(J)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1180
                } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1181
                    throw new IllegalStateException("Unhandled primitive StringBuilder.append: " + cl);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1182
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1183
            } else if (cl == String.class) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1184
                return "(Ljava/lang/String;)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1185
            } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1186
                return "(Ljava/lang/Object;)Ljava/lang/StringBuilder;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1187
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1188
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1189
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1190
        private static String getStringValueOfDesc(Class<?> cl) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1191
            if (cl.isPrimitive()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1192
                if (cl == Integer.TYPE || cl == Byte.TYPE || cl == Short.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1193
                    return "(I)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1194
                } else if (cl == Boolean.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1195
                    return "(Z)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1196
                } else if (cl == Character.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1197
                    return "(C)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1198
                } else if (cl == Double.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1199
                    return "(D)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1200
                } else if (cl == Float.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1201
                    return "(F)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1202
                } else if (cl == Long.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1203
                    return "(J)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1204
                } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1205
                    throw new IllegalStateException("Unhandled String.valueOf: " + cl);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1206
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1207
            } else if (cl == String.class) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1208
                return "(Ljava/lang/String;)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1209
            } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1210
                return "(Ljava/lang/Object;)Ljava/lang/String;";
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1211
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1212
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1213
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1214
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1215
         * The following method is copied from
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1216
         * org.objectweb.asm.commons.InstructionAdapter. Part of ASM: a very small
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1217
         * and fast Java bytecode manipulation framework.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1218
         * Copyright (c) 2000-2005 INRIA, France Telecom All rights reserved.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1219
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1220
        private static void iconst(MethodVisitor mv, final int cst) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1221
            if (cst >= -1 && cst <= 5) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1222
                mv.visitInsn(Opcodes.ICONST_0 + cst);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1223
            } else if (cst >= Byte.MIN_VALUE && cst <= Byte.MAX_VALUE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1224
                mv.visitIntInsn(Opcodes.BIPUSH, cst);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1225
            } else if (cst >= Short.MIN_VALUE && cst <= Short.MAX_VALUE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1226
                mv.visitIntInsn(Opcodes.SIPUSH, cst);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1227
            } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1228
                mv.visitLdcInsn(cst);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1229
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1230
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1231
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1232
        private static int getLoadOpcode(Class<?> c) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1233
            if (c == Void.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1234
                throw new InternalError("Unexpected void type of load opcode");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1235
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1236
            return ILOAD + getOpcodeOffset(c);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1237
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1238
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1239
        private static int getOpcodeOffset(Class<?> c) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1240
            if (c.isPrimitive()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1241
                if (c == Long.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1242
                    return 1;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1243
                } else if (c == Float.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1244
                    return 2;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1245
                } else if (c == Double.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1246
                    return 3;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1247
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1248
                return 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1249
            } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1250
                return 4;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1251
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1252
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1253
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1254
        private static int getParameterSize(Class<?> c) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1255
            if (c == Void.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1256
                return 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1257
            } else if (c == Long.TYPE || c == Double.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1258
                return 2;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1259
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1260
            return 1;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1261
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1262
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1263
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1264
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1265
     * MethodHandle StringBuilder strategy.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1266
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1267
     * <p>This strategy operates in two modes, gated by {@link Mode}.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1268
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1269
     * <p><b>{@link Strategy#MH_SB_SIZED}: "MethodHandles StringBuilder,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1270
     * sized".</b>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1271
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1272
     * <p>This strategy avoids spinning up the bytecode by building the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1273
     * computation on MethodHandle combinators. The computation is built with
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1274
     * public MethodHandle APIs, resolved from a public Lookup sequence, and
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1275
     * ends up calling the public StringBuilder API. Therefore, this strategy
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1276
     * does not use any private API at all, even the Unsafe.defineAnonymousClass,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1277
     * since everything is handled under cover by java.lang.invoke APIs.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1278
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1279
     * <p><b>{@link Strategy#MH_SB_SIZED_EXACT}: "MethodHandles StringBuilder,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1280
     * sized exactly".</b>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1281
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1282
     * <p>This strategy improves on @link Strategy#MH_SB_SIZED}, by first
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1283
     * converting all arguments to String in order to get the exact capacity
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1284
     * StringBuilder should have. The conversion is done via the public
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1285
     * String.valueOf and/or Object.toString methods, and does not touch any
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1286
     * private String API.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1287
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1288
    private static final class MethodHandleStringBuilderStrategy {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1289
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1290
        private MethodHandleStringBuilderStrategy() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1291
            // no instantiation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1292
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1293
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1294
        private static MethodHandle generate(MethodType mt, Recipe recipe, Mode mode) throws Exception {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1295
            int pc = mt.parameterCount();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1296
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1297
            Class<?>[] ptypes = mt.parameterArray();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1298
            MethodHandle[] filters = new MethodHandle[ptypes.length];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1299
            for (int i = 0; i < ptypes.length; i++) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1300
                MethodHandle filter;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1301
                switch (mode) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1302
                    case SIZED:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1303
                        // In sized mode, we convert all references and floats/doubles
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1304
                        // to String: there is no specialization for different
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1305
                        // classes in StringBuilder API, and it will convert to
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1306
                        // String internally anyhow.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1307
                        filter = Stringifiers.forMost(ptypes[i]);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1308
                        break;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1309
                    case SIZED_EXACT:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1310
                        // In exact mode, we convert everything to String:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1311
                        // this helps to compute the storage exactly.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1312
                        filter = Stringifiers.forAny(ptypes[i]);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1313
                        break;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1314
                    default:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1315
                        throw new StringConcatException("Not supported");
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1316
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1317
                if (filter != null) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1318
                    filters[i] = filter;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1319
                    ptypes[i] = filter.type().returnType();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1320
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1321
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1322
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1323
            MethodHandle[] lengthers = new MethodHandle[pc];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1324
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1325
            // Figure out lengths: constants' lengths can be deduced on the spot.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1326
            // All reference arguments were filtered to String in the combinators below, so we can
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1327
            // call the usual String.length(). Primitive values string sizes can be estimated.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1328
            int initial = 0;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1329
            for (RecipeElement el : recipe.getElements()) {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1330
                switch (el.getTag()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1331
                    case TAG_CONST:
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1332
                        initial += el.getValue().length();
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1333
                        break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1334
                    case TAG_ARG:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1335
                        final int i = el.getArgPos();
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1336
                        Class<?> type = ptypes[i];
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1337
                        if (type.isPrimitive()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1338
                            MethodHandle est = MethodHandles.constant(int.class, estimateSize(type));
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1339
                            est = MethodHandles.dropArguments(est, 0, type);
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1340
                            lengthers[i] = est;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1341
                        } else {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1342
                            lengthers[i] = STRING_LENGTH;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1343
                        }
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1344
                        break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1345
                    default:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1346
                        throw new StringConcatException("Unhandled tag: " + el.getTag());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1347
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1348
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1349
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1350
            // Create (StringBuilder, <args>) shape for appending:
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1351
            MethodHandle builder = MethodHandles.dropArguments(MethodHandles.identity(StringBuilder.class), 1, ptypes);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1352
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1353
            // Compose append calls. This is done in reverse because the application order is
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1354
            // reverse as well.
36737
67b2804be5ab 8152951: Avoid calculating the reverse of StringConcatFactory$Recipe elements
redestad
parents: 36668
diff changeset
  1355
            List<RecipeElement> elements = recipe.getElements();
67b2804be5ab 8152951: Avoid calculating the reverse of StringConcatFactory$Recipe elements
redestad
parents: 36668
diff changeset
  1356
            for (int i = elements.size() - 1; i >= 0; i--) {
67b2804be5ab 8152951: Avoid calculating the reverse of StringConcatFactory$Recipe elements
redestad
parents: 36668
diff changeset
  1357
                RecipeElement el = elements.get(i);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1358
                MethodHandle appender;
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1359
                switch (el.getTag()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1360
                    case TAG_CONST:
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1361
                        MethodHandle mh = appender(adaptToStringBuilder(String.class));
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1362
                        appender = MethodHandles.insertArguments(mh, 1, el.getValue());
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1363
                        break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1364
                    case TAG_ARG:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1365
                        int ac = el.getArgPos();
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1366
                        appender = appender(ptypes[ac]);
40256
c5e03eaf7ba2 8163370: Reduce number of classes loaded by common usage of java.lang.invoke
redestad
parents: 38868
diff changeset
  1367
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1368
                        // Insert dummy arguments to match the prefix in the signature.
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1369
                        // The actual appender argument will be the ac-ith argument.
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1370
                        if (ac != 0) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1371
                            appender = MethodHandles.dropArguments(appender, 1, Arrays.copyOf(ptypes, ac));
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1372
                        }
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1373
                        break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1374
                    default:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1375
                        throw new StringConcatException("Unhandled tag: " + el.getTag());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1376
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1377
                builder = MethodHandles.foldArguments(builder, appender);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1378
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1379
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1380
            // Build the sub-tree that adds the sizes and produces a StringBuilder:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1381
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1382
            // a) Start with the reducer that accepts all arguments, plus one
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1383
            //    slot for the initial value. Inject the initial value right away.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1384
            //    This produces (<ints>)int shape:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1385
            MethodHandle sum = getReducerFor(pc + 1);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1386
            MethodHandle adder = MethodHandles.insertArguments(sum, 0, initial);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1387
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1388
            // b) Apply lengthers to transform arguments to lengths, producing (<args>)int
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1389
            adder = MethodHandles.filterArguments(adder, 0, lengthers);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1390
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1391
            // c) Instantiate StringBuilder (<args>)int -> (<args>)StringBuilder
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1392
            MethodHandle newBuilder = MethodHandles.filterReturnValue(adder, NEW_STRING_BUILDER);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1393
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1394
            // d) Fold in StringBuilder constructor, this produces (<args>)StringBuilder
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1395
            MethodHandle mh = MethodHandles.foldArguments(builder, newBuilder);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1396
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1397
            // Convert non-primitive arguments to Strings
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1398
            mh = MethodHandles.filterArguments(mh, 0, filters);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1399
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1400
            // Convert (<args>)StringBuilder to (<args>)String
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1401
            if (DEBUG && mode.isExact()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1402
                mh = MethodHandles.filterReturnValue(mh, BUILDER_TO_STRING_CHECKED);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1403
            } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1404
                mh = MethodHandles.filterReturnValue(mh, BUILDER_TO_STRING);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1405
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1406
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1407
            return mh;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1408
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1409
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1410
        private static MethodHandle getReducerFor(int cnt) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1411
            return SUMMERS.computeIfAbsent(cnt, SUMMER);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1412
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1413
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1414
        private static MethodHandle appender(Class<?> appendType) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1415
            MethodHandle appender = lookupVirtual(MethodHandles.publicLookup(), StringBuilder.class, "append",
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1416
                    StringBuilder.class, adaptToStringBuilder(appendType));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1417
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1418
            // appenders should return void, this would not modify the target signature during folding
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1419
            MethodType nt = MethodType.methodType(void.class, StringBuilder.class, appendType);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1420
            return appender.asType(nt);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1421
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1422
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1423
        private static String toStringChecked(StringBuilder sb) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1424
            String s = sb.toString();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1425
            if (s.length() != sb.capacity()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1426
                throw new AssertionError("Exactness check failed: result length = " + s.length() + ", buffer capacity = " + sb.capacity());
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1427
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1428
            return s;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1429
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1430
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1431
        private static int sum(int v1, int v2) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1432
            return v1 + v2;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1433
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1434
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1435
        private static int sum(int v1, int v2, int v3) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1436
            return v1 + v2 + v3;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1437
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1438
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1439
        private static int sum(int v1, int v2, int v3, int v4) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1440
            return v1 + v2 + v3 + v4;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1441
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1442
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1443
        private static int sum(int v1, int v2, int v3, int v4, int v5) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1444
            return v1 + v2 + v3 + v4 + v5;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1445
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1446
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1447
        private static int sum(int v1, int v2, int v3, int v4, int v5, int v6) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1448
            return v1 + v2 + v3 + v4 + v5 + v6;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1449
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1450
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1451
        private static int sum(int v1, int v2, int v3, int v4, int v5, int v6, int v7) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1452
            return v1 + v2 + v3 + v4 + v5 + v6 + v7;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1453
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1454
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1455
        private static int sum(int v1, int v2, int v3, int v4, int v5, int v6, int v7, int v8) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1456
            return v1 + v2 + v3 + v4 + v5 + v6 + v7 + v8;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1457
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1458
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1459
        private static int sum(int initial, int[] vs) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1460
            int sum = initial;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1461
            for (int v : vs) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1462
                sum += v;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1463
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1464
            return sum;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1465
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1466
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1467
        private static final ConcurrentMap<Integer, MethodHandle> SUMMERS;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1468
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1469
        // This one is deliberately non-lambdified to optimize startup time:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1470
        private static final Function<Integer, MethodHandle> SUMMER = new Function<Integer, MethodHandle>() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1471
            @Override
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1472
            public MethodHandle apply(Integer cnt) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1473
                if (cnt == 1) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1474
                    return MethodHandles.identity(int.class);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1475
                } else if (cnt <= 8) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1476
                    // Variable-arity collectors are not as efficient as small-count methods,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1477
                    // unroll some initial sizes.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1478
                    Class<?>[] cls = new Class<?>[cnt];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1479
                    Arrays.fill(cls, int.class);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1480
                    return lookupStatic(Lookup.IMPL_LOOKUP, MethodHandleStringBuilderStrategy.class, "sum", int.class, cls);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1481
                } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1482
                    return lookupStatic(Lookup.IMPL_LOOKUP, MethodHandleStringBuilderStrategy.class, "sum", int.class, int.class, int[].class)
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1483
                            .asCollector(int[].class, cnt - 1);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1484
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1485
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1486
        };
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1487
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1488
        private static final MethodHandle NEW_STRING_BUILDER, STRING_LENGTH, BUILDER_TO_STRING, BUILDER_TO_STRING_CHECKED;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1489
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1490
        static {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1491
            SUMMERS = new ConcurrentHashMap<>();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1492
            Lookup publicLookup = MethodHandles.publicLookup();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1493
            NEW_STRING_BUILDER = lookupConstructor(publicLookup, StringBuilder.class, int.class);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1494
            STRING_LENGTH = lookupVirtual(publicLookup, String.class, "length", int.class);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1495
            BUILDER_TO_STRING = lookupVirtual(publicLookup, StringBuilder.class, "toString", String.class);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1496
            if (DEBUG) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1497
                BUILDER_TO_STRING_CHECKED = lookupStatic(MethodHandles.Lookup.IMPL_LOOKUP,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1498
                        MethodHandleStringBuilderStrategy.class, "toStringChecked", String.class, StringBuilder.class);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1499
            } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1500
                BUILDER_TO_STRING_CHECKED = null;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1501
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1502
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1503
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1504
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1505
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1506
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1507
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1508
     * <p><b>{@link Strategy#MH_INLINE_SIZED_EXACT}: "MethodHandles inline,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1509
     * sized exactly".</b>
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1510
     *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1511
     * <p>This strategy replicates what StringBuilders are doing: it builds the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1512
     * byte[] array on its own and passes that byte[] array to String
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1513
     * constructor. This strategy requires access to some private APIs in JDK,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1514
     * most notably, the read-only Integer/Long.stringSize methods that measure
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1515
     * the character length of the integers, and the private String constructor
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1516
     * that accepts byte[] arrays without copying. While this strategy assumes a
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1517
     * particular implementation details for String, this opens the door for
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1518
     * building a very optimal concatenation sequence. This is the only strategy
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1519
     * that requires porting if there are private JDK changes occur.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1520
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1521
    private static final class MethodHandleInlineCopyStrategy {
36668
ceb837301c53 8150463: StringConcat MH_INLINE_SIZED_EXACT should skip storage initialization
shade
parents: 36511
diff changeset
  1522
        static final Unsafe UNSAFE = Unsafe.getUnsafe();
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1523
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1524
        private MethodHandleInlineCopyStrategy() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1525
            // no instantiation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1526
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1527
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1528
        static MethodHandle generate(MethodType mt, Recipe recipe) throws Throwable {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1529
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1530
            // Fast-path two-argument Object + Object concatenations
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1531
            if (recipe.getElements().size() == 2) {
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1532
                // Two object arguments
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1533
                if (mt.parameterCount() == 2 &&
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1534
                        !mt.parameterType(0).isPrimitive() &&
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1535
                        !mt.parameterType(1).isPrimitive()) {
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1536
                    return SIMPLE;
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1537
                }
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1538
                // One element is a constant
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1539
                if (mt.parameterCount() == 1 && !mt.parameterType(0).isPrimitive()) {
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1540
                    MethodHandle mh = SIMPLE;
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1541
                    // Insert constant element
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1542
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1543
                    // First recipe element is a constant
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1544
                    if (recipe.getElements().get(0).getTag() == TAG_CONST &&
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1545
                        recipe.getElements().get(1).getTag() != TAG_CONST) {
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1546
                        return MethodHandles.insertArguments(mh, 0,
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1547
                                recipe.getElements().get(0).getValue());
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1548
                    } else if (recipe.getElements().get(1).getTag() == TAG_CONST &&
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1549
                               recipe.getElements().get(0).getTag() != TAG_CONST) {
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1550
                        return MethodHandles.insertArguments(mh, 1,
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1551
                                recipe.getElements().get(1).getValue());
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1552
                    }
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1553
                    // else... fall-through to slow-path
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1554
                }
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1555
            }
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1556
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1557
            // Create filters and obtain filtered parameter types. Filters would be used in the beginning
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1558
            // to convert the incoming arguments into the arguments we can process (e.g. Objects -> Strings).
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1559
            // The filtered argument type list is used all over in the combinators below.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1560
            Class<?>[] ptypes = mt.parameterArray();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1561
            MethodHandle[] filters = null;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1562
            for (int i = 0; i < ptypes.length; i++) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1563
                MethodHandle filter = Stringifiers.forMost(ptypes[i]);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1564
                if (filter != null) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1565
                    if (filters == null) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1566
                        filters = new MethodHandle[ptypes.length];
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1567
                    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1568
                    filters[i] = filter;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1569
                    ptypes[i] = filter.type().returnType();
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1570
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1571
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1572
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1573
            // Start building the combinator tree. The tree "starts" with (<parameters>)String, and "finishes"
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1574
            // with the (byte[], long)String shape to invoke newString in StringConcatHelper. The combinators are
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1575
            // assembled bottom-up, which makes the code arguably hard to read.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1576
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1577
            // Drop all remaining parameter types, leave only helper arguments:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1578
            MethodHandle mh;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1579
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1580
            mh = MethodHandles.dropArguments(NEW_STRING, 2, ptypes);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1581
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1582
            // Mix in prependers. This happens when (byte[], long) = (storage, indexCoder) is already
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1583
            // known from the combinators below. We are assembling the string backwards, so the index coded
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1584
            // into indexCoder is the *ending* index.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1585
            for (RecipeElement el : recipe.getElements()) {
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1586
                // Do the prepend, and put "new" index at index 1
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1587
                switch (el.getTag()) {
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1588
                    case TAG_CONST: {
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1589
                        MethodHandle prepender = MethodHandles.insertArguments(prepender(String.class), 2, el.getValue());
52226
b4b932c6001f 8212726: Replace some use of drop- and foldArguments with filtering argument combinator in StringConcatFactory
redestad
parents: 52177
diff changeset
  1590
                        mh = MethodHandles.filterArgumentsWithCombiner(mh, 1, prepender,
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1591
                                1, 0 // indexCoder, storage
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1592
                        );
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1593
                        break;
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1594
                    }
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1595
                    case TAG_ARG: {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1596
                        int pos = el.getArgPos();
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1597
                        MethodHandle prepender = prepender(ptypes[pos]);
52226
b4b932c6001f 8212726: Replace some use of drop- and foldArguments with filtering argument combinator in StringConcatFactory
redestad
parents: 52177
diff changeset
  1598
                        mh = MethodHandles.filterArgumentsWithCombiner(mh, 1, prepender,
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1599
                                1, 0, // indexCoder, storage
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1600
                                2 + pos  // selected argument
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1601
                        );
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1602
                        break;
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1603
                    }
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1604
                    default:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1605
                        throw new StringConcatException("Unhandled tag: " + el.getTag());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1606
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1607
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1608
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1609
            // Fold in byte[] instantiation at argument 0
52226
b4b932c6001f 8212726: Replace some use of drop- and foldArguments with filtering argument combinator in StringConcatFactory
redestad
parents: 52177
diff changeset
  1610
            mh = MethodHandles.foldArgumentsWithCombiner(mh, 0, NEW_ARRAY,
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1611
                    1 // index
40810
b88d5910ea1e 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
redestad
parents: 40272
diff changeset
  1612
            );
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1613
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1614
            // Start combining length and coder mixers.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1615
            //
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1616
            // Length is easy: constant lengths can be computed on the spot, and all non-constant
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1617
            // shapes have been either converted to Strings, or explicit methods for getting the
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1618
            // string length out of primitives are provided.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1619
            //
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1620
            // Coders are more interesting. Only Object, String and char arguments (and constants)
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1621
            // can have non-Latin1 encoding. It is easier to blindly convert constants to String,
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1622
            // and deduce the coder from there. Arguments would be either converted to Strings
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1623
            // during the initial filtering, or handled by specializations in MIXERS.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1624
            //
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1625
            // The method handle shape before and after all mixers are combined in is:
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1626
            //   (long, <args>)String = ("indexCoder", <args>)
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1627
            long initialLengthCoder = INITIAL_CODER;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1628
            for (RecipeElement el : recipe.getElements()) {
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1629
                switch (el.getTag()) {
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1630
                    case TAG_CONST:
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1631
                        String constant = el.getValue();
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1632
                        initialLengthCoder = (long)mixer(String.class).invoke(initialLengthCoder, constant);
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1633
                        break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1634
                    case TAG_ARG:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1635
                        int ac = el.getArgPos();
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1636
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1637
                        Class<?> argClass = ptypes[ac];
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1638
                        MethodHandle mix = mixer(argClass);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1639
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1640
                        // Compute new "index" in-place using old value plus the appropriate argument.
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1641
                        mh = MethodHandles.filterArgumentsWithCombiner(mh, 0, mix,
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1642
                                0, // old-index
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1643
                                1 + ac // selected argument
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1644
                        );
52177
430e6421d503 8212597: Optimize String concatenation setup when using primitive operands
redestad
parents: 48930
diff changeset
  1645
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1646
                        break;
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1647
                    default:
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1648
                        throw new StringConcatException("Unhandled tag: " + el.getTag());
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1649
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1650
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1651
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1652
            // Insert initial length and coder value here.
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1653
            // The method handle shape here is (<args>).
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1654
            mh = MethodHandles.insertArguments(mh, 0, initialLengthCoder);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1655
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1656
            // Apply filters, converting the arguments:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1657
            if (filters != null) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1658
                mh = MethodHandles.filterArguments(mh, 0, filters);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1659
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1660
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1661
            return mh;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1662
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1663
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1664
        private static MethodHandle prepender(Class<?> cl) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1665
            return PREPENDERS.computeIfAbsent(cl, PREPEND);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1666
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1667
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1668
        private static MethodHandle mixer(Class<?> cl) {
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1669
            return MIXERS.computeIfAbsent(cl, MIX);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1670
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1671
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1672
        // This one is deliberately non-lambdified to optimize startup time:
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1673
        private static final Function<Class<?>, MethodHandle> PREPEND = new Function<Class<?>, MethodHandle>() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1674
            @Override
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1675
            public MethodHandle apply(Class<?> c) {
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1676
                return lookupStatic(Lookup.IMPL_LOOKUP, STRING_HELPER, "prepend", long.class, long.class, byte[].class,
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1677
                        Wrapper.asPrimitiveType(c));
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1678
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1679
        };
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1680
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1681
        // This one is deliberately non-lambdified to optimize startup time:
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1682
        private static final Function<Class<?>, MethodHandle> MIX = new Function<Class<?>, MethodHandle>() {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1683
            @Override
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1684
            public MethodHandle apply(Class<?> c) {
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1685
                return lookupStatic(Lookup.IMPL_LOOKUP, STRING_HELPER, "mix", long.class, long.class,
46902
c6cefe631b18 8186500: StringConcatFactory.makeConcatWithConstants throws AssertionError when recipe contains non-String constants
redestad
parents: 42105
diff changeset
  1686
                        Wrapper.asPrimitiveType(c));
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1687
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1688
        };
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1689
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1690
        private static final MethodHandle SIMPLE;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1691
        private static final MethodHandle NEW_STRING;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1692
        private static final MethodHandle NEW_ARRAY;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1693
        private static final ConcurrentMap<Class<?>, MethodHandle> PREPENDERS;
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1694
        private static final ConcurrentMap<Class<?>, MethodHandle> MIXERS;
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1695
        private static final long INITIAL_CODER;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1696
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1697
        static {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1698
            try {
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1699
                MethodHandle initCoder = lookupStatic(Lookup.IMPL_LOOKUP, STRING_HELPER, "initialCoder", long.class);
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1700
                INITIAL_CODER = (long) initCoder.invoke();
35700
b933119b8f84 8148869: StringConcatFactory MH_INLINE_SIZED_EXACT strategy does not work with -XX:-CompactStrings
shade
parents: 35639
diff changeset
  1701
            } catch (Throwable e) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1702
                throw new AssertionError(e);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1703
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1704
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1705
            PREPENDERS = new ConcurrentHashMap<>();
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1706
            MIXERS = new ConcurrentHashMap<>();
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1707
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1708
            SIMPLE     = lookupStatic(Lookup.IMPL_LOOKUP, STRING_HELPER, "simpleConcat", String.class, Object.class, Object.class);
52326
77018c2b97df 8213035: Pack MethodHandleInlineStrategy coder and length into a long
redestad
parents: 52226
diff changeset
  1709
            NEW_STRING = lookupStatic(Lookup.IMPL_LOOKUP, STRING_HELPER, "newString", String.class, byte[].class, long.class);
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1710
            NEW_ARRAY  = lookupStatic(Lookup.IMPL_LOOKUP, STRING_HELPER, "newArray", byte[].class, long.class);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1711
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1712
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1713
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1714
    /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1715
     * Public gateways to public "stringify" methods. These methods have the form String apply(T obj), and normally
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1716
     * delegate to {@code String.valueOf}, depending on argument's type.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1717
     */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1718
    private static final class Stringifiers {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1719
        private Stringifiers() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1720
            // no instantiation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1721
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1722
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1723
        private static final MethodHandle OBJECT_INSTANCE =
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1724
            lookupStatic(Lookup.IMPL_LOOKUP, STRING_HELPER, "stringOf", String.class, Object.class);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1725
52497
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1726
        private static class FloatStringifiers {
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1727
            private static final MethodHandle FLOAT_INSTANCE =
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1728
                    lookupStatic(MethodHandles.publicLookup(), String.class, "valueOf", String.class, float.class);
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1729
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1730
            private static final MethodHandle DOUBLE_INSTANCE =
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1731
                    lookupStatic(MethodHandles.publicLookup(), String.class, "valueOf", String.class, double.class);
38868
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1732
        }
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1733
38868
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1734
        private static class StringifierAny extends ClassValue<MethodHandle> {
52497
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1735
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1736
            private static final ClassValue<MethodHandle> INSTANCE = new StringifierAny();
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1737
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1738
            @Override
38868
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1739
            protected MethodHandle computeValue(Class<?> cl) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1740
                if (cl == byte.class || cl == short.class || cl == int.class) {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1741
                    return lookupStatic(MethodHandles.publicLookup(), String.class, "valueOf", String.class, int.class);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1742
                } else if (cl == boolean.class) {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1743
                    return lookupStatic(MethodHandles.publicLookup(), String.class, "valueOf", String.class, boolean.class);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1744
                } else if (cl == char.class) {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1745
                    return lookupStatic(MethodHandles.publicLookup(), String.class, "valueOf", String.class, char.class);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1746
                } else if (cl == long.class) {
36001
5f0acf0668e0 8149835: StringConcatFactory should emit classes with the same package as the host class
shade
parents: 35776
diff changeset
  1747
                    return lookupStatic(MethodHandles.publicLookup(), String.class, "valueOf", String.class, long.class);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1748
                } else {
52497
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1749
                    MethodHandle mh = forMost(cl);
38868
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1750
                    if (mh != null) {
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1751
                        return mh;
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1752
                    } else {
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1753
                        throw new IllegalStateException("Unknown class: " + cl);
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1754
                    }
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1755
                }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1756
            }
38868
f353ab3063f8 8158933: String concat stringifiers setup should avoid unnecessary lookups
redestad
parents: 38428
diff changeset
  1757
        }
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1758
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1759
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1760
         * Returns a stringifier for references and floats/doubles only.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1761
         * Always returns null for other primitives.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1762
         *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1763
         * @param t class to stringify
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1764
         * @return stringifier; null, if not available
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1765
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1766
        static MethodHandle forMost(Class<?> t) {
52497
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1767
            if (!t.isPrimitive()) {
54550
5fa7fbddfe9d 8222484: Specialize generation of simple String concatenation expressions
redestad
parents: 52497
diff changeset
  1768
                return OBJECT_INSTANCE;
52497
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1769
            } else if (t == float.class) {
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1770
                return FloatStringifiers.FLOAT_INSTANCE;
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1771
            } else if (t == double.class) {
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1772
                return FloatStringifiers.DOUBLE_INSTANCE;
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1773
            }
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1774
            return null;
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1775
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1776
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1777
        /**
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1778
         * Returns a stringifier for any type. Never returns null.
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1779
         *
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1780
         * @param t class to stringify
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1781
         * @return stringifier
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1782
         */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1783
        static MethodHandle forAny(Class<?> t) {
52497
34510f65fb58 8213741: Consolidate Object and String Stringifiers
redestad
parents: 52326
diff changeset
  1784
            return StringifierAny.INSTANCE.get(t);
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1785
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1786
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1787
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1788
    /* ------------------------------- Common utilities ------------------------------------ */
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1789
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1790
    static MethodHandle lookupStatic(Lookup lookup, Class<?> refc, String name, Class<?> rtype, Class<?>... ptypes) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1791
        try {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1792
            return lookup.findStatic(refc, name, MethodType.methodType(rtype, ptypes));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1793
        } catch (NoSuchMethodException | IllegalAccessException e) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1794
            throw new AssertionError(e);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1795
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1796
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1797
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1798
    static MethodHandle lookupVirtual(Lookup lookup, Class<?> refc, String name, Class<?> rtype, Class<?>... ptypes) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1799
        try {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1800
            return lookup.findVirtual(refc, name, MethodType.methodType(rtype, ptypes));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1801
        } catch (NoSuchMethodException | IllegalAccessException e) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1802
            throw new AssertionError(e);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1803
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1804
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1805
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1806
    static MethodHandle lookupConstructor(Lookup lookup, Class<?> refc, Class<?> ptypes) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1807
        try {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1808
            return lookup.findConstructor(refc, MethodType.methodType(void.class, ptypes));
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1809
        } catch (NoSuchMethodException | IllegalAccessException e) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1810
            throw new AssertionError(e);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1811
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1812
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1813
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1814
    static int estimateSize(Class<?> cl) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1815
        if (cl == Integer.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1816
            return 11; // "-2147483648"
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1817
        } else if (cl == Boolean.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1818
            return 5; // "false"
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1819
        } else if (cl == Byte.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1820
            return 4; // "-128"
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1821
        } else if (cl == Character.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1822
            return 1; // duh
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1823
        } else if (cl == Short.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1824
            return 6; // "-32768"
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1825
        } else if (cl == Double.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1826
            return 26; // apparently, no larger than this, see FloatingDecimal.BinaryToASCIIBuffer.buffer
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1827
        } else if (cl == Float.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1828
            return 26; // apparently, no larger than this, see FloatingDecimal.BinaryToASCIIBuffer.buffer
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1829
        } else if (cl == Long.TYPE)  {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1830
            return 20; // "-9223372036854775808"
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1831
        } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1832
            throw new IllegalArgumentException("Cannot estimate the size for " + cl);
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1833
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1834
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1835
40272
6af4511ee5a4 8163878: Remove unnecessary bridge methods, allocations in java.lang.invoke
redestad
parents: 40256
diff changeset
  1836
    static Class<?> adaptToStringBuilder(Class<?> c) {
35388
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1837
        if (c.isPrimitive()) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1838
            if (c == Byte.TYPE || c == Short.TYPE) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1839
                return int.class;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1840
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1841
        } else {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1842
            if (c != String.class) {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1843
                return Object.class;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1844
            }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1845
        }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1846
        return c;
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1847
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1848
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1849
    private StringConcatFactory() {
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1850
        // no instantiation
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1851
    }
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1852
a33a232cf7b7 8148483: JEP 280: Indify String Concatenation
shade
parents:
diff changeset
  1853
}