[jdk8u-dev] Integrated: 8159720: Failure of C2 compilation with tiered prevents some C1 compilations

Yi Yang yyang at openjdk.org
Wed Nov 9 15:34:33 UTC 2022


On Tue, 8 Nov 2022 02:29:19 GMT, Yi Yang <yyang at openjdk.org> wrote:

> 8u backport request: Spark observes a significant performance deterioration between jdk17 and jdk8, too many live locals prevent C2 from compilation and it refreshes all tiers of results, please see [SPARK-40303](https://issues.apache.org/jira/browse/SPARK-40303?focusedCommentId=17619978&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-17619978) for more details and explanations. This fix makes C2 compilation failure not affect C1. Low risk and apply clean.
> 
> Please help review it, thanks in advance.

This pull request has now been integrated.

Changeset: 6e563e59
Author:    Yi Yang <yyang at openjdk.org>
Committer: Paul Hohensee <phh at openjdk.org>
URL:       https://git.openjdk.org/jdk8u-dev/commit/6e563e59f3b4a9b25b4d52975daf4047f52d5b61
Stats:     14 lines in 4 files changed: 0 ins; 4 del; 10 mod

8159720: Failure of C2 compilation with tiered prevents some C1 compilations

If C2 fails to compile a method with tiered compilation, then it should mark the method as not compileable on the C2 tier only.

Reviewed-by: phh
Backport-of: cc10eca0b07f03d4765713d86eae39959f555c1f

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

PR: https://git.openjdk.org/jdk8u-dev/pull/161


More information about the jdk8u-dev mailing list