tyler.durden
/
audk
mirror of https://github.com/acidanthera/audk.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

MdePkg/BaseSafeIntLib: Fix VS2015 IA32 NOOPT build failure 

v2: Add [LibraryClasses] section in INF file and refine coding style.

There are VS2015 NOOPT IA32 build failure like below in BaseSafeIntLib.
XXX.lib(XXX.obj): error LNK2001: unresolved external symbol __allmul
XXX.lib(XXX.obj): error LNK2001: unresolved external symbol __allshl
XXX.lib(XXX.obj): error LNK2001: unresolved external symbol __aullshr

This patch replaces direct shift/multiplication of 64-bit integer
with related function call to fix these failure.

Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael Kinney <michael.d.kinney@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
This commit is contained in:
Dandan Bi 2018-02-26 11:55:15 +08:00 committed by Liming Gao
parent 97e1ff1b91
commit dc9b2a5740
3 changed files with 10 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf
View File

@ -56,3 +56,6 @@
[Packages]
MdePkg/MdePkg.dec
[LibraryClasses]
BaseLib

9
MdePkg/Library/BaseSafeIntLib/SafeIntLib.c
View File

@ -28,6 +28,7 @@
#include <Base.h>
#include <Library/SafeIntLib.h>
#include <Library/BaseLib.h>
//
@ -3373,8 +3374,8 @@ SafeUint64Mult (
// b * c must be less than 2^32 or there would be bits in the high 64-bits
// then there must be no overflow of the resulting values summed up.
//
DwordA = (UINT32)(Multiplicand >> 32);
DwordC = (UINT32)(Multiplier >> 32);
DwordA = (UINT32)RShiftU64 (Multiplicand, 32);
DwordC = (UINT32)RShiftU64 (Multiplier, 32);
//
// common case -- if high dwords are both zero, no chance for overflow
@ -3409,7 +3410,7 @@ SafeUint64Mult (
// now sum them all up checking for overflow.
// shifting is safe because we already checked for overflow above
//
if (!RETURN_ERROR (SafeUint64Add (ProductBC << 32, ProductAD << 32, &UnsignedResult))) {
if (!RETURN_ERROR (SafeUint64Add (LShiftU64 (ProductBC, 32), LShiftU64 (ProductAD, 32), &UnsignedResult))) {
//
// b * d
//
@ -4075,7 +4076,7 @@ SafeInt32Mult (
OUT INT32 *Result
)
{
return SafeInt64ToInt32 (((INT64)Multiplicand) *((INT64)Multiplier), Result);
return SafeInt64ToInt32 (MultS64x64 (Multiplicand, Multiplier), Result);
}
/**

3
MdePkg/Library/BaseSafeIntLib/SafeIntLib32.c
View File

@ -28,6 +28,7 @@
#include <Base.h>
#include <Library/SafeIntLib.h>
#include <Library/BaseLib.h>
/**
INT32 -> UINTN conversion
@ -549,6 +550,6 @@ SafeIntnMult (
OUT INTN *Result
)
{
return SafeInt64ToIntn (((INT64)Multiplicand) *((INT64)Multiplier), Result);
return SafeInt64ToIntn (MultS64x64 (Multiplicand, Multiplier), Result);
}