RFR: 8160748: Inconsistent types for ideal_reg

Kim Barrett kim.barrett at oracle.com
Wed Apr 12 05:12:18 UTC 2017


> On Apr 11, 2017, at 10:38 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
> 
> Good.
> 
> Thanks,
> Vladimir

Thanks.

> On 4/11/17 5:42 PM, Kim Barrett wrote:
>>> On Apr 10, 2017, at 9:00 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
>>> 
>>>> On Apr 10, 2017, at 3:43 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>>> 
>>>> Hi, Kim
>>>> 
>>>> Please, also correct vector registers (need to fix *.ad files too):
>>>> 
>>>> // Vector ideal reg
>>>> static const int vector_ideal_reg(int len);
>>>> static const int vector_shift_count_ideal_reg(int len);
>>> 
>>> Oops, I missed those.  Testing that change.  I’ll put out a new webrev when that’s done.
>> 
>> Here are the requested additional fixes for vector_ideal_reg and
>> vector_shift_count_ideal_reg.
>> 
>> full: http://cr.openjdk.java.net/~kbarrett/8160748/hotspot.01/
>> incr: http://cr.openjdk.java.net/~kbarrett/8160748/hotspot.01.inc/
>> 
>> Note that I've not tested the aarch64, ppc, or s390 changes at all.
>> They are pretty simple though, so hopefully no problems.




More information about the hotspot-compiler-dev mailing list