Request for review: 7104173 sun/tools tests fail with debug build after 7012206

Coleen Phillimore coleen.phillimore at oracle.com
Thu Oct 27 04:14:48 PDT 2011


Looks good to me too.
Thanks,
Coleen

On 10/27/2011 3:23 AM, Frederic Parain wrote:
> Looks good.
>
> Fred
>
> On 10/27/11 4:36 AM, David Holmes wrote:
>> Did I forget to say this is a trivial review? ;-)
>>
>> Thanks,
>> David
>>
>> On 26/10/2011 4:33 PM, David Holmes wrote:
>>> webrev:
>>>
>>> http://cr.openjdk.java.net/~dholmes/7104173/webrev/
>>>
>>> A trivial change for our embedded code to aid in testing. Some tests
>>> need to be passed -XX:+UsePerfData to make them work (7012206). A debug
>>> build, by default, prints any -XX options passed to it. Any such tests
>>> that parse the test output then fail with a debug build because of this
>>> extra unexpected output.
>>>
>>> So for embedded only we simply turn PrintVMOptions off by default for
>>> all builds.
>>>
>>> Thanks,
>>> David
>



More information about the hotspot-runtime-dev mailing list