RFR(XS) 8161292: [JVMCI] missing test files from 8159368
Tom Rodriguez
tom.rodriguez at oracle.com
Tue Jul 19 18:23:02 UTC 2016
Thanks!
tom
> On Jul 18, 2016, at 4:32 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>
> Good.
>
> Thanks,
> Vladimir
>
> On 7/18/16 10:42 AM, Tom Rodriguez wrote:
>> The native calling convention was broken on windows and this also
>> exposed general problems with the AMD64TestAssembler. Windows native
>> calling only ever uses registers for the first 4 arguments so logic had
>> to be added to AMD64HotSpotRegisterConfig to support this. Additionally
>> the test assembler wasn’t computing the location of the integer stack
>> slots properly and the argument loading needs to use scratch registers.
>> I expanded the test to have cases that pass 32 of float, double, int
>> and long to make sure that extreme cases are exercised on all platforms.
>> This properly passes JPRT.
>>
>> The changes relative to the original review
>> are http://cr.openjdk.java.net/~never/8161292.diff/webrev/ and the full
>> webrev is http://cr.openjdk.java.net/~never/8161292.v2/webrev/.
>>
>> tom
>>
>>> On Jul 13, 2016, at 4:44 PM, Tom Rodriguez <tom.rodriguez at oracle.com
>>> <mailto:tom.rodriguez at oracle.com <mailto:tom.rodriguez at oracle.com>>> wrote:
>>>
>>>
>>>> On Jul 13, 2016, at 12:19 PM, Tom Rodriguez <tom.rodriguez at oracle.com <mailto:tom.rodriguez at oracle.com>
>>>> <mailto:tom.rodriguez at oracle.com <mailto:tom.rodriguez at oracle.com>>> wrote:
>>>>
>>>>
>>>>> On Jul 13, 2016, at 11:49 AM, Christian Thalinger
>>>>> <cthalinger at twitter.com <mailto:cthalinger at twitter.com> <mailto:cthalinger at twitter.com <mailto:cthalinger at twitter.com>>> wrote:
>>>>>
>>>>> Looks good. How did it pass JPRT?
>>>>
>>>> Thanks. It’s a completely new test.
>>>>
>>>> tom
>>>
>>> Sending it through JPRT exposed a bug where the native calling
>>> convention isn’t properly described on windows. There are only 4 XMM
>>> registers used there. Since the native calling convention is really
>>> only used for things which take at most 2 floats, we never noticed.
>>> I’m fixing that now and will send out a new webrev in a bit.
>>>
>>> tom
>>>
>>>>
>>>>>
>>>>>> On Jul 13, 2016, at 5:55 AM, Tom Rodriguez
>>>>>> <tom.rodriguez at oracle.com <mailto:tom.rodriguez at oracle.com> <mailto:tom.rodriguez at oracle.com <mailto:tom.rodriguez at oracle.com>>> wrote:
>>>>>>
>>>>>> http://cr.openjdk.java.net/~never/8161292/webrev <http://cr.openjdk.java.net/~never/8161292/webrev>
>>>>>> <http://cr.openjdk.java.net/~never/8161292/webrev <http://cr.openjdk.java.net/~never/8161292/webrev>>
>>>>>>
>>>>>> I missed some test files file the push for 8159368. They are
>>>>>> unchanged from the review. Can I get a quick review?
>>>>>>
>>>>>> tom
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20160719/333d39a7/attachment-0001.html>
More information about the hotspot-compiler-dev
mailing list