> Hi, > > Could I have a review of an enhancement that replaces the use of ASM with the > new Class-File API. This change only deals with bytecode that writes event > data into buffers. Bytecode transformations carried out by classes in > jdk.jfr.internal.intrument package are kept as is. Plan is to try to replace > those with events in java.base. > > To simplify the review process, I have tried to keep the code in the classes > EventInstrumentation and EventClassBuilder similar to what existed before. > Further refactorizations may happen at a later stage. > > Testing: tier1-3 + jdk/jdk/jfr > > Thanks > Erik
Erik Gahlin has updated the pull request incrementally with one additional commit since the last revision: Blessed order ------------- Changes: - all: https://git.openjdk.org/jdk/pull/16195/files - new: https://git.openjdk.org/jdk/pull/16195/files/ba96d273..a0278e7a Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=16195&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=16195&range=01-02 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/16195.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/16195/head:pull/16195 PR: https://git.openjdk.org/jdk/pull/16195
