RFR(S): 8174202 : jtreg AOT tests should not assume library extension of .so

Igor Ignatyev igor.ignatyev at oracle.com
Fri Aug 11 02:11:06 UTC 2017


Thank you Vladimir.
I've found a couple of copy-paste errors in jdk/tools/jaotc/Main.java, I always used '.so' substringing output filename to object filename. 

> --- a/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/Main.java	Thu Aug 10 14:23:51 2017 -0700
> +++ b/src/jdk.aot/share/classes/jdk.tools.jaotc/src/jdk/tools/jaotc/Main.java	Thu Aug 10 19:07:31 2017 -0700
> @@ -577,7 +577,7 @@
>                      break;
>                  case "Mac OS X":
>                      if (name.endsWith(".dylib")) {
> -                        objectFileName = name.substring(0, name.length() - ".so".length());
> +                        objectFileName = name.substring(0, name.length() - ".dylib".length());
>                      }
>                      objectFileName = objectFileName + ".o";
>                      linkerPath = (options.linkerpath != null) ? options.linkerpath : "ld";
> @@ -586,7 +586,7 @@
>                  default:
>                      if (osName.startsWith("Windows")) {
>                          if (name.endsWith(".dll")) {
> -                            objectFileName = name.substring(0, name.length() - ".so".length());
> +                            objectFileName = name.substring(0, name.length() - ".dll".length());
>                          }
>                          objectFileName = objectFileName + ".obj";
>                          linkerPath = (options.linkerpath != null) ? options.linkerpath : getWindowsLinkPath();

Thanks,
-- Igor

> On Aug 10, 2017, at 5:24 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> Good.
> 
> Thanks,
> Vladimir
> 
> On 8/10/17 4:33 PM, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8174202/webrev.00/index.html
>>> 67 lines changed: 47 ins; 11 del; 9 mod;
>> Hi all,
>> could you please review this small fix?
>> problem: some of aot tests assumed that output file is named unnamed.so regardless the platform. the problem was worked around in jaotc tool.
>> fix: the workaround was removed and now we use platform-specific extension for default output name. it's unnamed.dll on windows, unnamed.dylib on mac and unnamed.so on all other platforms.
>> the tests have been updated correspondingly.
>> webrev: http://cr.openjdk.java.net/~iignatyev//8174202/webrev.00/index.html
>> jbs: https://bugs.openjdk.java.net/browse/JDK-8174202
>> testing: hotspot/compiler/aot (w/ 8183337[1] applied) on all supported platforms
>> [1] http://cr.openjdk.java.net/~iignatyev//8183337/webrev.00/index.html
>> Thanks,
>> -- Igor



More information about the hotspot-compiler-dev mailing list