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

Unified Diff: tests/validator/rewrite_nontemporals.c

Issue 1837733002: x86 validator: Implement rewriting "movntps" to "movaps" on x86-32 (Closed) Base URL: https://chromium.googlesource.com/native_client/src/native_client.git@master
Patch Set: Created 4 years, 9 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 | « src/trusted/validator_ragel/dfa_validate_common.c ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/validator/rewrite_nontemporals.c
diff --git a/tests/validator/rewrite_nontemporals.c b/tests/validator/rewrite_nontemporals.c
index d7cd2c89c482a2b464e1f0c03028bb29c39d0154..baa35ca705c823e8b4077ae1ca321c9e6ce73c0e 100644
--- a/tests/validator/rewrite_nontemporals.c
+++ b/tests/validator/rewrite_nontemporals.c
@@ -51,13 +51,13 @@ int main(void) {
"movntq %%mm0, g_dest" MEM_SUFFIX "\n" : : : "mm0");
ASSERT_EQ(memcmp(g_dest, g_src, 8), 0);
-#if defined(__x86_64__)
/* Test movntps. */
reset_test_vars();
- asm("movdqa g_src(%%r15), %%xmm0\n"
- "movntps %%xmm0, g_dest(%%r15)\n" : : : "xmm0");
+ asm("movdqa g_src" MEM_SUFFIX ", %%xmm0\n"
+ "movntps %%xmm0, g_dest" MEM_SUFFIX "\n" : : : "xmm0");
ASSERT_EQ(memcmp(g_dest, g_src, 16), 0);
+#if defined(__x86_64__)
/* Test movnti, using 32-bit operand. */
reset_test_vars();
asm("mov g_src(%%r15), %%eax\n"
« no previous file with comments | « src/trusted/validator_ragel/dfa_validate_common.c ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698