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

Tobias Hartmann tobias.hartmann at oracle.com
Fri May 16 08:45:21 UTC 2014


On 16.05.2014 10:41, Volker Simonis wrote:
> 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.

Okay, indeed.

Thanks,
Tobias

>
>> 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