Mailing-List: contact commits-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Message-ID: <32789631.1202897528988.JavaMail.jira@brutus>
Date: Wed, 13 Feb 2008 02:12:08 -0800 (PST)
From: "Evgeniya Maenkova (JIRA)" <jira@apache.org>
To: commits@harmony.apache.org
Subject: [jira] Commented: (HARMONY-5480) [drlvm][jit] Some of
 java.lang.Math methods could be implemented by api_magics
In-Reply-To: <31974095.1202565907866.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/HARMONY-5480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12568491#action_12568491 ] 

Evgeniya Maenkova commented on HARMONY-5480:
--------------------------------------------

George:
Thank you so much for your comments?

Aleksey:
Could you please correct this line  before check in? Obviously, the change is pretty trivial. Thanks!


> [drlvm][jit] Some of java.lang.Math methods could be implemented by api_magics
> ------------------------------------------------------------------------------
>
>                 Key: HARMONY-5480
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5480
>             Project: Harmony
>          Issue Type: Improvement
>          Components: DRLVM
>            Reporter: Evgeniya Maenkova
>            Assignee: Alexey Varlamov
>         Attachments: MathTest.java, vm.math.patch2, vm.math.patch4
>
>
> The patch gives >7x speedup on my laptop on test mentioned below:
> package math;
> public class MathTest {
> 	static void f() {
> 		Math.abs(-123987.1236d);
> 		Math.asin(0.7);
> 		Math.acos(0.7);		
> 		Math.log(123.123);
> 		Math.log10(123.123);
> 		Math.log1p(123.123);		
> 		Math.sin(12312.123);
> 		Math.cos(12312.123);
> 		Math.sqrt(234234.234234);
> 		Math.tan(234234.12342134);
> 		Math.atan(2347.234);
> 		Math.atan2(231.123, 0);
> 		Math.abs(-123.1231123f);				
> 	}
>     public static void main(String[] args) {    	
>     	System.out.println("Warmup started....");
>     	for (int i = 0; i < 2500 * 2500; i ++) {
> 		     f();
> 		}
>     	for (int i = 0; i < 250 * 2500; i ++) {
> 		     f();
> 		}
>     	System.out.println("Warmup ended....");
>     	long start = System.currentTimeMillis();		
> 		for (int i = 0; i < 2500 * 2500; i ++) {
>     		f();
> 		}
> 		System.out.println("floor result: " + (System.currentTimeMillis() - start));	
> 	}
> }
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -XX:jit.arg.Math
> _as_magic=true -Xem:server math.MathTest
> Warmup started....
> Warmup ended....
> floor result: 4875
> Administrator@EGMAENKO-MOBL /cygdrive/c/ecl_311_ws/MicroBenches/bin1
> $ /cygdrive/c/home_test/temp_builds/egm_r618602_Mj_Mc/bin/java -XX:jit.arg.Math
> _as_magic=false -Xem:server math.MathTest
> Warmup started....
> Warmup ended....
> floor result: 34141

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.