RFR: 8354949: JFR: Split up the EventInstrumentation class [v2]

[Erik Gahlin]

> Could I have a review of a PR that cleans up the EventInstrumentation class and moves the non-modifying parts into a separate class (ClassInspector)? 
> 
> I deleted the code that removes the instrumentation (buildUninstrumented), because it has never been used. If there should be a need for it in the future, it's not hard to add code that inserts empty method bodies.
> 
> Testing: tier1 + jdk/jdk/jfr/test
> 
> Thanks
> Erik

Erik Gahlin has updated the pull request incrementally with one additional commit since the last revision:

  Build settings and fields only when needed

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/24726/files
  - new: https://git.openjdk.org/jdk/pull/24726/files/e692363d..8ecb2732

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=24726&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=24726&range=00-01

  Stats: 53 lines in 3 files changed: 11 ins; 11 del; 31 mod
  Patch: https://git.openjdk.org/jdk/pull/24726.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/24726/head:pull/24726

PR: https://git.openjdk.org/jdk/pull/24726