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

Unified Diff: src/trusted/validator_x86/testdata/32/direct-cond-jump-16.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/32/direct-cond-jump-16.tf
diff --git a/src/trusted/validator_x86/testdata/64/direct-cond-jump-16.hex b/src/trusted/validator_x86/testdata/32/direct-cond-jump-16.tf
similarity index 74%
copy from src/trusted/validator_x86/testdata/64/direct-cond-jump-16.hex
copy to src/trusted/validator_x86/testdata/32/direct-cond-jump-16.tf
index b0d445e0f3578ee94eb7a5d221d4eb5cdcfa2716..892d97cae50c0b0aba9c75a08d1220b0d92b8beb 100644
--- a/src/trusted/validator_x86/testdata/64/direct-cond-jump-16.hex
+++ b/src/trusted/validator_x86/testdata/32/direct-cond-jump-16.tf
@@ -1,9 +1,16 @@
+BITS: 32
+OUTCOME: invalid
+
# This file tests how we handle a 16-bit conditional direct jump. In particular,
# it tests whether we actually disallow prefix 66 on direct jumps.
#
# Define direct (16-bit) conditional jump, which NaCl doesn't allow
-66 0f 84 05 00
+hex: 66 0f 84 05 00
+nc_out: Undefined instruction
+
# Padding in case the parser of the previous instruction recognizes
# the wrong byte length for the jump constant.
-90 90 90
+asm: nop; nop; nop
+hex: 90 90 90
+

Powered by Google App Engine
This is Rietveld 408576698