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

Unified Diff: src/IceAssemblerX86Base.h

Issue 1231903002: Adds tests to the AssemblerX8632. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Addresses comments. Created 5 years, 5 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 | « Makefile.standalone ('k') | unittest/IceAssemblerX8632Test.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/IceAssemblerX86Base.h
diff --git a/src/IceAssemblerX86Base.h b/src/IceAssemblerX86Base.h
index 8a556f080eeddff030f099ee7bf347cb8703322c..1db270ab20af84ff7d7f07670930fb124b280e37 100644
--- a/src/IceAssemblerX86Base.h
+++ b/src/IceAssemblerX86Base.h
@@ -332,10 +332,10 @@ public:
void setcc(typename Traits::Cond::BrCond condition,
const typename Traits::Address &address);
+ // All mov() overloads are tested.
void mov(Type Ty, typename Traits::GPRRegister dst, const Immediate &src);
void mov(Type Ty, typename Traits::GPRRegister dst,
typename Traits::GPRRegister src);
-
void mov(Type Ty, typename Traits::GPRRegister dst,
const typename Traits::Address &src);
void mov(Type Ty, const typename Traits::Address &dst,
@@ -824,6 +824,9 @@ public:
void hlt();
void ud2();
+ // j(Label) is fully tested.
+ // j(ConstantRelocatable) is not tested as the test can not easily create such
+ // an argument.
void j(typename Traits::Cond::BrCond condition, Label *label,
bool near = kFarJump);
void j(typename Traits::Cond::BrCond condition,
« no previous file with comments | « Makefile.standalone ('k') | unittest/IceAssemblerX8632Test.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698