Hello -

This fix has been pushed to OpenJDK jdk10/jdk10:

  http://hg.openjdk.java.net/jdk10/jdk10/rev/2fe66ca1e2b3

Tim

On 07/18/17 01:58, Volker Simonis wrote:
You need a sponsor from Oracle because of the generated configure file
which also needs to be generated for the Oracle closed parts.

Can somebody from the build folks please push this?

Thanks,
Volker


On Tue, Jul 18, 2017 at 9:18 AM, Thomas Stüfe <thomas.stu...@gmail.com> wrote:
Danke Volker!

...

So, do I need a sponsor for this or can I push this on my own?

Thanks, Thomas

On Tue, Jul 18, 2017 at 8:58 AM, Volker Simonis <volker.simo...@gmail.com>
wrote:

Looks good!

Thanks,
Volker

On Tue, Jul 18, 2017 at 8:30 AM, Thomas Stüfe <thomas.stu...@gmail.com>
wrote:
Hi Volker,

new webrev:

http://cr.openjdk.java.net/~stuefe/webrevs/8184344-TOC-overflow/webrev.01/webrev/

Only change is the comment, as you suggested.

Kind Regards, Thomas

On Tue, Jul 18, 2017 at 7:46 AM, Thomas Stüfe <thomas.stu...@gmail.com>
wrote:

Thank you Volker!

I'll restore the original comment and prepare a new webrev.

..Thomas

On Mon, Jul 17, 2017 at 5:46 PM, Volker Simonis
<volker.simo...@gmail.com>
wrote:

Hi Thomas,

the change looks good, but I'd prefer if you leave the initial comment
in place which also mentions "-qminimaltoc" as a way of resolving TOC
overflow errors.

I actually don't remember exactly, but I think "-qminimaltoc" works by
creating distinct TOCs for each compilation unit. That comes with an
performance impact, but  "-qpic=large" / "-bbigtoc" can have an
performance impact as well.

As I said, for the slowdebug build your changes are fine. I'd just
like to keep the reference to "-qminimaltoc" for the case where we
have to re-evaluate the different solutions for the product build.

Thank you and best regards,
Volker


On Mon, Jul 17, 2017 at 3:19 PM, Langer, Christoph
<christoph.lan...@sap.com> wrote:
Hi Thomas,

the fix looks ok to me.

I’m copying build-dev because of the changes in
generated-configure.sh.
Don’t know if this can just be pushed from extern or if it needs
some
special handling from Oracle folks or other things to take care of?

Best regards
Christoph

From: ppc-aix-port-dev
[mailto:ppc-aix-port-dev-boun...@openjdk.java.net] On Behalf Of
Thomas Stüfe
Sent: Montag, 17. Juli 2017 12:55
To: ppc-aix-port-...@openjdk.java.net
Subject: RFR(xs): 8184344: [aix] libjvm.so TOC overflow for
slowdebug

Hi all,

may I please have a review for the following fix:

webrev:

http://cr.openjdk.java.net/~stuefe/webrevs/8184344-TOC-overflow/webrev.00/webrev/
Issue: https://bugs.openjdk.java.net/browse/JDK-8184344

Basically, the TOC on AIX overflows on slowdebug builds. I am not
sure
yet which change caused that - I suspect one of the recent
template-metaprogramming changes but have not investigated yet.

Thanks, Thomas







Reply via email to