Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(278)

Unified Diff: test/mjsunit/asm/uint32mod-constant.js

Issue 2663243002: [asm] Fix lots of invalid asm.js tests (Closed)
Patch Set: Rebase Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/mjsunit/asm/uint32mod.js ('k') | test/mjsunit/asm/uint8array-outofbounds.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/asm/uint32mod-constant.js
diff --git a/test/mjsunit/asm/uint32mod-constant.js b/test/mjsunit/asm/uint32mod-constant.js
index 4ba94dad20393c4a4eebd01492a6a1f5bfc5d627..562dcd11f924a15b85bc562f05be5007dc86fe04 100644
--- a/test/mjsunit/asm/uint32mod-constant.js
+++ b/test/mjsunit/asm/uint32mod-constant.js
@@ -9,13 +9,15 @@ var heap = new ArrayBuffer(64 * 1024);
function Uint32Mod(divisor) {
var name = "mod_";
name += divisor;
- var m = eval("function Module(stdlib, foreign, heap) {\n"
- + " \"use asm\";\n"
- + " function " + name + "(dividend) {\n"
- + " return ((dividend >>> 0) % " + divisor + ") >>> 0;\n"
- + " }\n"
- + " return { f: " + name + "}\n"
- + "}; Module");
+ var m = eval(
+ 'function Module(stdlib, foreign, heap) {\n' +
+ ' "use asm";\n' +
+ ' function ' + name + '(dividend) {\n' +
+ ' dividend = dividend | 0;\n' +
+ ' return ((dividend >>> 0) % ' + divisor + ') | 0;\n' +
+ ' }\n' +
+ ' return { f: ' + name + '}\n' +
+ '}; Module');
return m(stdlib, foreign, heap).f;
}
@@ -24,6 +26,6 @@ for (var i in divisors) {
var divisor = divisors[i];
var mod = Uint32Mod(divisor);
for (var dividend = 0; dividend < 4294967296; dividend += 3999773) {
- assertEquals((dividend % divisor) >>> 0, mod(dividend));
+ assertEquals((dividend % divisor) | 0, mod(dividend));
}
}
« no previous file with comments | « test/mjsunit/asm/uint32mod.js ('k') | test/mjsunit/asm/uint8array-outofbounds.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698