From 1b98d6ce44baea807223c67ddf97cc0dc9108741 Mon Sep 17 00:00:00 2001 From: "Long, Qin" Date: Wed, 16 Jul 2014 02:48:20 +0000 Subject: [PATCH] Add comments for clarification about memset implementation. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Long, Qin Reviewed-by: Ye, Ting Reviewed-by: Fu, Siyuan git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15662 6f19259b-4bc3-4df7-8a09-765794883524 --- CryptoPkg/Library/IntrinsicLib/MemoryIntrinsics.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) 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.
+Copyright (c) 2010 - 2014, Intel Corporation. All rights reserved.
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 @@ -23,6 +23,13 @@ int _fltused = 1; /* Sets buffers to a specified character */ 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 -- 2.39.2