[15] RFR: 8193210: [JVMCI/Graal] add JFR compiler phase/inlining events

[Jamsheed C M]

Hi,

i messed up with order of includes, got undetected due to PCH. corrected 
it, updated webrev in place.

corrected the title.

Best regards,

Jamsheed

On 26/02/2020 07:35, Jamsheed C M wrote:
>
> Hi,
>
> This enhancement adds jvmci supports that is required for extending 
> existing phase/inlining events support in graal compilers.
>
> During this enhancement all jfr events related codes are moved to 
> compiler/compilerEvent.[.hpp, inline.hpp, .cpp]
>
> Graal use these events via jvmci JFR helper class, which uses c2v 
> interface to post and get information from hotspot
>
> Overview of functionalities exposed by JFR helper class
> <http://cr.openjdk.java.net/~jcm/8193210/webrev/src/jdk.internal.vm.ci/share/classes/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/JFR.java.html>
>
> Ticks, for getting current jfr counter/ time stamp. used for getting 
> start time of events by Graal compiler.
>
> CompilerPhaseEvent for posting phase event to hotspot JFR.
>
> CompilerInlininingEvent for posting inlininig events.
>
> CompilerPhaseEvent extends and use TYPE_COMPILERPHASETYPE content type 
> for phase events, this is achieved using reregistration support added 
> by Markus [1]
>
> CompilerEvent::PhaseType::register_phases is used for registering 
> phases. This function is usually called on compiler singleton object 
> creation. It is also called lazily if phases are not known beforehand.
>
> The event generation codes in c1/c1_GraphBuilder.cpp, 
> opto/bytecodeInfo.cpp, opto/compile.hpp uses common functions exposed 
> by compiler/compilerEvent.hpp 
> <http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/compiler/compilerEvent.hpp.html>/ 
> inline.hpp 
> <http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/compiler/compilerEvent.inline.hpp.html>/ ..cpp 
> <http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/compiler/compilerEvent.cpp.html>
>
> webrev: http://cr.openjdk.java.net/~jcm/8193210/webrev/
>
> jbs: https://bugs.openjdk.java.net/browse/JDK-8193210
>
> Testing: ran jdk/jfr in my local machine. mach5 run created, links in jbs.
>
> Please review,
>
> Best regards,
>
> Jamsheed
>
> [1] 
> http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/jfr/recorder/checkpoint/jfrCheckpointManager.cpp.udiff.html 
>
>
> http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/jfr/recorder/checkpoint/jfrCheckpointWriter.hpp.udiff.html
>
> http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/jfr/recorder/checkpoint/types/jfrTypeManager.cpp.udiff.html
>
> http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/jfr/recorder/checkpoint/types/jfrTypeManager.hpp.udiff.html
>
> http://cr.openjdk.java.net/~jcm/8193210/webrev/src/hotspot/share/jfr/utilities/jfrTypes.hpp.udiff.html
>
> http://cr.openjdk.java.net/~jcm/8193210/webrev/src/jdk.jfr/share/classes/jdk/jfr/internal/consumer/ChunkParser.java.udiff.html
>