RFR(XXS) 8232162: Object reallocation in Deoptimization::fetch_unroll_info_helper should not depend on EliminateNestedLocks

[Reingruber, Richard]

Hi Nils and Tobias,

thanks for your reviews.

I will push the change tomorrow.

Best regards, Richard.

-----Original Message-----
From: hotspot-compiler-dev <hotspot-compiler-dev-bounces at openjdk.java.net> On Behalf Of Nils Eliasson
Sent: Dienstag, 15. Oktober 2019 12:28
To: hotspot-compiler-dev at openjdk.java.net
Subject: Re: RFR(XXS) 8232162: Object reallocation in Deoptimization::fetch_unroll_info_helper should not depend on EliminateNestedLocks

+1

// Nils

On 2019-10-11 11:22, Tobias Hartmann wrote:
> Hi Richard,
>
> looks good to me.
>
> Bets regards,
> Tobias
>
> On 11.10.19 10:41, Reingruber, Richard wrote:
>> Hi,
>>
>> could I please get reviews for
>>
>> Webrev: http://cr.openjdk.java.net/~rrich/webrevs/2019/8232162/webrev.0/
>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8232162
>>
>> The fix removes the C2 flag EliminateNestedLocks from the condition for reallocating objects,
>> because C2 eliminates allocations depending only on DoEscapeAnalysis && EliminateAllocations.
>>
>> I put the patch through SAP's nightly testing (includes all platforms, jtreg tests, and more).
>>
>> Thanks, Richard.
>>