> On 24 Nov 2015, at 15:17, Aleksey Shipilev <aleksey.shipi...@oracle.com> > wrote: > > On 11/24/2015 02:07 PM, Aleksey Shipilev wrote: >> On 11/24/2015 03:35 AM, John Rose wrote: >>> On Nov 23, 2015, at 1:42 PM, Aleksey Shipilev >>> <aleksey.shipi...@oracle.com <mailto:aleksey.shipi...@oracle.com>> wrote: >>>> >>>> Okay, here it is (only tests changed): >>>> http://cr.openjdk.java.net/~shade/8136500/webrev.06/ >>>> <http://cr.openjdk.java.net/%7Eshade/8136500/webrev.06/> >>> >>> Thanks; that's a solid test. >>> >>> Cleanups are good too. >>> >>> You can count me as a reviewer. >> >> Thanks for reviews, Ivan, John, Sherman and Paul! >> >> I am pushing this change now. > > Oh no, I don't. Pre-integration JPRT runs exposed an opaque dependency > on Integer.sizeTable field from C2 OptimizeStringConcat. I added the > field declaration back: > http://cr.openjdk.java.net/~shade/8136500/webrev.07/ > > Cleaning up the compiler code would require much more time, so I would > like to push this JDK part now,
Yes, push it! Paul. > and deal with Hotspot compiler later: > https://bugs.openjdk.java.net/browse/JDK-8143900 > > Objections? > > Thanks, > -Aleksey >