[jdk11u-dev] RFR: 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

> > The GHA report macos-x64 and macos-aarch64 build fails `error: 'sprintf' is deprecated`, which has been recorded by [JDK-8296812](https://bugs.openjdk.org/browse/JDK-8296812). The build fails is unreleated to this PR.
> 
> Hi @sendaoYan , please note that GHA for macos-x64 and macos-aarch64 must pass after #2940 was merged.

Sorry, I forgot update the baseline from jdk11u-dev. I will update and run the GHA test again.

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

PR Comment: https://git.openjdk.org/jdk11u-dev/pull/2951#issuecomment-2412984411