8038953,Add sanity tests for BMI1 and LZCNT instructions

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Apr 1 17:03:42 UTC 2014


I mean why do you need to create separate tests and not use (add new code) existing one?
For example, you added bmi/sanity/AddnTestI.java when there is already bmi/TestAndnI.java.

Thanks,
Vladimir

On 4/1/14 9:57 AM, Vladimir Kozlov wrote:
> Anton,
>
> How your changes relate to Filipp's?:
>
> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/15d507abfc7a
>
> Thanks,
> Vladimir
>
> On 4/1/14 9:40 AM, Anton Ivanov wrote:
>> Hi all,
>>
>> Please review the patch
>>
>> New tests for using of blsi,blsr,andn,blsmsk,lzcnt,tzcnt instructions were developed.
>> They verify that instructions above are really presented in emitted native code on supported hardware
>>
>> webrev: http://cr.openjdk.java.net/~iignatyev/aaivanov/8038953/webrev.00
>> jbs: https://bugs.openjdk.java.net/browse/JDK-8038953
>> testing: test/compiler/intrinsics/bmi/sanity
>>


More information about the hotspot-compiler-dev mailing list