[11u] RFR: 8059241: C2: Excessive RemoveUseless passes during incremental inlining
Andrew Haley
aph at redhat.com
Fri Mar 26 09:41:53 UTC 2021
On 3/26/21 9:21 AM, Roland Westrelin wrote:
>
>>> I’d like to request a review for this backport of JDK-8059241
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8059241
>>> Webrev: http://cr.openjdk.java.net/~dlemmond/8059241/webrev.11u.00/
>>
>> Ouch. We'd need a C2 specialist to have a very close look at the risk
>> of backporting this.
>
> This should mostly affect workloads that make heavy use of
> invokedynamic. That alone would decrease the risk. The fix has been
> integrated a while ago so it's well tested. I would say the change
> itself is clearly not trivial but fairly low risk.
Okay. Will you review this, please? And please consider the likely
performance gain v.s churn and risk.
--
Andrew Haley (he/him)
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
https://keybase.io/andrewhaley
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671
More information about the jdk-updates-dev
mailing list