RFR(S) : 8185536 : @key aot should be replaced w/ @requires vm.aot

Igor Ignatyev igor.ignatyev at oracle.com
Wed Aug 2 17:53:52 UTC 2017


Vladimir, Goetz,

thank you for your review. Although abbreviations should not be upper-cased in method names, I've renamed vmAot to vmAOT to be consistent w/ the rest of the class.

-- Igor

> On Aug 2, 2017, at 5:49 AM, Lindenmaier, Goetz <goetz.lindenmaier at sap.com> wrote:
> 
> Hi Igor, 
> 
> the change is looking good.
> 
> A small thing, I would spell vmAot() as vmAOT(), all abbreviations
> (except for Jvmci) are capitalized here.
> 
> It passed the tests on all platforms that have been running so far.
> (Some had an unrelated issue tonight).
> 
> Best regards,
>  Goetz.
> 
>> -----Original Message-----
>> From: Igor Ignatyev [mailto:igor.ignatyev at oracle.com]
>> Sent: Dienstag, 1. August 2017 17:51
>> To: hotspot-compiler-dev at openjdk.java.net compiler <hotspot-compiler-
>> dev at openjdk.java.net>
>> Cc: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
>> Subject: RFR(S) : 8185536 : @key aot should be replaced w/ @requires
>> vm.aot
>> 
>> http://cr.openjdk.java.net/~iignatyev//8185536/webrev.00/index.html
>>> 81 lines changed: 19 ins; 0 del; 62 mod
>> 
>> Hi all,
>> 
>> could you please review this small patch which introduces 'vm.aot' property
>> into jtreg @requires context and replaces all '@key aot' added by 8185046[1]
>> w/ '@requiers vm.aot'?
>> Goetz(author of 8185046) is checking that it works for their configurations.
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8185536
>> webrev:
>> http://cr.openjdk.java.net/~iignatyev//8185536/webrev.00/index.html
>> testing: hotspot/test/compiler/aot tests
>> - on linux-x64 : 56 tests selected/run
>> - on linux-x86 : 0 tests selected/run
>> - on windows-x64 (after remove tests from problemlist) : 56 tests
>> selected/run
>> - on windows-x86 (after remove tests from problemlist) : 0 tests
>> selected/run
>> 
>> [1] https://bugs.openjdk.java.net/browse/JDK-8185046
>> 
>> Thanks,
>> -- Igor



More information about the hotspot-compiler-dev mailing list