src/jdk.jfr/share/classes/jdk/jfr/internal/instrument/JIMethodMergeAdapter.java
author egahlin
Tue, 15 May 2018 20:24:34 +0200
changeset 50113 caf115bb98ad
child 52515 746df0ae4fe1
permissions -rw-r--r--
8199712: Flight Recorder Reviewed-by: coleenp, ihse, erikj, dsamersoff, mseledtsov, egahlin, mgronlun Contributed-by: erik.gahlin@oracle.com, markus.gronlund@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     2
 * Copyright (c) 2013, 2018, Oracle and/or its affiliates. All rights reserved.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     4
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    10
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    15
 * accompanied this code).
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    16
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    20
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    23
 * questions.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    24
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    25
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    26
package jdk.jfr.internal.instrument;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
import java.lang.reflect.Method;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
import java.util.HashMap;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
import java.util.List;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
import java.util.Map;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
import jdk.internal.org.objectweb.asm.ClassVisitor;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
import jdk.internal.org.objectweb.asm.MethodVisitor;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
import jdk.internal.org.objectweb.asm.Opcodes;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
import jdk.internal.org.objectweb.asm.Type;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
import jdk.internal.org.objectweb.asm.commons.RemappingMethodAdapter;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
import jdk.internal.org.objectweb.asm.commons.SimpleRemapper;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
import jdk.internal.org.objectweb.asm.tree.ClassNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
import jdk.internal.org.objectweb.asm.tree.MethodNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
import jdk.jfr.internal.LogLevel;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
import jdk.jfr.internal.LogTag;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
import jdk.jfr.internal.Logger;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
/**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
 * This class will merge (some) methods from one class into another one.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
 * @author Staffan Larsen
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
@Deprecated
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
final class JIMethodMergeAdapter extends ClassVisitor {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
    private final ClassNode cn;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
    private final List<Method> methodFilter;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
    private final Map<String, String> typeMap;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
     * Methods in methodFilter that exist in cn will be merged into cv. If the method already exists,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
     * the original method will be deleted.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
     *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
     * @param cv
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
     * @param cn - a ClassNode with Methods that will be merged into this class
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
     * @param methodFilter - only methods in this list will be merged
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
     * @param typeMappings - while merging, type references in the methods will be changed according to this map
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
    public JIMethodMergeAdapter(ClassVisitor cv, ClassNode cn, List<Method> methodFilter, JITypeMapping[] typeMappings) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
        super(Opcodes.ASM5, cv);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
        this.cn = cn;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
        this.methodFilter = methodFilter;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
        this.typeMap = new HashMap<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
        for (JITypeMapping tm : typeMappings) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
            typeMap.put(tm.from().replace('.', '/'), tm.to().replace('.', '/'));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
    @Override
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
    public void visit(int version, int access, String name, String signature, String superName, String[] interfaces) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
        super.visit(version, access, name, signature, superName, interfaces);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
        typeMap.put(cn.name, name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
    @Override
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
    public MethodVisitor visitMethod(int access, String name, String desc, String signature, String[] exceptions) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
        if(methodInFilter(name, desc)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
            // If the method is one that we will be replacing, delete the method
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
            Logger.log(LogTag.JFR_SYSTEM_BYTECODE, LogLevel.DEBUG, "Deleting " + name + desc);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
            return null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
        return super.visitMethod(access, name, desc, signature, exceptions);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
    @Override
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
    public void visitEnd() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
        SimpleRemapper remapper = new SimpleRemapper(typeMap);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
        for (MethodNode mn : cn.methods) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
            // Check if the method is in the list of methods to copy
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
            if (methodInFilter(mn.name, mn.desc)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
                Logger.log(LogTag.JFR_SYSTEM_BYTECODE, LogLevel.DEBUG, "Copying method: " + mn.name + mn.desc);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
                Logger.log(LogTag.JFR_SYSTEM_BYTECODE, LogLevel.DEBUG,  "   with mapper: " + typeMap);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
                String[] exceptions = new String[mn.exceptions.size()];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
                mn.exceptions.toArray(exceptions);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
                MethodVisitor mv = cv.visitMethod(mn.access, mn.name, mn.desc, mn.signature, exceptions);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
                mn.instructions.resetLabels();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
                mn.accept(new RemappingMethodAdapter(mn.access, mn.desc, mv, remapper));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
        super.visitEnd();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
    private boolean methodInFilter(String name, String desc) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
        for(Method m : methodFilter) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
            if (m.getName().equals(name) && Type.getMethodDescriptor(m).equals(desc)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
                return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
        return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
}