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

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Jul 11 00:08:28 UTC 2018


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