Re: Invokedynamic slower than reflection?

2010-07-31 Thread Stephen Bannasch
Eric's test Hello.java works OK, but your test, Rémi, fails (Fib4.java). So it is a second bug. Stephen, I left a comment: http://gist.github.com/500808 Gist of the gist is: -XX:+UnlockExperimentalVMOptions -XX:+EnableInvokeDynamic Thanks John, I figured it was something simple like that

Auto Reply: Re: Invokedynamic slower than reflection?

2010-07-30 Thread henrik . osterdahl
Hello, I'm out of the office between July 19 and August 13. Christian Törnqvist (christian.tornqv...@oracle.com) will be substituting for me. Please make sure to include him in mail conversations and telephone conferences. Regards, Henrik Österdahl JRockit Codegen Runtime Dev Lead

Re: Invokedynamic slower than reflection?

2010-07-30 Thread Rémi Forax
Le 30/07/2010 08:40, Chanwit Kaewkasi a écrit : Hi Rémi, I did freshly build mlvm and run Eric's test. Here's my result: Duration invokedynamic: 103 Duration reflection: 1463 The patch seems to be working for me as InvokeDynamic is clearly faster than Reflection on my machine. Cheers,

Re: Invokedynamic slower than reflection?

2010-07-30 Thread John Rose
Eric's test Hello.java works OK, but your test, Rémi, fails (Fib4.java). So it is a second bug. Stephen, I left a comment: http://gist.github.com/500808 Gist of the gist is: -XX:+UnlockExperimentalVMOptions -XX:+EnableInvokeDynamic -- John On Jul 30, 2010, at 8:30 AM, Rémi Forax wrote:

Re: Invokedynamic slower than reflection?

2010-07-28 Thread John Rose
On Jul 27, 2010, at 10:11 AM, Rémi Forax wrote: Le 16/07/2010 00:48, John Rose a écrit : It was some sort of bitrot. I pushed a fix for this to mlvm/hotspot. Thanks for the reports! -- John Is it enable by default, it doesn't seem to work ? The fix is titled this way: 6969574:

Re: Invokedynamic slower than reflection?

2010-07-28 Thread Rémi Forax
Le 29/07/2010 01:26, John Rose a écrit : On Jul 28, 2010, at 4:26 PM, Rémi Forax wrote: My question was more, is it enable in mlvm if I use default guards. Yes. In the series file, you'll see that it has the same guards (-/meth +hexstuff) as the other patches:

Re: Invokedynamic slower than reflection?

2010-07-27 Thread Rémi Forax
Le 16/07/2010 00:48, John Rose a écrit : It was some sort of bitrot. I pushed a fix for this to mlvm/hotspot. Thanks for the reports! -- John Is it enable by default, it doesn't seem to work ? Rémi ___ mlvm-dev mailing list

Re: Invokedynamic slower than reflection?

2010-07-15 Thread John Rose
It was some sort of bitrot. I pushed a fix for this to mlvm/hotspot. Thanks for the reports! -- John On Jun 5, 2010, at 10:58 AM, Rémi Forax wrote: Le 05/06/2010 02:01, John Rose a écrit : Is the call site megamutable? (Is it linked many times instead of once?) -- John no ! The

Re: Invokedynamic slower than reflection?

2010-06-07 Thread Christian Thalinger
On Sat, 2010-06-05 at 19:58 +0200, Rémi Forax wrote: Le 05/06/2010 02:01, John Rose a écrit : Is the call site megamutable? (Is it linked many times instead of once?) -- John no ! The callsite is linked only once. Yeah, that bug is around since: 6939134: JSR 292 adjustments to

Re: Invokedynamic slower than reflection?

2010-06-05 Thread Rémi Forax
Le 05/06/2010 02:01, John Rose a écrit : Is the call site megamutable? (Is it linked many times instead of once?) -- John no ! The callsite is linked only once. Rémi On Jun 4, 2010, at 4:40 AM, Rémi Forax wrote: It's funny, I've found the same error last night. Yes, there is

Re: Invokedynamic slower than reflection?

2010-06-04 Thread Rémi Forax
It's funny, I've found the same error last night. Yes, there is a problem, if you run with -XX:+PrintCompilation, you will see lot of made not entrant on the same bytecode location. 183% made not entrant (2) Hello::main @ -2 (145 bytes) 184% Hello::main @ 6 (145 bytes) It seems that

Re: Invokedynamic slower than reflection?

2010-06-04 Thread John Rose
Is the call site megamutable? (Is it linked many times instead of once?) -- John On Jun 4, 2010, at 4:40 AM, Rémi Forax wrote: It's funny, I've found the same error last night. Yes, there is a problem, if you run with -XX:+PrintCompilation, you will see lot of made not entrant on the