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