Subzero. Removes references to %ah.

src/IceRegistersX8632.h

Index: src/IceRegistersX8632.h
diff --git a/src/IceRegistersX8632.h b/src/IceRegistersX8632.h
index da920e28d9552238ed0773fcddfe82f179df15ab..e169192ae55424ab9451da44ea86f748ec31c71e 100644
--- a/src/IceRegistersX8632.h
+++ b/src/IceRegistersX8632.h
@@ -88,9 +88,7 @@ public:
   }
 
   static inline ByteRegister getEncodedByteReg(int32_t RegNum) {
-    assert(RegNum == Reg_ah || (Reg_GPR_First <= RegNum && RegNum <= Reg_ebx));
-    if (RegNum == Reg_ah)
-      return Encoded_Reg_ah;
+    assert(Reg_GPR_First <= RegNum && RegNum <= Reg_ebx);

[Jim Stichnoth, 2015/07/28 15:58:23]

I must have missed this before, but now that it's in my sights...

For asserts like
  assert(A && B);
I prefer breaking it into
  assert(A);
  assert(B);
which gives more precise information when the assertion fails.

[John, 2015/07/28 18:21:16]

Nothing that a simple

:%s/assert(\([^&]\+\) && \([^)]\+\))/assert(\1);assert(\2)

won't solve. :)

     return ByteRegister(RegNum - Reg_GPR_First);
   }