> On 16 Jun 2016, at 15:17, Michael Haupt <michael.ha...@oracle.com> wrote: > > Dear all, > > please review this change. > RFE: https://bugs.openjdk.java.net/browse/JDK-8143211 > Webrev: http://cr.openjdk.java.net/~mhaupt/8143211/webrev.00/ >
InvokerBytecodeGenerator — I agree with Vladimir on the loop state held by InvokerBytecodeGenerator. 758 i = i+2; // Jump to the end of GWC idiom 759 continue; 760 case TRY_FINALLY: 761 assert isTryFinally(i); 762 onStack = emitTryFinally(i); 763 i += 2; // jump to the end of the TF idiom Might as wall fix up #759 to be consistent. MethodHandleImpl — 1762 LambdaForm form = makeLoopForm(type.basicType(), 1763 init.stream().map(h -> h.type().returnType()). 1764 map(BasicType::basicTypeChar). 1765 reduce("", (s, c) -> s + c, (s, t) -> s + t)); map to string and use the joining collector? map(rt -> String.valueOf(BasicType.basicTypeChar(rt)). collect(Collectors.joining()); Paul. > The change puts the tryFinally and loop method handle combinator > implementations, which were introduced as part of the JEP 274 effort, on a > LambdaForm basis, which executes in bytecode generating (default) and > LambdaForm interpretation > (-Djava.lang.invoke.MethodHandle.COMPILE_THRESHOLD=-1) modes. It also changes > the output formatting of LambdaForms, introducing a (hopefully) more readable > format. > > Thanks, > > Michael > > -- > > <http://www.oracle.com/> > Dr. Michael Haupt | Principal Member of Technical Staff > Phone: +49 331 200 7277 | Fax: +49 331 200 7561 > Oracle Java Platform Group | LangTools Team | Nashorn > Oracle Deutschland B.V. & Co. KG | Schiffbauergasse 14 | 14467 Potsdam, > Germany > > ORACLE Deutschland B.V. & Co. KG | Hauptverwaltung: Riesstraße 25, D-80992 > München > Registergericht: Amtsgericht München, HRA 95603 > > Komplementärin: ORACLE Deutschland Verwaltung B.V. | Hertogswetering 163/167, > 3543 AS Utrecht, Niederlande > Handelsregister der Handelskammer Midden-Nederland, Nr. 30143697 > Geschäftsführer: Alexander van der Ven, Jan Schultheiss, Val Maher > <http://www.oracle.com/commitment> Oracle is committed to developing > practices and products that help protect the environment >