RFR(XL) 8186158: Update Graal

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR(XL) 8186158: Update Graal

Igor Veresov
This is a Graal update. The list of changesets included is listed in the JBS entry.

JBS: https://bugs.openjdk.java.net/browse/JDK-8186158
Webrev: http://cr.openjdk.java.net/~iveresov/8186158/

Thanks,
igor
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(XL) 8186158: Update Graal

Vladimir Kozlov
Igor,

Changes in org.graalvm.graphio files added trailing spaces!
Are you running jcheck?

Vladimir

On 8/11/17 3:18 PM, Igor Veresov wrote:
> This is a Graal update. The list of changesets included is listed in the JBS entry.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8186158
> Webrev: http://cr.openjdk.java.net/~iveresov/8186158/
>
> Thanks,
> igor
>
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(XL) 8186158: Update Graal

Igor Veresov
No yet. I’ll fix those.

igor

> On Aug 11, 2017, at 3:35 PM, Vladimir Kozlov <[hidden email]> wrote:
>
> Igor,
>
> Changes in org.graalvm.graphio files added trailing spaces!
> Are you running jcheck?
>
> Vladimir
>
> On 8/11/17 3:18 PM, Igor Veresov wrote:
>> This is a Graal update. The list of changesets included is listed in the JBS entry.
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8186158
>> Webrev: http://cr.openjdk.java.net/~iveresov/8186158/
>> Thanks,
>> igor

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(XL) 8186158: Update Graal

Igor Veresov
Webrev update in-place. I’ll also push the whitespace fixes upstream.

igor

> On Aug 11, 2017, at 3:50 PM, Igor Veresov <[hidden email]> wrote:
>
> No yet. I’ll fix those.
>
> igor
>
>> On Aug 11, 2017, at 3:35 PM, Vladimir Kozlov <[hidden email]> wrote:
>>
>> Igor,
>>
>> Changes in org.graalvm.graphio files added trailing spaces!
>> Are you running jcheck?
>>
>> Vladimir
>>
>> On 8/11/17 3:18 PM, Igor Veresov wrote:
>>> This is a Graal update. The list of changesets included is listed in the JBS entry.
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8186158
>>> Webrev: http://cr.openjdk.java.net/~iveresov/8186158/
>>> Thanks,
>>> igor
>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(XL) 8186158: Update Graal

Vladimir Kozlov
Good.

Thanks,
Vladimir

On 8/11/17 4:04 PM, Igor Veresov wrote:

> Webrev update in-place. I’ll also push the whitespace fixes upstream.
>
> igor
>
>> On Aug 11, 2017, at 3:50 PM, Igor Veresov <[hidden email]> wrote:
>>
>> No yet. I’ll fix those.
>>
>> igor
>>
>>> On Aug 11, 2017, at 3:35 PM, Vladimir Kozlov <[hidden email]> wrote:
>>>
>>> Igor,
>>>
>>> Changes in org.graalvm.graphio files added trailing spaces!
>>> Are you running jcheck?
>>>
>>> Vladimir
>>>
>>> On 8/11/17 3:18 PM, Igor Veresov wrote:
>>>> This is a Graal update. The list of changesets included is listed in the JBS entry.
>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8186158
>>>> Webrev: http://cr.openjdk.java.net/~iveresov/8186158/
>>>> Thanks,
>>>> igor
>>
>
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(XL) 8186158: Update Graal

Igor Veresov
Thanks!

igor

> On Aug 11, 2017, at 5:21 PM, Vladimir Kozlov <[hidden email]> wrote:
>
> Good.
>
> Thanks,
> Vladimir
>
> On 8/11/17 4:04 PM, Igor Veresov wrote:
>> Webrev update in-place. I’ll also push the whitespace fixes upstream.
>> igor
>>> On Aug 11, 2017, at 3:50 PM, Igor Veresov <[hidden email]> wrote:
>>>
>>> No yet. I’ll fix those.
>>>
>>> igor
>>>
>>>> On Aug 11, 2017, at 3:35 PM, Vladimir Kozlov <[hidden email]> wrote:
>>>>
>>>> Igor,
>>>>
>>>> Changes in org.graalvm.graphio files added trailing spaces!
>>>> Are you running jcheck?
>>>>
>>>> Vladimir
>>>>
>>>> On 8/11/17 3:18 PM, Igor Veresov wrote:
>>>>> This is a Graal update. The list of changesets included is listed in the JBS entry.
>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8186158
>>>>> Webrev: http://cr.openjdk.java.net/~iveresov/8186158/
>>>>> Thanks,
>>>>> igor
>>>

Loading...