RFR(M) 8072383: resolve conflicts between open and closed ports
Dean Long
dean.long at oracle.com
Thu Feb 19 21:40:17 UTC 2015
Thanks Volker, I will list you as a reviewer for 8u60. Did you also
look at the 9
changes? If so I will list you as a reviewer for 9 as well.
dl
On 2/19/2015 2:27 AM, Volker Simonis wrote:
> Hi Dean,
>
> I've just checked webrev.8u60.01 on Linux/PPC64. It cleanly builds and
> runs so thumbs up from me.
>
> Regards,
> Volker
>
>
> On Thu, Feb 19, 2015 at 5:12 AM, Dean Long <dean.long at oracle.com> wrote:
>> Yes, good catch. I've update the webrev to webrev.8u60.01. Thanks for the
>> review.
>>
>> dl
>>
>>
>> On 2/18/2015 4:45 PM, Vladimir Kozlov wrote:
>>> Did you missed to remove #elif ppc_32 in 8u60 changes in
>>> src/share/vm/interpreter/templateTable.hpp?
>>>
>>> Otherwise both versions look good.
>>>
>>> Thanks,
>>> Vlaidmir
>>>
>>> On 2/18/15 4:26 PM, Dean Long wrote:
>>>> Just to avoid confusion, let me clarify that the jdk9 changes below
>>>> can't be pushed until after the open aarch64 port is merged into the
>>>> main jdk9, as they are based on the staging repo.
>>>>
>>>> dl
>>>>
>>>> On 2/18/2015 4:02 PM, Dean Long wrote:
>>>>> These changes resolve some issues with references to closed ports in
>>>>> open hotspot code,
>>>>> primarily by removing those references completely. I have included
>>>>> the 8u60 backport
>>>>> as well because it won't apply cleanly, and I may push it first
>>>>> because the 9 changes are
>>>>> blocked by JEP 237: Linux/AArch64 Port.
>>>>>
>>>>> http://cr.openjdk.java.net/~dlong/8072383/webrev.8u60.00/
>>>>> http://cr.openjdk.java.net/~dlong/8072383/webrev.9.00/
>>>>>
>>>>> dl
>>>>
More information about the hotspot-compiler-dev
mailing list