Liming,

Looks good.

Reviewed-by: Andrew Fish <af...@apple.com>

On Sep 4, 2014, at 7:28 PM, Gao, Liming <liming....@intel.com> wrote:

> Andrew:
>   Could you help review this patch?
>  
> MdeModulePkg: Fix XCODE Link Issue in DxeCore
>  
> In XCODE tool chain, the 64-bit bit wise and operation is causing the 
> compiler to emit an __umoddi3.
> This patch uses BaseLib API to replace 64-bit bit operation.
>  
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Gao, Liming <liming....@intel.com>
> CC: af...@apple.com
>  
> Index: Stall.c
> ===================================================================
> --- Stall.c              (revision 15972)
> +++ Stall.c           (working copy)
> @@ -32,7 +32,7 @@
>    IN UINT64  Counter
>    )
> {
> -  while ((Counter & 0xffffffff00000000ULL) != 0) {
> +  while (RShiftU64 (Counter, 32) > 0) {
>      gMetronome->WaitForTick (gMetronome, 0xffffffff);
>      Counter -= 0xffffffff;
>    }
>  
> Thanks
> Liming

------------------------------------------------------------------------------
Slashdot TV.  
Video for Nerds.  Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to