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

[Vladimir Kozlov]

Looks good.

Thanks,
Vladimir

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