Subzero: Fix over-aggressive bool folding.

src/IceIntrinsics.h

Index: src/IceIntrinsics.h
diff --git a/src/IceIntrinsics.h b/src/IceIntrinsics.h
index aea0bd202f12eb63e6c8b5193fea6d8ee6a03fbf..954f00bd1e8ef9ce048d04cbe920c05abd26eca3 100644
--- a/src/IceIntrinsics.h
+++ b/src/IceIntrinsics.h
@@ -106,14 +106,17 @@ public:
 
   enum ReturnsTwice { ReturnsTwice_F = 0, ReturnsTwice_T = 1 };
 
+  enum MemoryWrite { MemoryWrite_F = 0, MemoryWrite_T = 1 };
+
   /// Basic attributes related to each intrinsic, that are relevant to code
   /// generation. Perhaps the attributes representation can be shared with
   /// general function calls, but PNaCl currently strips all attributes from
   /// functions.
   struct IntrinsicInfo {
-    enum IntrinsicID ID : 30;
+    enum IntrinsicID ID : 29;
     enum SideEffects HasSideEffects : 1;
     enum ReturnsTwice ReturnsTwice : 1;
+    enum MemoryWrite IsMemoryWrite : 1;

[Eric Holk, 2016/04/21 22:09:43]

It looks like this struct is meant to be 32-bits. What about adding a
static_assert to make sure it stays the right size?

[Jim Stichnoth, 2016/04/21 22:50:29]

Done.

   };
 
   /// The types of validation values for FullIntrinsicInfo.validateCall.