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

Vladimir Kozlov vladimir.kozlov at oracle.com
Thu May 19 18:33:10 UTC 2016

Looks good.


On 5/19/16 4:33 AM, Nils Eliasson wrote:
> 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