1. Fixed incorrect ASSERT conditions in wrapper functions.
authorbxing <bxing@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 30 May 2006 05:42:49 +0000 (05:42 +0000)
committerbxing <bxing@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 30 May 2006 05:42:49 +0000 (05:42 +0000)
2. Removed unnecessary SUB instructions in CompareMem.asm

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

20 files changed:
MdePkg/Library/BaseMemoryLib/SetMem16Wrapper.c
MdePkg/Library/BaseMemoryLib/SetMem32Wrapper.c
MdePkg/Library/BaseMemoryLib/SetMem64Wrapper.c
MdePkg/Library/BaseMemoryLibMmx/Ia32/CompareMem.asm
MdePkg/Library/BaseMemoryLibMmx/SetMem16Wrapper.c
MdePkg/Library/BaseMemoryLibMmx/SetMem32Wrapper.c
MdePkg/Library/BaseMemoryLibMmx/SetMem64Wrapper.c
MdePkg/Library/BaseMemoryLibRepStr/SetMem16Wrapper.c
MdePkg/Library/BaseMemoryLibRepStr/SetMem32Wrapper.c
MdePkg/Library/BaseMemoryLibRepStr/SetMem64Wrapper.c
MdePkg/Library/BaseMemoryLibSse2/Ia32/CompareMem.asm
MdePkg/Library/BaseMemoryLibSse2/SetMem16Wrapper.c
MdePkg/Library/BaseMemoryLibSse2/SetMem32Wrapper.c
MdePkg/Library/BaseMemoryLibSse2/SetMem64Wrapper.c
MdePkg/Library/PeiMemoryLib/SetMem16Wrapper.c
MdePkg/Library/PeiMemoryLib/SetMem32Wrapper.c
MdePkg/Library/PeiMemoryLib/SetMem64Wrapper.c
MdePkg/Library/UefiMemoryLib/SetMem16Wrapper.c
MdePkg/Library/UefiMemoryLib/SetMem32Wrapper.c
MdePkg/Library/UefiMemoryLib/SetMem64Wrapper.c

index 827e6ec..3acbdac 100644 (file)
@@ -54,8 +54,8 @@ SetMem16 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4b9d241..620b69a 100644 (file)
@@ -54,8 +54,8 @@ SetMem32 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index dbf3b62..73c28af 100644 (file)
@@ -54,8 +54,8 @@ SetMem64 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4e35d1c..3cf6fe7 100644 (file)
@@ -39,7 +39,6 @@ InternalMemCompareMem   PROC    USES    esi edi
     movzx   eax, byte ptr [esi - 1]\r
     movzx   edx, byte ptr [edi - 1]\r
     sub     eax, edx\r
-    sub     eax, edx\r
     ret\r
 InternalMemCompareMem   ENDP\r
 \r
index 827e6ec..3acbdac 100644 (file)
@@ -54,8 +54,8 @@ SetMem16 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4b9d241..620b69a 100644 (file)
@@ -54,8 +54,8 @@ SetMem32 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index dbf3b62..73c28af 100644 (file)
@@ -54,8 +54,8 @@ SetMem64 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 827e6ec..3acbdac 100644 (file)
@@ -54,8 +54,8 @@ SetMem16 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4b9d241..620b69a 100644 (file)
@@ -54,8 +54,8 @@ SetMem32 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index dbf3b62..73c28af 100644 (file)
@@ -54,8 +54,8 @@ SetMem64 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4e35d1c..3cf6fe7 100644 (file)
@@ -39,7 +39,6 @@ InternalMemCompareMem   PROC    USES    esi edi
     movzx   eax, byte ptr [esi - 1]\r
     movzx   edx, byte ptr [edi - 1]\r
     sub     eax, edx\r
-    sub     eax, edx\r
     ret\r
 InternalMemCompareMem   ENDP\r
 \r
index 827e6ec..3acbdac 100644 (file)
@@ -54,8 +54,8 @@ SetMem16 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4b9d241..620b69a 100644 (file)
@@ -54,8 +54,8 @@ SetMem32 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index dbf3b62..73c28af 100644 (file)
@@ -54,8 +54,8 @@ SetMem64 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 827e6ec..3acbdac 100644 (file)
@@ -54,8 +54,8 @@ SetMem16 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4b9d241..620b69a 100644 (file)
@@ -54,8 +54,8 @@ SetMem32 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index dbf3b62..73c28af 100644 (file)
@@ -54,8 +54,8 @@ SetMem64 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 827e6ec..3acbdac 100644 (file)
@@ -54,8 +54,8 @@ SetMem16 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index 4b9d241..620b69a 100644 (file)
@@ -54,8 +54,8 @@ SetMem32 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r
index dbf3b62..73c28af 100644 (file)
@@ -54,8 +54,8 @@ SetMem64 (
 {\r
   ASSERT (Buffer != NULL);\r
   ASSERT (Length <= MAX_ADDRESS - (UINTN)Buffer + 1);\r
-  ASSERT ((((UINTN)Buffer) & 1) != 0);\r
-  ASSERT ((Length & 1) != 0);\r
+  ASSERT ((((UINTN)Buffer) & (sizeof (Value) - 1)) != 0);\r
+  ASSERT ((Length & (sizeof (Value) - 1)) != 0);\r
 \r
   if ((Length /= sizeof (Value)) == 0) {\r
     return Buffer;\r