8186721: AOT tests fail with: section alignment is not valid: 128
authorkvn
Thu, 24 Aug 2017 13:11:02 -0700
changeset 46980 28e20e703392
parent 46975 59d4586da7bc
child 46981 add50afae9e1
8186721: AOT tests fail with: section alignment is not valid: 128 Summary: add missing negation in assert chech, add -ea -esa to AOT testing Reviewed-by: dlong
hotspot/src/jdk.aot/share/classes/jdk.tools.jaotc.binformat/src/jdk/tools/jaotc/binformat/pecoff/PECoffSection.java
hotspot/test/compiler/aot/AotCompiler.java
--- a/hotspot/src/jdk.aot/share/classes/jdk.tools.jaotc.binformat/src/jdk/tools/jaotc/binformat/pecoff/PECoffSection.java	Thu Aug 24 14:00:04 2017 +0000
+++ b/hotspot/src/jdk.aot/share/classes/jdk.tools.jaotc.binformat/src/jdk/tools/jaotc/binformat/pecoff/PECoffSection.java	Thu Aug 24 13:11:02 2017 -0700
@@ -50,7 +50,7 @@
         byte[] Name = sectName.getBytes();
         int max = Name.length <= IMAGE_SECTION_HEADER.Name.sz ? Name.length : IMAGE_SECTION_HEADER.Name.sz;
 
-        assert (sectAlign < 1 || sectAlign > 1024 || (sectAlign & (sectAlign - 1)) != 0) : "section alignment is not valid: " + sectAlign;
+        assert !(sectAlign < 1 || sectAlign > 1024 || (sectAlign & (sectAlign - 1)) != 0) : "section alignment is not valid: " + sectAlign;
         align = sectAlign;
 
         // Using 32 because IMAGE_SCN_ALIGN_*BYTES is value + 1
--- a/hotspot/test/compiler/aot/AotCompiler.java	Thu Aug 24 14:00:04 2017 +0000
+++ b/hotspot/test/compiler/aot/AotCompiler.java	Thu Aug 24 13:11:02 2017 -0700
@@ -113,6 +113,9 @@
             args.add("--linker-path");
             args.add(linker);
         }
+        // Execute with asserts
+        args.add("-J-ea");
+        args.add("-J-esa");
         return launchJaotc(args, extraopts);
     }