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

Unified Diff: src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.tf

Issue 10908137: (abandoned) Validator tests: convert hexes to TFs and run on bots (for prod. validator only) (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: restore 'read overflow' and 'SegmentationError' Created 8 years, 3 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
Index: src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.tf
diff --git a/src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.hex b/src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.tf
similarity index 51%
copy from src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.hex
copy to src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.tf
index f07e0e812c56b563bf7843487849dfc4954b479d..2a3772dd7fa36d4180960318a3396e762fa72532 100644
--- a/src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.hex
+++ b/src/trusted/validator_x86/testdata/64/mov-lea-rbp-bad-5.tf
@@ -1,7 +1,16 @@
+BITS: 64
+OUTCOME: invalid
+
# Bad example of using lea solution (instead of add) to update rbp.
# That is where we use rsp in address instead of rbp.
# That is, apply a mov to non 32-bit register rbp
# mov %rbp, %rcx
# lea %rbp, [%rsp+%r15*1]
-48 89 cd
-4a 8d 2c 3c
+asm: mov %rcx,%rbp
+hex: 48 89 cd
+nc_out: ERROR: Illegal change to register RBP
+
+asm: lea (%rsp,%r15,1),%rbp
+hex: 4a 8d 2c 3c
+nc_out: ERROR: Illegal change to register RBP
+

Powered by Google App Engine
This is Rietveld 408576698