[9] RFR(L): 8034812: remove IDX_INIT macro hack in Node class

Volker Simonis volker.simonis at gmail.com
Fri May 16 08:41:25 UTC 2014


On Fri, May 16, 2014 at 10:28 AM, Tobias Hartmann
<tobias.hartmann at oracle.com> wrote:
> Vladimir, Goetz, thanks for the feedback.
>
>
> On 16.05.2014 10:01, Lindenmaier, Goetz wrote:
>>
>> Hi Tobias,
>>
>> thanks for editing ppc!  I'll test the change and also supply a fix
>> for the issue mentioned by Igor.  Is it ok if I just send a patch with
>> the diffs?
>
>
> I fixed it, please find the new webrev at
> http://cr.openjdk.java.net/~anoll/8034812/webrev.01/
>
>
>> Best regards,
>>    Geotz.
>>
>> -----Original Message-----
>> From: hotspot-compiler-dev
>> [mailto:hotspot-compiler-dev-bounces at openjdk.java.net] On Behalf Of Igor
>> Veresov
>> Sent: Donnerstag, 15. Mai 2014 21:05
>> To: Tobias Hartmann
>> Cc: hotspot-compiler-dev at openjdk.java.net
>> Subject: Re: [9] RFR(L): 8034812: remove IDX_INIT macro hack in Node class
>>
>> In ppc.ad you did s/new (C)//g instead of s/new (C)/new/g.
>> Otherwise seems alright.
>
>
> Thank you! I wonder why this was not caught by JPRT.
>

Because JPRT doesn't build 64-bit PPC.

> Best,
> Tobias
>
>
>> igor
>>
>> On May 15, 2014, at 11:53 AM, Tobias Hartmann <tobias.hartmann at oracle.com>
>> wrote:
>>
>>> Hi,
>>>
>>> please review the following patch that removes the IDX_INIT macro used by
>>> Node::Node(...) to retrieve the Compile object and replaces it by a call to
>>> Compile::current(). The Node constructor, new operator and all calls to it
>>> are adapted accordingly.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8034812
>>> Webrev: http://cr.openjdk.java.net/~anoll/8034812/webrev.00/
>>>
>>> Tested with JPRT.
>>>
>>> Thanks,
>>> Tobias
>
>


More information about the hotspot-compiler-dev mailing list