RFR(S) : 8186537 : remove unnecessary @requires from hotspot/compiler/aot tests

Vladimir Kozlov vladimir.kozlov at oracle.com
Mon Aug 21 23:36:57 UTC 2017


Right. No need for @require now.

Thanks,
Vladimir

On 8/21/17 2:55 PM, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8186537/webrev.00/index.html
>> 54 lines changed: 0 ins; 54 del; 0 mod;
> 
> Hi all,
> 
> could you please review this small patch which cleans up @requires in aot tests?
> after JDK-8185536[1], all aot tests are marked w/ '@requires vm.aot', so they will be executed only if a VM under test supports AOT. this made '@requires vm.bits == "64" & (os.arch == "amd64" | os.arch == "x86_64")' in these tests redundant.
> 
> webrev: http://cr.openjdk.java.net/~iignatyev//8186537/webrev.00/index.html
> jbs: https://bugs.openjdk.java.net/browse/JDK-8186537sca
> testing: hotspot/compiler/aot tests on platforms w/ and w/o AOT support. number of selected/run tests is the same as before the fix.
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8185536
> 
> Thanks,
> -- Igor
> 
> 


More information about the hotspot-compiler-dev mailing list