Reviewers: titzer, Benedikt Meurer (OOO),

Message:
Ben: PTAL.
Benedikt: FYI.

Description:
[turbofan] Fix test of %_MathClz32 intrinsic.

This test will fail once we optimize top-level code, because the
aforementioned intrinsic doesn't perform a NumberToUint32 conversion.

[email protected]
TEST=mjsunit/asm/math-clz32

Please review this at https://codereview.chromium.org/1041173002/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+1, -1 lines):
  M test/mjsunit/asm/math-clz32.js


Index: test/mjsunit/asm/math-clz32.js
diff --git a/test/mjsunit/asm/math-clz32.js b/test/mjsunit/asm/math-clz32.js
index 004aa65b4639542a80391dd01e58bbdf413de1c5..5a3aa202dbcec10d5ec7adc2882d975ae08e774e 100644
--- a/test/mjsunit/asm/math-clz32.js
+++ b/test/mjsunit/asm/math-clz32.js
@@ -27,5 +27,5 @@ for (var i = 0; i < 32; ++i) {
 }
 for (var i = -2147483648; i < 2147483648; i += 3999773) {
   assertEquals(%MathClz32(i), f(i));
-  assertEquals(%_MathClz32(i), f(i));
+  assertEquals(%MathClz32(i), %_MathClz32(i >>> 0));
 }


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to