[9] RFR(L): 8034812: remove IDX_INIT macro hack in Node class
Lindenmaier, Goetz
goetz.lindenmaier at sap.com
Fri May 16 09:23:32 UTC 2014
Hi Tobias,
attached the patch for ppc.ad. It worked fine except for what
Igor detected.
The patch applies to the original ppc.ad version, not on top
of your change.
Tell me in case the patch was stripped by some mailing program,
I assume this happens for the mailing list.
Best regards,
Goetz.
-----Original Message-----
From: Tobias Hartmann [mailto:tobias.hartmann at oracle.com]
Sent: Freitag, 16. Mai 2014 10:29
To: Lindenmaier, Goetz; Igor Veresov
Cc: hotspot-compiler-dev at openjdk.java.net
Subject: Re: [9] RFR(L): 8034812: remove IDX_INIT macro hack in Node class
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.
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ppc.ad.patch
Type: application/octet-stream
Size: 20631 bytes
Desc: ppc.ad.patch
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140516/f4d89847/ppc.ad-0001.patch>
More information about the hotspot-compiler-dev
mailing list