review for 7092236: java/util/EnumSet/EnumSetBash.java fails

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Sep 20 10:59:24 PDT 2011


Good.

+ // other we want to log all the dependences which were
      ^ otherwise?

Vladimir

Tom Rodriguez wrote:
> http://cr.openjdk.java.net/~never/7092236
> 27 lines changed: 17 ins; 7 del; 3 mod; 1161 unchg
> 
> 7092236: java/util/EnumSet/EnumSetBash.java fails
> Reviewed-by:
> 
> The changes for 7083184 moved an early bailout outside of the guarding
> test which caused us to skip checking some dependencies when classes
> are loaded during compilation.  The fix is to move it back inside.  I
> also simplified the logic since I found it quite confusing.  Tested by
> failing test from bug report.
> 


More information about the hotspot-compiler-dev mailing list