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

Unified Diff: src/trusted/validator_x86/testdata/64/size_error.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/size_error.tf
diff --git a/src/trusted/validator_x86/testdata/64/size_error.hex b/src/trusted/validator_x86/testdata/64/size_error.tf
similarity index 50%
copy from src/trusted/validator_x86/testdata/64/size_error.hex
copy to src/trusted/validator_x86/testdata/64/size_error.tf
index 48b648c0ffb6658ce979e6f8d7abc0910376ec3f..d14a50ba53acf73fbf095f5e9a8001178537e723 100644
--- a/src/trusted/validator_x86/testdata/64/size_error.hex
+++ b/src/trusted/validator_x86/testdata/64/size_error.tf
@@ -1,13 +1,25 @@
+BITS: 64
+OUTCOME: invalid
+
# The following checks if we catch the use of data 66/Rex.W specifying
# undetermined operand size.
# The following checks that we allow Rex.w.
# 0000000000000000: 48 01 c3 add %rbx, %rax
-48 01 c3
+asm: add %rax,%rbx
+hex: 48 01 c3
+
# The following checks that we allow data 66.
# 0000000000000003: 66 01 c3 add %bx, %ax
-66 01 c3
+asm: add %ax,%bx
+hex: 66 01 c3
+
# The following checks that we don't allow Rex.w and data 66
-66 48 01 c3
+hex: 66 48 01 c3
+nc_out: ERROR: Opcode sequence doesn't define a valid x86 instruction
+nc_out: ERROR: Use of DATA16 (66) prefix for instruction not allowed by Native Client
+nc_out: [at +3] ERROR: This instruction has been marked illegal by Native Client
+nc_out: [at +3] ERROR: Illegal assignment to RSP
+

Powered by Google App Engine
This is Rietveld 408576698