On 2016-09-29 07:12:43, Liming Gao wrote:
> After enable optimization, C tool build time is reduced by ~50%. Please see 
> below example to use LzmaCompress to compress OVMF DXEFV.
> 
> Tool                         Compression time Decompression time
> LzmaCompress (GCC O0)        3.476s           0.204s
> LzmaCompress (GCC Ofast)     1.655s           0.107s
> LzmaCompress (VS Od)         3.047s           0.210s
> LzmaCompress (VS O2)         1.551s           0.126s
> 
> Liming Gao (4):
>   BaseTools EfiLdrImage: Remove unnecessary exit (0)

Reviewed-by: Jordan Justen <[email protected]>

>   BaseTools Makefile: Enable O2 option to replace Od for VS tool chain

I assume these flag changes work fine with the older versions of VS?
If so,

Acked-by: Jordan Justen <[email protected]>

>   BaseTools GenVtf: Initialize the return point as NULL

Reviewed-by: Jordan Justen <[email protected]>

>   BaseTools Makefile: Enable Ofast option for GCC tool chain

If changed to -O2 (Thanks for the suggesion Laszlo):

Reviewed-by: Jordan Justen <[email protected]>

> 
>  BaseTools/Source/C/EfiLdrImage/EfiLdrImage.c | 1 -
>  BaseTools/Source/C/GenVtf/GenVtf.c           | 1 +
>  BaseTools/Source/C/Makefiles/header.makefile | 6 +++---
>  BaseTools/Source/C/Makefiles/ms.app          | 2 +-
>  BaseTools/Source/C/Makefiles/ms.common       | 4 ++--
>  5 files changed, 7 insertions(+), 7 deletions(-)
> 
> -- 
> 2.8.0.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> [email protected]
> https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to