RFR(XS) : 8187020 : AOT tests should not fail if devkit dependency isn't resolved

Igor Ignatyev igor.ignatyev at oracle.com
Fri Sep 1 18:07:22 UTC 2017


H Vladimir,

I was planning to add execution of ld -v by a separate patch, but if you want I can merge it into this one. 

-- Igor

> On Aug 31, 2017, at 2:12 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> On MacOS we also have problem when 'ld' is present on path but Xcode is missing:
> 
> % /usr/bin/ld -v
> xcode-select: error: no developer tools were found at '/Applications/Xcode.app', and no install could be requested (perhaps no UI is present), please install manually from 'developer.apple.com'.
> 
> AOT tests will fail in such case too. May be AotCompiler test code also have to execute ld -v (or ld -V on Solaris) to check that we can use linker similar to JAOTC tool:
> 
> http://hg.openjdk.java.net/jdk10/hs/hotspot/file/a8ec32aa385e/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/Linker.java#l104
> 
> Vladimir
> 
> On 8/31/17 11:02 AM, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8187020/webrev.00/index.html
>>> 11 lines changed: 5 ins; 0 del; 6 mod;
>> Hi all,
>> could you please review this tiny fix or AOT tests? Prior this fix, the tests would fail w/ FileNotFoundException when devkit dependency can't be resolved even if jaotc is able to find a linker, e.g. in one of VS*COMNTOOLS. after this fix, if tests aren't able to download devkit, they will allow jaotc to try to find a linker. This patch also remove search for 'ld' in PATH on windows b/c jaotc doesn't use it on windows.
>> webrev: http://cr.openjdk.java.net/~iignatyev//8187020/webrev.00/index.html
>> jbs: https://bugs.openjdk.java.net/browse/JDK-8187020
>> testing: compiler/aot tests
>> Thanks,
>> -- Igor



More information about the hotspot-compiler-dev mailing list