[jdk17u-dev] Integrated: 8322583: RISC-V: Enable fast class initialization checks

[Gui Cao]

On Mon, 8 Jan 2024 13:15:47 GMT, Gui Cao <gcao at openjdk.org> wrote:

> Hi, The same issue also exists in the JDK17u. Since corresponding fixes for the other ports are already there in jdk17u, I would like to backport this to jdk17u too. So I would like to backport this to JDK17u. Tier1-3 tested with release build using qemu systems. This is a risc-v specific change. Backport is clean, risk is low.
> 
> The original performance issue reported by JDK-8219233 is resolved when this optimization is enabled (x2.1 improvement for reported case on linux-riscv64).
> 
> OpenJDK17 linux-riscv64 when setting VM_Version::supports_fast_class_init_checks returns false(default):
> 
> 
> $ clojure -A:user
> "Elapsed time: 78459.498421 msecs"
> $ clojure -A:user
> "Elapsed time: 77513.115408 msecs"
> $ clojure -A:user
> "Elapsed time: 75132.908155 msecs"
> $ clojure foo.clj
> "Elapsed time: 36337.585315 msecs"
> $ clojure foo.clj
> "Elapsed time: 35009.33778 msecs"
> $ clojure foo.clj
> "Elapsed time: 35490.216435 msecs"
> 
> 
> OpenJDK17 linux-riscv64 when setting VM_Version::supports_fast_class_init_checks returns true:
> 
> 
> $ clojure -A:user
> "Elapsed time: 37013.598933 msecs"
> $ clojure -A:user
> "Elapsed time: 36210.164686 msecs"
> $ clojure -A:user
> "Elapsed time: 36674.069441 msecs"
> $ clojure foo.clj
> "Elapsed time: 36813.877226 msecs"
> $ clojure foo.clj
> "Elapsed time: 37079.673782 msecs"

This pull request has now been integrated.

Changeset: 262b83c8
Author:    Gui Cao <gcao at openjdk.org>
Committer: Fei Yang <fyang at openjdk.org>
URL:       https://git.openjdk.org/jdk17u-dev/commit/262b83c84ee29bec2a067ab02d6979fba5b7ae55
Stats:     3 lines in 1 file changed: 3 ins; 0 del; 0 mod

8322583: RISC-V: Enable fast class initialization checks

Backport-of: 5235cc987d8c4455622acda947bed7321086a385

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

PR: https://git.openjdk.org/jdk17u-dev/pull/2110