Integrated: 8304138: [JVMCI] Test FailedSpeculation existence before appending.

Sat Mar 18 09:45:31 UTC 2023

On Tue, 14 Mar 2023 14:55:18 GMT, Yudi Zheng <yzheng at openjdk.org> wrote:

> Upon uncommon_trap, JVMCI runtime appends a FailedSpeculation entry to the nmethod using an [atomic operation](https://github.com/openjdk/jdk/blob/55aa122462c34d8f4cafa58f4d1f2d900449c83e/src/hotspot/share/oops/methodData.cpp#L852). It becomes a performance bottleneck when there is a large amount of (virtual) threads deoptimizing in the nmethod. In this PR, we test if a FailedSpeculation exists in the list before appending it.

This pull request has now been integrated.

Changeset: 7503ecc0
Author:    Yudi Zheng <yzheng at openjdk.org>
Committer: Doug Simon <dnsimon at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7503ecc0f185f6da777c022a66d7af6c40dcd05f
Stats:     28 lines in 1 file changed: 19 ins; 9 del; 0 mod

8304138: [JVMCI] Test FailedSpeculation existence before appending.

Reviewed-by: kvn, dnsimon

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

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