Fix the prediction warnings in EBC.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@5195 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/MdeModulePkg/Universal/EbcDxe/EbcExecute.c b/MdeModulePkg/Universal/EbcDxe/EbcExecute.c
index c3c7b915f98ea6a5ba0f3edfe0815e1c2ebaaba1..31ff76d80d4237562ce81b5f6f2c1c348637c11f 100644 (file)
--- a/MdeModulePkg/Universal/EbcDxe/EbcExecute.c
+++ b/MdeModulePkg/Universal/EbcDxe/EbcExecute.c
@@ -1527,7 +1527,7 @@ EbcExecute (
   // instruction sets it if it runs out of stack.\r
   //\r
   VmPtr->StopFlags = 0;\r
-  while (!(VmPtr->StopFlags & STOPFLAG_APP_DONE)) {\r
+  while ((VmPtr->StopFlags & STOPFLAG_APP_DONE) == 0) {\r
     //\r
     // If we've found a simple debugger protocol, call it\r
     //\r
@@ -1575,11 +1575,11 @@ EbcExecute (
     //\r
     // Make sure stack has not been corrupted. Only report it once though.\r
     //\r
-    if (!StackCorrupted && (*VmPtr->StackMagicPtr != (UINTN) VM_STACK_KEY_VALUE)) {\r
+    if ((StackCorrupted == 0) && (*VmPtr->StackMagicPtr != (UINTN) VM_STACK_KEY_VALUE)) {\r
       EbcDebugSignalException (EXCEPT_EBC_STACK_FAULT, EXCEPTION_FLAG_FATAL, VmPtr);\r
       StackCorrupted = 1;\r
     }\r
-    if (!StackCorrupted && ((UINT64)VmPtr->R[0] <= (UINT64)(UINTN) VmPtr->StackTop)) {\r
+    if ((StackCorrupted == 0) && ((UINT64)VmPtr->R[0] <= (UINT64)(UINTN) VmPtr->StackTop)) {\r
       EbcDebugSignalException (EXCEPT_EBC_STACK_FAULT, EXCEPTION_FLAG_FATAL, VmPtr);\r
       StackCorrupted = 1;\r
     }\r
@@ -1652,7 +1652,7 @@ ExecuteMOVxx (
   // is 2 (opcode + operands). Add to this size each index specified.\r
   //\r
   Size = 2;\r
-  if (Opcode & (OPCODE_M_IMMED_OP1 | OPCODE_M_IMMED_OP2)) {\r
+  if ((Opcode & (OPCODE_M_IMMED_OP1 | OPCODE_M_IMMED_OP2)) != 0) {\r
     //\r
     // Determine size of the index from the opcode. Then get it.\r
     //\r
@@ -1661,13 +1661,13 @@ ExecuteMOVxx (
       // MOVBW, MOVWW, MOVDW, MOVQW, and MOVNW have 16-bit immediate index.\r
       // Get one or both index values.\r
       //\r
-      if (Opcode & OPCODE_M_IMMED_OP1) {\r
+      if ((Opcode & OPCODE_M_IMMED_OP1) != 0) {\r
         Index16     = VmReadIndex16 (VmPtr, 2);\r
         Index64Op1  = (INT64) Index16;\r
         Size += sizeof (UINT16);\r
       }\r
 \r
-      if (Opcode & OPCODE_M_IMMED_OP2) {\r
+      if ((Opcode & OPCODE_M_IMMED_OP2) != 0) {\r
         Index16     = VmReadIndex16 (VmPtr, Size);\r
         Index64Op2  = (INT64) Index16;\r
         Size += sizeof (UINT16);\r
@@ -1676,13 +1676,13 @@ ExecuteMOVxx (
       //\r
       // MOVBD, MOVWD, MOVDD, MOVQD, and MOVND have 32-bit immediate index\r
       //\r
-      if (Opcode & OPCODE_M_IMMED_OP1) {\r
+      if ((Opcode & OPCODE_M_IMMED_OP1) != 0) {\r
         Index32     = VmReadIndex32 (VmPtr, 2);\r
         Index64Op1  = (INT64) Index32;\r
         Size += sizeof (UINT32);\r
       }\r
 \r
-      if (Opcode & OPCODE_M_IMMED_OP2) {\r
+      if ((Opcode & OPCODE_M_IMMED_OP2) != 0) {\r
         Index32     = VmReadIndex32 (VmPtr, Size);\r
         Index64Op2  = (INT64) Index32;\r
         Size += sizeof (UINT32);\r
@@ -1691,12 +1691,12 @@ ExecuteMOVxx (
       //\r
       // MOVqq -- only form with a 64-bit index\r
       //\r
-      if (Opcode & OPCODE_M_IMMED_OP1) {\r
+      if ((Opcode & OPCODE_M_IMMED_OP1) != 0) {\r
         Index64Op1 = VmReadIndex64 (VmPtr, 2);\r
         Size += sizeof (UINT64);\r
       }\r
 \r
-      if (Opcode & OPCODE_M_IMMED_OP2) {\r
+      if ((Opcode & OPCODE_M_IMMED_OP2) != 0) {\r
         Index64Op2 = VmReadIndex64 (VmPtr, Size);\r
         Size += sizeof (UINT64);\r
       }\r
@@ -1786,7 +1786,7 @@ ExecuteMOVxx (
     // Did Operand2 have an index? If so, treat as two signed values since\r
     // indexes are signed values.\r
     //\r
-    if (Opcode & OPCODE_M_IMMED_OP2) {\r
+    if ((Opcode & OPCODE_M_IMMED_OP2) != 0) {\r
       //\r
       // NOTE: need to find a way to fix this, most likely by changing the VM\r
       // implementation to remove the stack gap. To do that, we'd need to\r
@@ -1853,7 +1853,7 @@ ExecuteMOVxx (
     // Operand1 direct.\r
     // Make sure we didn't have an index on operand1.\r
     //\r
-    if (Opcode & OPCODE_M_IMMED_OP1) {\r
+    if ((Opcode & OPCODE_M_IMMED_OP1) != 0) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_INSTRUCTION_ENCODING,\r
         EXCEPTION_FLAG_FATAL,\r
@@ -2037,9 +2037,9 @@ ExecuteJMP (
   // Decode instruction conditions\r
   // If we haven't met the condition, then simply advance the IP and return.\r
   //\r
-  CompareSet    = (UINT8) ((Operand & JMP_M_CS) ? 1 : 0);\r
+  CompareSet    = (UINT8) (((Operand & JMP_M_CS) != 0) ? 1 : 0);\r
   ConditionFlag = (UINT8) VMFLAG_ISSET (VmPtr, VMFLAGS_CC);\r
-  if (Operand & CONDITION_M_CONDITIONAL) {\r
+  if ((Operand & CONDITION_M_CONDITIONAL) != 0) {\r
     if (CompareSet != ConditionFlag) {\r
       VmPtr->Ip += Size;\r
       return EFI_SUCCESS;\r
@@ -2049,12 +2049,12 @@ ExecuteJMP (
   // Check for 64-bit form and do it right away since it's the most\r
   // straight-forward form.\r
   //\r
-  if (Opcode & OPCODE_M_IMMDATA64) {\r
+  if ((Opcode & OPCODE_M_IMMDATA64) != 0) {\r
     //\r
     // Double check for immediate-data, which is required. If not there,\r
     // then signal an exception\r
     //\r
-    if (!(Opcode & OPCODE_M_IMMDATA)) {\r
+    if ((Opcode & OPCODE_M_IMMDATA) == 0) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_INSTRUCTION_ENCODING,\r
         EXCEPTION_FLAG_ERROR,\r
@@ -2080,7 +2080,7 @@ ExecuteJMP (
     //\r
     // Take jump -- relative or absolute\r
     //\r
-    if (Operand & JMP_M_RELATIVE) {\r
+    if ((Operand & JMP_M_RELATIVE) != 0) {\r
       VmPtr->Ip += (UINTN) Data64 + Size;\r
     } else {\r
       VmPtr->Ip = (VMIP) (UINTN) Data64;\r
@@ -2095,7 +2095,7 @@ ExecuteJMP (
   //   JMP32 @R1 Index32 -- immediate data is an index\r
   //   JMP32 R1 Immed32  -- immedate data is an offset\r
   //\r
-  if (Opcode & OPCODE_M_IMMDATA) {\r
+  if ((Opcode & OPCODE_M_IMMDATA) != 0) {\r
     if (OPERAND1_INDIRECT (Operand)) {\r
       Index32 = VmReadIndex32 (VmPtr, 2);\r
     } else {\r
@@ -2130,7 +2130,7 @@ ExecuteJMP (
       return EFI_UNSUPPORTED;\r
     }\r
 \r
-    if (Operand & JMP_M_RELATIVE) {\r
+    if ((Operand & JMP_M_RELATIVE) != 0) {\r
       VmPtr->Ip += (UINTN) Addr + Size;\r
     } else {\r
       VmPtr->Ip = (VMIP) Addr;\r
@@ -2150,7 +2150,7 @@ ExecuteJMP (
       return EFI_UNSUPPORTED;\r
     }\r
 \r
-    if (Operand & JMP_M_RELATIVE) {\r
+    if ((Operand & JMP_M_RELATIVE) != 0) {\r
       VmPtr->Ip += (UINTN) Addr + Size;\r
     } else {\r
       VmPtr->Ip = (VMIP) Addr;\r
@@ -2187,13 +2187,13 @@ ExecuteJMP8 (
   // Decode instruction.\r
   //\r
   Opcode        = GETOPCODE (VmPtr);\r
-  CompareSet    = (UINT8) ((Opcode & JMP_M_CS) ? 1 : 0);\r
+  CompareSet    = (UINT8) (((Opcode & JMP_M_CS) != 0) ? 1 : 0);\r
   ConditionFlag = (UINT8) VMFLAG_ISSET (VmPtr, VMFLAGS_CC);\r
 \r
   //\r
   // If we haven't met the condition, then simply advance the IP and return\r
   //\r
-  if (Opcode & CONDITION_M_CONDITIONAL) {\r
+  if ((Opcode & CONDITION_M_CONDITIONAL) != 0) {\r
     if (CompareSet != ConditionFlag) {\r
       VmPtr->Ip += 2;\r
       return EFI_SUCCESS;\r
@@ -2256,7 +2256,7 @@ ExecuteMOVI (
   //\r
   // Get the index (16-bit) if present\r
   //\r
-  if (Operands & MOVI_M_IMMDATA) {\r
+  if ((Operands & MOVI_M_IMMDATA) != 0) {\r
     Index16 = VmReadIndex16 (VmPtr, 2);\r
     Size    = 4;\r
   } else {\r
@@ -2293,7 +2293,7 @@ ExecuteMOVI (
     //\r
     // Operand1 direct. Make sure it didn't have an index.\r
     //\r
-    if (Operands & MOVI_M_IMMDATA) {\r
+    if ((Operands & MOVI_M_IMMDATA) != 0) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_INSTRUCTION_ENCODING,\r
         EXCEPTION_FLAG_FATAL,\r
@@ -2376,7 +2376,7 @@ ExecuteMOVIn (
   //\r
   // Get the operand1 index (16-bit) if present\r
   //\r
-  if (Operands & MOVI_M_IMMDATA) {\r
+  if ((Operands & MOVI_M_IMMDATA) != 0) {\r
     Index16 = VmReadIndex16 (VmPtr, 2);\r
     Size    = 4;\r
   } else {\r
@@ -2416,7 +2416,7 @@ ExecuteMOVIn (
     // Check for MOVIn R1 Index16, Immed (not indirect, with index), which\r
     // is illegal\r
     //\r
-    if (Operands & MOVI_M_IMMDATA) {\r
+    if ((Operands & MOVI_M_IMMDATA) != 0) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_INSTRUCTION_ENCODING,\r
         EXCEPTION_FLAG_FATAL,\r
@@ -2478,7 +2478,7 @@ ExecuteMOVREL (
   //\r
   // Get the Operand 1 index (16-bit) if present\r
   //\r
-  if (Operands & MOVI_M_IMMDATA) {\r
+  if ((Operands & MOVI_M_IMMDATA) != 0) {\r
     Index16 = VmReadIndex16 (VmPtr, 2);\r
     Size    = 4;\r
   } else {\r
@@ -2516,7 +2516,7 @@ ExecuteMOVREL (
     //\r
     // Check for illegal combination of operand1 direct with immediate data\r
     //\r
-    if (Operands & MOVI_M_IMMDATA) {\r
+    if ((Operands & MOVI_M_IMMDATA) != 0) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_INSTRUCTION_ENCODING,\r
         EXCEPTION_FLAG_FATAL,\r
@@ -2587,7 +2587,7 @@ ExecuteMOVsnw (
   // Get the indexes if present.\r
   //\r
   Size = 2;\r
-  if (Opcode & OPCODE_M_IMMED_OP1) {\r
+  if ((Opcode & OPCODE_M_IMMED_OP1) !=0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
       Op1Index = VmReadIndex16 (VmPtr, 2);\r
     } else {\r
@@ -2605,7 +2605,7 @@ ExecuteMOVsnw (
     Size += sizeof (UINT16);\r
   }\r
 \r
-  if (Opcode & OPCODE_M_IMMED_OP2) {\r
+  if ((Opcode & OPCODE_M_IMMED_OP2) != 0) {\r
     if (OPERAND2_INDIRECT (Operands)) {\r
       Op2Index = VmReadIndex16 (VmPtr, Size);\r
     } else {\r
@@ -2681,7 +2681,7 @@ ExecuteMOVsnd (
   // Get the indexes if present.\r
   //\r
   Size = 2;\r
-  if (Opcode & OPCODE_M_IMMED_OP1) {\r
+  if ((Opcode & OPCODE_M_IMMED_OP1) != 0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
       Op1Index = VmReadIndex32 (VmPtr, 2);\r
     } else {\r
@@ -2699,7 +2699,7 @@ ExecuteMOVsnd (
     Size += sizeof (UINT32);\r
   }\r
 \r
-  if (Opcode & OPCODE_M_IMMED_OP2) {\r
+  if ((Opcode & OPCODE_M_IMMED_OP2) != 0) {\r
     if (OPERAND2_INDIRECT (Operands)) {\r
       Op2Index = VmReadIndex32 (VmPtr, Size);\r
     } else {\r
@@ -2762,7 +2762,7 @@ ExecutePUSHn (
   //\r
   // Get index if present\r
   //\r
-  if (Opcode & PUSHPOP_M_IMMDATA) {\r
+  if ((Opcode & PUSHPOP_M_IMMDATA) != 0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
       Index16 = VmReadIndex16 (VmPtr, 2);\r
     } else {\r
@@ -2822,7 +2822,7 @@ ExecutePUSH (
   //\r
   // Get immediate index if present, then advance the IP.\r
   //\r
-  if (Opcode & PUSHPOP_M_IMMDATA) {\r
+  if ((Opcode & PUSHPOP_M_IMMDATA) != 0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
       Index16 = VmReadIndex16 (VmPtr, 2);\r
     } else {\r
@@ -2837,7 +2837,7 @@ ExecutePUSH (
   //\r
   // Get the data to push\r
   //\r
-  if (Opcode & PUSHPOP_M_64) {\r
+  if ((Opcode & PUSHPOP_M_64) != 0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
       Data64 = VmReadMem64 (VmPtr, (UINTN) (VmPtr->R[OPERAND1_REGNUM (Operands)] + Index16));\r
     } else {\r
@@ -2898,7 +2898,7 @@ ExecutePOPn (
   //\r
   // Get immediate data if present, and advance the IP\r
   //\r
-  if (Opcode & PUSHPOP_M_IMMDATA) {\r
+  if ((Opcode & PUSHPOP_M_IMMDATA) != 0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
       Index16 = VmReadIndex16 (VmPtr, 2);\r
     } else {\r
@@ -2959,7 +2959,7 @@ ExecutePOP (
   //\r
   // Get immediate data if present, and advance the IP.\r
   //\r
-  if (Opcode & PUSHPOP_M_IMMDATA) {\r
+  if ((Opcode & PUSHPOP_M_IMMDATA) != 0) {\r
     if (OPERAND1_INDIRECT (Operands)) {\r
       Index16 = VmReadIndex16 (VmPtr, 2);\r
     } else {\r
@@ -2974,7 +2974,7 @@ ExecutePOP (
   //\r
   // Get the data off the stack, then write it to the appropriate location\r
   //\r
-  if (Opcode & PUSHPOP_M_64) {\r
+  if ((Opcode & PUSHPOP_M_64) != 0) {\r
     //\r
     // Read the data off the stack, then adjust the stack pointer\r
     //\r
@@ -3052,8 +3052,8 @@ ExecuteCALL (
   //\r
   // Determine the instruction size, and get immediate data if present\r
   //\r
-  if (Opcode & OPCODE_M_IMMDATA) {\r
-    if (Opcode & OPCODE_M_IMMDATA64) {\r
+  if ((Opcode & OPCODE_M_IMMDATA) != 0) {\r
+    if ((Opcode & OPCODE_M_IMMDATA64) != 0) {\r
       Immed64 = VmReadImmed64 (VmPtr, 2);\r
       Size    = 10;\r
     } else {\r
@@ -3085,7 +3085,7 @@ ExecuteCALL (
   //\r
   // If 64-bit data, then absolute jump only\r
   //\r
-  if (Opcode & OPCODE_M_IMMDATA64) {\r
+  if ((Opcode & OPCODE_M_IMMDATA64) != 0) {\r
     //\r
     // Native or EBC call?\r
     //\r
@@ -3122,7 +3122,7 @@ ExecuteCALL (
       // EBC call. Relative or absolute? If relative, then it's relative to the\r
       // start of the next instruction.\r
       //\r
-      if (Operands & OPERAND_M_RELATIVE_ADDR) {\r
+      if ((Operands & OPERAND_M_RELATIVE_ADDR) != 0) {\r
         VmPtr->Ip += Immed64 + Size;\r
       } else {\r
         VmPtr->Ip = (VMIP) (UINTN) Immed64;\r
@@ -3131,10 +3131,10 @@ ExecuteCALL (
       //\r
       // Native call. Relative or absolute?\r
       //\r
-      if (Operands & OPERAND_M_RELATIVE_ADDR) {\r
+      if ((Operands & OPERAND_M_RELATIVE_ADDR) != 0) {\r
         EbcLLCALLEX (VmPtr, (UINTN) (Immed64 + VmPtr->Ip + Size), (UINTN) VmPtr->R[0], FramePtr, Size);\r
       } else {\r
-        if (VmPtr->StopFlags & STOPFLAG_BREAK_ON_CALLEX) {\r
+        if ((VmPtr->StopFlags & STOPFLAG_BREAK_ON_CALLEX) != 0) {\r
           CpuBreakpoint ();\r
         }\r
 \r
@@ -3233,7 +3233,7 @@ ExecuteCMP (
   //\r
   // Get immediate data\r
   //\r
-  if (Opcode & OPCODE_M_IMMDATA) {\r
+  if ((Opcode & OPCODE_M_IMMDATA) != 0) {\r
     if (OPERAND2_INDIRECT (Operands)) {\r
       Index16 = VmReadIndex16 (VmPtr, 2);\r
     } else {\r
@@ -3249,7 +3249,7 @@ ExecuteCMP (
   // Now get Op2\r
   //\r
   if (OPERAND2_INDIRECT (Operands)) {\r
-    if (Opcode & OPCODE_M_64BIT) {\r
+    if ((Opcode & OPCODE_M_64BIT) != 0) {\r
       Op2 = (INT64) VmReadMem64 (VmPtr, (UINTN) (VmPtr->R[OPERAND2_REGNUM (Operands)] + Index16));\r
     } else {\r
       //\r
@@ -3264,7 +3264,7 @@ ExecuteCMP (
   // Now do the compare\r
   //\r
   Flag = 0;\r
-  if (Opcode & OPCODE_M_64BIT) {\r
+  if ((Opcode & OPCODE_M_64BIT) != 0) {\r
     //\r
     // 64-bit compares\r
     //\r
@@ -3344,7 +3344,7 @@ ExecuteCMP (
   //\r
   // Now set the flag accordingly for the comparison\r
   //\r
-  if (Flag) {\r
+  if (Flag != 0) {\r
     VMFLAG_SET (VmPtr, VMFLAGS_CC);\r
   } else {\r
     VMFLAG_CLEAR (VmPtr, VMFLAGS_CC);\r
@@ -3393,7 +3393,7 @@ ExecuteCMPI (
   // Get operand1 index if present\r
   //\r
   Size = 2;\r
-  if (Operands & OPERAND_M_CMPI_INDEX) {\r
+  if ((Operands & OPERAND_M_CMPI_INDEX) != 0) {\r
     Index16 = VmReadIndex16 (VmPtr, 2);\r
     Size += 2;\r
   } else {\r
@@ -3407,7 +3407,7 @@ ExecuteCMPI (
     //\r
     // Indirect operand1. Fetch 32 or 64-bit value based on compare size.\r
     //\r
-    if (Opcode & OPCODE_M_CMPI64) {\r
+    if ((Opcode & OPCODE_M_CMPI64) != 0) {\r
       Op1 = (INT64) VmReadMem64 (VmPtr, (UINTN) Op1 + Index16);\r
     } else {\r
       Op1 = (INT64) VmReadMem32 (VmPtr, (UINTN) Op1 + Index16);\r
@@ -3417,7 +3417,7 @@ ExecuteCMPI (
     // Better not have been an index with direct. That is, CMPI R1 Index,...\r
     // is illegal.\r
     //\r
-    if (Operands & OPERAND_M_CMPI_INDEX) {\r
+    if ((Operands & OPERAND_M_CMPI_INDEX) != 0) {\r
       EbcDebugSignalException (\r
         EXCEPT_EBC_INSTRUCTION_ENCODING,\r
         EXCEPTION_FLAG_ERROR,\r
@@ -3430,7 +3430,7 @@ ExecuteCMPI (
   //\r
   // Get immediate data -- 16- or 32-bit sign extended\r
   //\r
-  if (Opcode & OPCODE_M_CMPI32_DATA) {\r
+  if ((Opcode & OPCODE_M_CMPI32_DATA) != 0) {\r
     Op2 = (INT64) VmReadImmed32 (VmPtr, Size);\r
     Size += 4;\r
   } else {\r
@@ -3444,7 +3444,7 @@ ExecuteCMPI (
   // Now do the compare\r
   //\r
   Flag = 0;\r
-  if (Opcode & OPCODE_M_CMPI64) {\r
+  if ((Opcode & OPCODE_M_CMPI64) != 0) {\r
     //\r
     // 64 bit comparison\r
     //\r
@@ -3524,7 +3524,7 @@ ExecuteCMPI (
   //\r
   // Now set the flag accordingly for the comparison\r
   //\r
-  if (Flag) {\r
+  if (Flag != 0) {\r
     VMFLAG_SET (VmPtr, VMFLAGS_CC);\r
   } else {\r
     VMFLAG_CLEAR (VmPtr, VMFLAGS_CC);\r
@@ -3633,7 +3633,7 @@ ExecuteSUB (
   IN UINT64       Op2\r
   )\r
 {\r
-  if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+  if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return (UINT64) ((INT64) ((INT64) Op1 - (INT64) Op2));\r
   } else {\r
     return (UINT64) ((INT64) ((INT32) Op1 - (INT32) Op2));\r
@@ -3662,7 +3662,7 @@ ExecuteMUL (
   IN UINT64       Op2\r
   )\r
 {\r
-  if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+  if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return MultS64x64 ((INT64)Op1, (INT64)Op2);\r
   } else {\r
     return (UINT64) ((INT64) ((INT32) Op1 * (INT32) Op2));\r
@@ -3691,7 +3691,7 @@ ExecuteMULU (
   IN UINT64       Op2\r
   )\r
 {\r
-  if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+  if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return MultU64x64 (Op1, Op2);\r
   } else {\r
     return (UINT64) ((UINT32) Op1 * (UINT32) Op2);\r
@@ -3734,7 +3734,7 @@ ExecuteDIV (
 \r
     return 0;\r
   } else {\r
-    if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+    if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
       return (UINT64) (DivS64x64Remainder (Op1, Op2, &Remainder));\r
     } else {\r
       return (UINT64) ((INT64) ((INT32) Op1 / (INT32) Op2));\r
@@ -3780,7 +3780,7 @@ ExecuteDIVU (
     //\r
     // Get the destination register\r
     //\r
-    if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+    if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
       return (UINT64) (DivU64x64Remainder ((INT64)Op1, (INT64)Op2, &Remainder));\r
     } else {\r
       return (UINT64) ((UINT32) Op1 / (UINT32) Op2);\r
@@ -3965,7 +3965,7 @@ ExecuteSHL (
   IN UINT64       Op2\r
   )\r
 {\r
-  if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+  if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return LShiftU64 (Op1, (UINTN)Op2);\r
   } else {\r
     return (UINT64) ((UINT32) ((UINT32) Op1 << (UINT32) Op2));\r
@@ -3994,7 +3994,7 @@ ExecuteSHR (
   IN UINT64       Op2\r
   )\r
 {\r
-  if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+  if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return RShiftU64 (Op1, (UINTN)Op2);\r
   } else {\r
     return (UINT64) ((UINT32) Op1 >> (UINT32) Op2);\r
@@ -4023,7 +4023,7 @@ ExecuteASHR (
   IN UINT64       Op2\r
   )\r
 {\r
-  if (*VmPtr->Ip & DATAMANIP_M_64) {\r
+  if ((*VmPtr->Ip & DATAMANIP_M_64) != 0) {\r
     return ARShiftU64 (Op1, (UINTN)Op2);\r
   } else {\r
     return (UINT64) ((INT64) ((INT32) Op1 >> (UINT32) Op2));\r
@@ -4243,7 +4243,7 @@ ExecuteDataManip (
   //\r
   // Determine if we have immediate data by the opcode\r
   //\r
-  if (Opcode & DATAMANIP_M_IMMDATA) {\r
+  if ((Opcode & DATAMANIP_M_IMMDATA) != 0) {\r
     //\r
     // Index16 if Ry is indirect, or Immed16 if Ry direct.\r
     //\r
@@ -4266,7 +4266,7 @@ ExecuteDataManip (
     //\r
     // Indirect form: @R2 Index16. Fetch as 32- or 64-bit data\r
     //\r
-    if (Opcode & DATAMANIP_M_64) {\r
+    if ((Opcode & DATAMANIP_M_64) != 0) {\r
       Op2 = VmReadMem64 (VmPtr, (UINTN) Op2);\r
     } else {\r
       //\r
@@ -4293,7 +4293,7 @@ ExecuteDataManip (
   //\r
   Op1 = VmPtr->R[OPERAND1_REGNUM (Operands)];\r
   if (OPERAND1_INDIRECT (Operands)) {\r
-    if (Opcode & DATAMANIP_M_64) {\r
+    if ((Opcode & DATAMANIP_M_64) != 0) {\r
       Op1 = VmReadMem64 (VmPtr, (UINTN) Op1);\r
     } else {\r
       if (IsSignedOp) {\r
@@ -4335,7 +4335,7 @@ ExecuteDataManip (
   //\r
   if (OPERAND1_INDIRECT (Operands)) {\r
     Op1 = VmPtr->R[OPERAND1_REGNUM (Operands)];\r
-    if (Opcode & DATAMANIP_M_64) {\r
+    if ((Opcode & DATAMANIP_M_64) != 0) {\r
       VmWriteMem64 (VmPtr, (UINTN) Op1, Op2);\r
     } else {\r
       VmWriteMem32 (VmPtr, (UINTN) Op1, (UINT32) Op2);\r
@@ -4546,7 +4546,7 @@ VmReadIndex16 (
   //\r
   // Now set the sign\r
   //\r
-  if (Index & 0x8000) {\r
+  if ((Index & 0x8000) != 0) {\r
     //\r
     // Do it the hard way to work around a bogus compiler warning\r
     //\r
@@ -4615,7 +4615,7 @@ VmReadIndex32 (
   //\r
   // Now set the sign\r
   //\r
-  if (Index & 0x80000000) {\r
+  if ((Index & 0x80000000) != 0) {\r
     Offset = Offset * -1;\r
   }\r
 \r
@@ -4679,7 +4679,7 @@ VmReadIndex64 (
   //\r
   // Now set the sign\r
   //\r
-  if (Index & 0x8000000000000000ULL) {\r
+  if ((Index & 0x8000000000000000ULL) != 0) {\r
     Offset = MultS64x64 (Offset, -1);\r
   }\r
 \r

diff --git a/MdeModulePkg/Universal/EbcDxe/EbcInt.c b/MdeModulePkg/Universal/EbcDxe/EbcInt.c
index b3fe0d5e4fc8dac152751c22b548038d2ba8875d..f48176ef1bcc8fb9f0889d25f9b2782c29f70216 100644 (file)
--- a/MdeModulePkg/Universal/EbcDxe/EbcInt.c
+++ b/MdeModulePkg/Universal/EbcDxe/EbcInt.c
@@ -769,7 +769,7 @@ EbcDebugSignalException (
   // If it's a fatal exception, then flag it in the VM context in case an\r
   // attached debugger tries to return from it.\r
   //\r
-  if (ExceptionFlags & EXCEPTION_FLAG_FATAL) {\r
+  if ((ExceptionFlags & EXCEPTION_FLAG_FATAL) != 0) {\r
     VmPtr->StopFlags |= STOPFLAG_APP_DONE;\r
   }\r
 \r

diff --git a/MdeModulePkg/Universal/EbcDxe/Ia32/EbcSupport.c b/MdeModulePkg/Universal/EbcDxe/Ia32/EbcSupport.c
index 11fa4e6659343fcc0c0fd2ee6adb0779dda1dc9b..787f5ee88540edc2c8956843efaccb7fa9ddc139 100644 (file)
--- a/MdeModulePkg/Universal/EbcDxe/Ia32/EbcSupport.c
+++ b/MdeModulePkg/Universal/EbcDxe/Ia32/EbcSupport.c
@@ -494,7 +494,7 @@ EbcCreateThunks (
   // Stick in a load of ecx with the address of appropriate VM function.\r
   //  mov ecx 12345678h  => 0xB9 0x78 0x56 0x34 0x12\r
   //\r
-  if (Flags & FLAG_THUNK_ENTRY_POINT) {\r
+  if ((Flags & FLAG_THUNK_ENTRY_POINT) != 0) {\r
     Addr = (UINT32) (UINTN) ExecuteEbcImageEntryPoint;\r
   } else {\r
     Addr = (UINT32) (UINTN) EbcInterpret;\r

diff --git a/MdeModulePkg/Universal/EbcDxe/Ipf/EbcSupport.c b/MdeModulePkg/Universal/EbcDxe/Ipf/EbcSupport.c
index 83d596c4c7e4c5fb383d60049268507bb4ae7fb0..0f46e6db7947380d5b9b1d0ecea094f540057ddf 100644 (file)
--- a/MdeModulePkg/Universal/EbcDxe/Ipf/EbcSupport.c
+++ b/MdeModulePkg/Universal/EbcDxe/Ipf/EbcSupport.c
@@ -591,7 +591,7 @@ EbcCreateThunks (
   // to the address of the entry point of the interpreter.\r
   //\r
   Ptr += 16;\r
-  if (Flags & FLAG_THUNK_ENTRY_POINT) {\r
+  if ((Flags & FLAG_THUNK_ENTRY_POINT) != 0) {\r
     Addr = (UINT64) ExecuteEbcImageEntryPoint;\r
   } else {\r
     Addr = (UINT64) EbcInterpret;\r
@@ -746,13 +746,13 @@ WriteBundle (
   //\r
   // Verify no more than 5 bits in template\r
   //\r
-  if (Template &~0x1F) {\r
+  if ((Template &~0x1F) != 0) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
   //\r
   // Verify max of 41 bits used in code\r
   //\r
-  if ((Slot0 | Slot1 | Slot2) &~0x1ffffffffff) {\r
+  if (((Slot0 | Slot1 | Slot2) &~0x1ffffffffff) != 0) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r

diff --git a/MdeModulePkg/Universal/EbcDxe/x64/EbcSupport.c b/MdeModulePkg/Universal/EbcDxe/x64/EbcSupport.c
index d72facd8e2c970ee532193ebd7e9f16f676c69e0..7f31260a7134494683e095044cd762919cedbaad 100644 (file)
--- a/MdeModulePkg/Universal/EbcDxe/x64/EbcSupport.c
+++ b/MdeModulePkg/Universal/EbcDxe/x64/EbcSupport.c
@@ -439,7 +439,7 @@ EbcCreateThunks (
   // point.\r
   // mov r11 123456789abcdef0h  => 49 BB F0 DE BC 9A 78 56 34 12\r
   //\r
-  if (Flags & FLAG_THUNK_ENTRY_POINT) {\r
+  if ((Flags & FLAG_THUNK_ENTRY_POINT) != 0) {\r
     Addr = (UINTN) ExecuteEbcImageEntryPoint;\r
   } else {\r
     Addr = (UINTN) EbcInterpret;\r