small tests change

Filipp Zhinkin filipp.zhinkin at oracle.com
Mon Sep 15 13:27:47 UTC 2014


The change looks good.

Thanks,
Filipp.

On 09/15/2014 04:31 PM, Dmitrij Pochepko wrote:
> Hi,
> Filipp have put it:
>
> http://cr.openjdk.java.net/~fzhinkin/dpochepk/8043674/webrev.00/
>
> Thanks,
> Dmitrij
>> Email subject should follow review request style.
>> Changes for review should be sent as webrev on cr.openjdk server. Ask Filipp 
>> or Igor to put it there for you. They also can be your sponsors. Filipp is 
>> these tests author and should review changes too.
>>
>> I am fine with running tests on all platforms.
>>
>> Thanks,
>> Vladimir
>>
>> On 9/12/14 1:38 PM, dmitrij pochepko wrote:
>>> Hi hotspot compiler team,
>>>
>>> I'm a member of java sqe group, and currently i'm proposing a small fix for 
>>> hotspot tests.
>>> A respective Jira issue is https://bugs.openjdk.java.net/browse/JDK-8043674 
>>> - Update compiler/intrinsic/bmi tests to run
>>> it on all platforms.
>>> A diff is attached. A bmi-related tests compare vm work using Xint and Xcomp 
>>> modes(assuming Xcomp uses bmi
>>> instructions). In case of running on another platforms it's just usual 
>>> comparison of Xint and Xcomp modes, which won't
>>> test bmi, however, some platforms might have some bmi-like optimizations in 
>>> future(for example, a powerpc platform have
>>> the some of such instructions), so, it's only natural to allow bmi tests to 
>>> be run anywhere.
>>>
>>> What i am asking is some reviewers for this diff and(in case review is 
>>> successfull) commiter to do the push(since i have
>>> no commiter status).
>>>
>>>
>>> Thanks,
>>> Dmitrij
>



More information about the hotspot-compiler-dev mailing list