Add comments for clarification about memset implementation.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Long, Qin <qin.long@intel.com>
Reviewed-by: Ye, Ting <ting.ye@intel.com>
Reviewed-by: Fu, Siyuan <siyuan.fu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15662 6f19259b-4bc3-4df7-8a09-765794883524

1 files changed, 8 insertions, 1 deletions

diff --git a/CryptoPkg/Library/IntrinsicLib/MemoryIntrinsics.c b/CryptoPkg/Library/IntrinsicLib/MemoryIntrinsics.c
index afaa0b74ee..9d6867ebce 100644
--- a/CryptoPkg/Library/IntrinsicLib/MemoryIntrinsics.c
+++ b/CryptoPkg/Library/IntrinsicLib/MemoryIntrinsics.c
@@ -2,7 +2,7 @@
   Intrinsic Memory Routines Wrapper Implementation for OpenSSL-based

   Cryptographic Library.

 

-Copyright (c) 2010, Intel Corporation. All rights reserved.<BR>

+Copyright (c) 2010 - 2014, Intel Corporation. All rights reserved.<BR>

 This program and the accompanying materials

 are licensed and made available under the terms and conditions of the BSD License

 which accompanies this distribution.  The full text of the license may be found at

@@ -24,6 +24,13 @@ int _fltused = 1;
 void * memset (void *dest, char ch, unsigned int count)

 {

   //

+  // NOTE: Here we use one base implementation for memset, instead of the direct

+  //       optimized SetMem() wrapper. Because the IntrinsicLib has to be built 

+  //       without whole program optimization option, and there will be some 

+  //       potential register usage errors when calling other optimized codes.

+  //

+

+  //

   // Declare the local variables that actually move the data elements as

   // volatile to prevent the optimizer from replacing this function with

   // the intrinsic memset()