RFR: 8268398: 15% increase in JFR footprint in Noop-Base [v2]

[Erik Gahlin]

On Fri, 17 Jun 2022 11:53:20 GMT, Jaroslav Bachorik <jbachorik at openjdk.org> wrote:

>> Erik Gahlin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update comment
>
> src/jdk.jfr/share/classes/jdk/jfr/internal/MetadataRepository.java line 221:
> 
>> 219:         PlatformEventType pe = configuration.getPlatformEventType();
>> 220:         pe.setRegistered(true);
>> 221:         if (jvm.isInstrumented(eventClass) || !Utils.shouldInstrument(pe.isJDK(), pe.getName())) {
> 
> This will call `setInstrumented()` even for events which are not really instrumented when they are not required to be instrumented, correct?

Yes, I added comment to make it more clear.

-------------

PR: https://git.openjdk.org/jdk19/pull/31