8038098: test/compiler/codegen/LoadWithMask2.java tags changes to also test C1 inlining

Evgeniya Stepanova evgeniya.stepanova at oracle.com
Mon Sep 15 11:42:08 UTC 2014


Hi all!

Please review fix for 8038098

bug: https://bugs.openjdk.java.net/browse/JDK-8038098

Problem: Since we have inlining in C1 now, we should check that issue 
8031743 will not appear in C1.
Solution: Added one more run in test for C1 inlining checking

A am also need someone to push changes.

Diff is in the attachments.
I've also included diff to this letter since it is very small.


Thanks,
Jane

diff -r 087443edb74a test/compiler/codegen/LoadWithMask2.java
--- a/test/compiler/codegen/LoadWithMask2.java	Mon Sep 15 08:08:22 2014 
+0200
+++ b/test/compiler/codegen/LoadWithMask2.java	Mon Sep 15 14:35:50 2014 
+0400
@@ -26,6 +26,7 @@
   * @bug 8031743
   * @summary loadI2L_immI broken for negative memory values
   * @run main/othervm -server -Xbatch -XX:-TieredCompilation 
-XX:CompileCommand=compileonly,*.foo* LoadWithMask2
+ * @run main/othervm -client -Xbatch -XX:+TieredCompilation 
-XX:CompileCommand=compileonly,*.foo* LoadWithMask2
   *
   */
  public class LoadWithMask2 {
-------------- next part --------------
A non-text attachment was scrubbed...
Name: diff
Type: application/octet-stream
Size: 577 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140915/e9574ac4/diff.obj>


More information about the hotspot-compiler-dev mailing list