[9] RFR(S): 8155108: CompilerControl: Enable option set incorrectly

Nils Eliasson nils.eliasson at oracle.com
Thu May 19 11:33:22 UTC 2016


Hi,

Please review,

Summary:
This patches removes the special treatment of "enable"-option in 
directives parsing. In early version Enable was not part of the 
directive but a special case in the parser. This was refactored into 
more general code where the Enable-option was parsed and accessed like 
any other option. Somewhere along the way the old code was not removed 
properly. That code contains a bug where the argument to the 
set-function is casted improperly causing test failures on sparc.

Testing:
All jtreg tests on all platforms. Running.

Bug: https://bugs.openjdk.java.net/browse/JDK-8155108
Webrev: http://cr.openjdk.java.net/~neliasso/8155108/webrev.01/

Best regards,
Nils Eliasson


More information about the hotspot-compiler-dev mailing list