Re: Managing parallel change in JDK 8 and JDK 9

2013-12-10 Thread mark . reinhold
2013/12/2 7:36 -0800, mark.reinh...@oracle.com: > ... > > To simplify the release-endgame process, I hereby make the following > proposal: > > The JDK 9 development forests will be initialized from a specific > build of JDK 8. After that build, merges between the two code lines > will not

Re: Review request for JDK-8027709: JDK8 docs on -XX:CompileOnly option are incorrect

2013-12-10 Thread Vladimir Kozlov
Who did review of other flags description? I see that description of -Xcomp is also incorrect - it does not disable Interpreter, methods still can be executed in it. Also JIT compiler does not execute methods. I think it should be: -Xcomp Force methods compilation on first invoca

jdk8-b120: HotSpot

2013-12-10 Thread Alejandro E Murillo
hs25-b62 has been integrated into jdk8-b120. http://hg.openjdk.java.net/jdk8/jdk8/rev/f204455b60cc http://hg.openjdk.java.net/jdk8/jdk8/corba/rev/53fd772d28c8 http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/fca262db9c43 http://hg.openjdk.java.net/jdk8/jdk8/jaxp/rev/64d8b228a72c http://hg.openjdk

hg: jdk8/jdk8/hotspot: 20 new changesets

2013-12-10 Thread alejandro . murillo
Changeset: b6b9a5d4cda0 Author:amurillo Date: 2013-11-29 11:20 -0800 URL: http://hg.openjdk.java.net/jdk8/jdk8/hotspot/rev/b6b9a5d4cda0 8029367: new hotspot build - hs25-b62 Reviewed-by: jcoomes ! make/hotspot_version Changeset: 77b028ba548c Author:jprovino Date: 2013-11-

jdk8-b120: JSN, Tools, Core Libraries, Serviceability, 2d, Awt, and Swing

2013-12-10 Thread lana . steuck
http://hg.openjdk.java.net/jdk8/jdk8/rev/f204455b60cc http://hg.openjdk.java.net/jdk8/jdk8/nashorn/rev/55cbc2d00c93 http://hg.openjdk.java.net/jdk8/jdk8/langtools/rev/b3d7e86a0647 http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/7ecaa4402c4e http://hg.openjdk.java.net/jdk8/jdk8/jaxws/rev/6c152deb60

Re: RFR: 8029795 : LinkedHashMap.getOrDefault() doesn't update access order. (was Why doesn't new Map methods generate entry accesses on LinkedHashMap?)

2013-12-10 Thread Vitaly Davidovich
I believe C2 compiler is more aggressive about inlining, and will inline hot methods over the 35 bytes limit. I guess interpreter and C1 (and other JVMs, potentially) may get impacted though. In this case, the issue may be more about the fact that getOrDefault() is virtual and not so much about b

hg: jdk8/jdk8/nashorn: 3 new changesets

2013-12-10 Thread lana . steuck
Changeset: e0b4483668a7 Author:jlaskey Date: 2013-11-26 11:58 -0400 URL: http://hg.openjdk.java.net/jdk8/jdk8/nashorn/rev/e0b4483668a7 8029173: Debugger support doesn't handle ConsString Reviewed-by: lagergren, hannesw, sundar Contributed-by: james.las...@oracle.com ! src/jdk/nasho

hg: jdk8/jdk8: 2 new changesets

2013-12-10 Thread lana . steuck
Changeset: c009462c1e92 Author:erikj Date: 2013-12-04 12:45 +0100 URL: http://hg.openjdk.java.net/jdk8/jdk8/rev/c009462c1e92 8027963: Create unlimited policy jars. Reviewed-by: wetmore, ihse ! common/autoconf/spec.gmk.in Changeset: f204455b60cc Author:lana Date: 2013-12-0

hg: jdk8/jdk8/langtools: 6 new changesets

2013-12-10 Thread lana . steuck
Changeset: a746587a1ff1 Author:jlahoda Date: 2013-12-03 18:50 +0100 URL: http://hg.openjdk.java.net/jdk8/jdk8/langtools/rev/a746587a1ff1 8028699: Compiler crash during speculative attribution of annotated type Summary: Moving the checkForDeclarationAnnotations check into Attr.TypeA

hg: jdk8/jdk8/jaxp: 2 new changesets

2013-12-10 Thread lana . steuck
Changeset: aed9ca4d33ec Author:joehw Date: 2013-12-04 00:17 -0800 URL: http://hg.openjdk.java.net/jdk8/jdk8/jaxp/rev/aed9ca4d33ec 8027973: Error in the documentation for newFactory method of the javax.xml.stream factories Reviewed-by: alanb, dfuchs, lancea, rriggs ! src/javax/xml/

Re: RFR: 8029795 : LinkedHashMap.getOrDefault() doesn't update access order. (was Why doesn't new Map methods generate entry accesses on LinkedHashMap?)

2013-12-10 Thread Mike Duigou
I considered it but was worried that the combined size would prevent inlining of the method. The getOrDefault method is currently 33 bytes long which is right at the common borderline for inlining. It does seem a shame to duplicate the code but I couldn't be confident I wouldn't degrade performa

Re: Review request for JDK-8027709: JDK8 docs on -XX:CompileOnly option are incorrect

2013-12-10 Thread alexey zhebel
I am sorry for another email, but the previous one was rejected. I should be a member of the jdk8-dev list now. Best regards, Alexey Zhebel On 10.12.2013 13:05, alexey zhebel wrote: Sorry, here are the instructions. *Scope of Review* _Sanity check content only_ as opposed to reviewing the nro

Re: RFR: 8029795 : LinkedHashMap.getOrDefault() doesn't update access order. (was Why doesn't new Map methods generate entry accesses on LinkedHashMap?)

2013-12-10 Thread Paul Sandoz
On Dec 10, 2013, at 10:47 AM, Paul Sandoz wrote: > > On Dec 10, 2013, at 5:37 AM, Mike Duigou wrote: > >> Hello all; >> >> I've posted a webrev for review which corrects the problem and adds >> appropriate tests. >> >> http://cr.openjdk.java.net/~mduigou/JDK-8029795/0/webrev/ >> >> I also

Re: RFR: 8029795 : LinkedHashMap.getOrDefault() doesn't update access order. (was Why doesn't new Map methods generate entry accesses on LinkedHashMap?)

2013-12-10 Thread Paul Sandoz
On Dec 10, 2013, at 5:37 AM, Mike Duigou wrote: > Hello all; > > I've posted a webrev for review which corrects the problem and adds > appropriate tests. > > http://cr.openjdk.java.net/~mduigou/JDK-8029795/0/webrev/ > > I also updated the documentation to mention that getOrDefault as well as