> This PR adds a new annotation, `@AOTInitialize` that forces a class to be (a) 
> initialized in the AOT assembly phase, and (b) stored in the AOT cache in an 
> already initialized state. This means that all the static fields in this 
> class will be immediately available upon JVM bootstrap when the AOT cache is 
> used in an application's production run.
> 
> This PR annotates a single class, `jdk.internal.math.MathUtils` (also the 
> object hierarchy root class, `Object.class`, which has no associated AOT 
> initialization but is required for completeness). More classes will be added 
> in future PRs.
> 
> If a class `K` has the `@AOTInitialize` annotation, the same annotation must 
> be also added to
> - All of `K`'s super classes
> - All of `K`'s super interfaces that require to be initialized when `K` is 
> initialized (see JVMS 5.5. Initialization, step 7; also C++ function 
> `InstanceKlass::interface_needs_clinit_execution_as_super()`
> 
> Note, the check of the above requirement has been moved to 
> `AOTClassInitializer::check_aot_annotations()`. The previous check in 
> `ClassFileParser` was not executed because the class is loaded in the AOT 
> training run, where `CDSConfig::is_initing_classes_at_dump_time()` returns 
> `false` (this function returns `true` only in the AOT assembly phase).
> 
> This annotation is awfully similar to `@AOTSafeClassInitializer`, and I am 
> not sure if we need both. Please see the javadoc in `@AOTInitialize` to see 
> the difference between the two annotations.

Ioi Lam has updated the pull request incrementally with one additional commit 
since the last revision:

  Added logging about @AOTSafeClassInitializer classes that have not been 
initialized

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/27024/files
  - new: https://git.openjdk.org/jdk/pull/27024/files/15798558..24b52a49

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=27024&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27024&range=02-03

  Stats: 5 lines in 1 file changed: 5 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/27024.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/27024/head:pull/27024

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

Reply via email to