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

Tom Rodriguez tom.rodriguez at oracle.com
Tue Sep 20 11:13:36 PDT 2011


Fixed.  Thanks.

tom

On Sep 20, 2011, at 10:59 AM, Vladimir Kozlov wrote:

> 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