RFR(S): 8206075: add assertion for unbound assembler Labels for x86

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Jul 11 01:33:58 UTC 2018


I hit new assert in few other tests:

compiler/codegen/TestCharVect2.java
compiler/c2/cr6340864/*

Regards,
Vladimir

On 7/10/18 5:08 PM, Vladimir Kozlov wrote:
> Fix looks reasonable. I will test it in our framework.
> 
> Thanks,
> Vladimir
> 
> On 7/10/18 9:50 AM, Liu Xin wrote:
>> Hi, Community,
>> Could you please review this small patch?
>> Bug:  https://bugs.openjdk.java.net/browse/JDK-8206075 
>> <https://bugs.openjdk.java.net/browse/JDK-8206075>
>> CR:  http://cr.openjdk.java.net/~phh/8206075/webrev.00/ 
>> <http://cr.openjdk.java.net/~phh/8206075/webrev.00/>
>> Problem:
>> X86-32/64 will leave an unbound label if UseOnStackReplacement is OFF.
>> This patch align up x86 with other architectures(ppc, arm).
>> Add an assertion to the destructor of Label. It  will be wiped out in release build.
>> Previously, hotspot cannot pass this test with assertion on x86-64.
>> make run-test TEST=test/hotspot/jtreg/compiler/c1/Test7090976.java
>> If this CR is approved, Paul Hohensee will push it.
>> Thanks,
>> --lx
>>


More information about the hotspot-runtime-dev mailing list