Changeset 90317 in webkit

Timestamp:

Jul 2, 2011 1:45:55 PM (13 years ago)

Author:

mjs@apple.com

Message:

2011-07-02 Maciej Stachowiak <​mjs@apple.com>

Reviewed by Darin Adler.

SunSpider: The main function of math-cordic is dead code
​https://bugs.webkit.org/show_bug.cgi?id=63863

Try to prevent math-cordic from being eliminated as dead code or
compiled down to a constant.

• tests/sunspider-1.0/math-cordic.js: (cordicsincos): (cordic):

Location:

trunk/PerformanceTests/SunSpider

Files:

• ChangeLog (modified) (1 diff)
• tests/sunspider-1.0/math-cordic.js (modified) (4 diffs)

trunk/PerformanceTests/SunSpider/ChangeLog

@@ +1 @@
+2011-07-02  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by Darin Adler.
+
+        SunSpider: The main function of math-cordic is dead code
+        https://bugs.webkit.org/show_bug.cgi?id=63863
+        
+        Try to prevent math-cordic from being eliminated as dead code or
+        compiled down to a constant.
+
+        * tests/sunspider-1.0/math-cordic.js:
+        (cordicsincos):
+        (cordic):
+
 2011-06-30  Maciej Stachowiak  <mjs@apple.com>
 

trunk/PerformanceTests/SunSpider/tests/sunspider-1.0/math-cordic.js

@@ -50 +50 @@
               ];
 
+var Target = 28.027;
 
-function cordicsincos() {
+function cordicsincos(Target) {
     var X;
     var Y;
@@ -61 +62 @@
     Y = 0;                       /* AG_CONST * sin(0) */
 
-    TargetAngle = FIXED(28.027);
+    TargetAngle = FIXED(Target);
     CurrAngle = 0;
     for (Step = 0; Step < 12; Step++) {
@@ -77 +78 @@
         }
     }
+
+    return FLOAT(X) * FLOAT(Y);
 }
 
 ///// End CORDIC
+
+var total = 0;
 
 function cordic( runs ) {
@@ -85 +90 @@
 
   for ( var i = 0 ; i < runs ; i++ ) {
-      cordicsincos();
+      total += cordicsincos(Target);
   }