Re: [edk2] [PATCH v2 1/3] MdePkg/BaseMemoryLib: widen aligned accesses to 32 or 64 bits

2016-09-07 Thread Ard Biesheuvel
On 7 September 2016 at 04:03, Gao, Liming wrote: > Ard: > In InternalMemSetMem, Value64 = (((UINT64)Value32) << 32) | Value32; may > cause the below link error with VS IA32 build. It (<<) should be replaced by > BaseLib LShift API > > BaseMemoryLib.lib(SetMem.obj) : error

Re: [edk2] [PATCH v2 1/3] MdePkg/BaseMemoryLib: widen aligned accesses to 32 or 64 bits

2016-09-06 Thread Gao, Liming
Ard: In InternalMemSetMem, Value64 = (((UINT64)Value32) << 32) | Value32; may cause the below link error with VS IA32 build. It (<<) should be replaced by BaseLib LShift API BaseMemoryLib.lib(SetMem.obj) : error LNK2001: unresolved external symbol __allshl Thanks Liming > -Original