[jdk11u-dev] Integrated: 8224624: Inefficiencies in CodeStrings::add_comment cause timeouts

[SendaoYan]

On Tue, 15 Oct 2024 02:49:09 GMT, SendaoYan <syan at openjdk.org> wrote:

> Hi all,
> This is prefix backport of [JDK-8231058](https://bugs.openjdk.org/browse/JDK-8231058), after apply this backport will make backport of [JDK-8231058](https://bugs.openjdk.org/browse/JDK-8231058) cleanly.
> This PR improves C1 compiled time of the test `compiler/c1/TestLinearScanOrderMain.java` method dramatically from [92s to 1.8s](https://bugs.openjdk.org/browse/JDK-8224624), and solve this test intermittent timeout.
> 
> Additonal testing:
> 
> - [x] linux aarch64 build with release/slowdebug configure
> - [x] linux x86_64 build with release/slowdebug configure
> - [x] jtreg tests(include tier1/2/3 etc.) with release build on linux aarch64
> - [x] jtreg tests(include tier1/2/3 etc.) with release build on linux x86_64

This pull request has now been integrated.

Changeset: 96805ae9
Author:    SendaoYan <syan at openjdk.org>
URL:       https://git.openjdk.org/jdk11u-dev/commit/96805ae97b5153b48d5fa5d1dac99a3763f9cde9
Stats:     32 lines in 2 files changed: 23 ins; 3 del; 6 mod

8224624: Inefficiencies in CodeStrings::add_comment cause timeouts

Changing CodeStrings to a doubly-linked-list and searching for the comment with the right offset in reverse.

Backport-of: 7cff981f5a3aa192e57545c7df069fb48ba69edf

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

PR: https://git.openjdk.org/jdk11u-dev/pull/2951