[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