Passing all rewriter tests

dyninst · Jul 14, 2016 · 328cf8e · 328cf8e
1 parent 99cc585
commit 328cf8e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/common/src/arch-x86.C b/common/src/arch-x86.C
@@ -61,9 +61,9 @@
 // #define VEX_PEDANTIC
 
 /* Whether or not to dump instruction information on decode failure */
-#define DECODE_FAILURE_DUMP
+// #define DECODE_FAILURE_DUMP
 /* Throw an assert when decoding errors are encountered (debugging) */
-#define DECODE_FAILURE_ASSERT
+// #define DECODE_FAILURE_ASSERT
 
 using namespace std;
 using namespace boost::assign;

diff --git a/dyninstAPI/src/codegen-x86.C b/dyninstAPI/src/codegen-x86.C
@@ -1174,7 +1174,7 @@ bool insnCodeGen::modifyData(Address targetAddr, instruction &insn, codeGen &gen
      * We are only going to do the prefix and opcode decodings
      */
     ia32_instruction instruct;
-    if(!ia32_decode(0, origInsn, instruct))
+    if(ia32_decode(0, origInsn, instruct))
         assert(!"Couldn't decode prefix of already known instruction!\n");
 
     /* get the prefix and opcode count */
@@ -1288,7 +1288,7 @@ bool insnCodeGen::modifyDisp(signed long newDisp, instruction &insn,
      * We are only going to do the prefix and opcode decodings
      */
     ia32_instruction instruct; 
-    if(!ia32_decode(0, origInsn, instruct))
+    if(ia32_decode(0, origInsn, instruct))
         assert(!"Couldn't decode prefix of already known instruction!\n");
 
     /* get the prefix count */