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

Unified Diff: test/mjsunit/asm/int32-mod.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/int32-div.js ('k') | test/mjsunit/asm/int32-mul.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/asm/int32-mod.js
diff --git a/test/mjsunit/asm/int32-mod.js b/test/mjsunit/asm/int32-mod.js
index b3a7c0ecf4ce216d3b5fff41dd164bc0e4c769e0..141bd0282f9f11ae8033d3392744bbc838c5cc57 100644
--- a/test/mjsunit/asm/int32-mod.js
+++ b/test/mjsunit/asm/int32-mod.js
@@ -6,19 +6,19 @@ function Module(stdlib, foreign, heap) {
"use asm";
function f1(i) {
i = i|0;
- return i % 3;
+ return ((i | 0) % 3) | 0;
}
function f2(i) {
i = i|0;
- return i % 9;
+ return ((i | 0) % 9) | 0;
}
function f3(i) {
i = i|0;
- return i % 1024;
+ return ((i | 0) % 1024) | 0;
}
function f4(i) {
i = i|0;
- return i % 3133335;
+ return ((i | 0) % 3133335) | 0;
}
return { f1: f1, f2: f2, f3: f3, f4: f4 };
}
@@ -26,8 +26,8 @@ function Module(stdlib, foreign, heap) {
var m = Module(this, {}, new ArrayBuffer(1024));
for (var i = -2147483648; i < 2147483648; i += 3999773) {
- assertEquals(i % 3, m.f1(i));
- assertEquals(i % 9, m.f2(i));
- assertEquals(i % 1024, m.f3(i));
- assertEquals(i % 3133335, m.f4(i));
+ assertEquals((i % 3) | 0, m.f1(i));
+ assertEquals((i % 9) | 0, m.f2(i));
+ assertEquals((i % 1024) | 0, m.f3(i));
+ assertEquals((i % 3133335) | 0, m.f4(i));
}
« no previous file with comments | « test/mjsunit/asm/int32-div.js ('k') | test/mjsunit/asm/int32-mul.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698