On 28 August 2017 at 04:27, Gao, Liming <[email protected]> wrote: > Laszlo: > I will update the patch with your comments. > > Ard: > We collect the size impact in Ovmf platform. Its impact is small. So, my > patch enable this option for all targets. Below is the data collected on > OvmfIa32X64.dsc with GCC5 tool chain. Raw image is a little bigger. But, the > compressed size is a little smaller. > > PEIFV 178472 --> 179176 +704 (Bytes) > DXEFV 4062512 --> 4075056 +12544 (Bytes) > FVMAIN_COMPACT 1190896 --> 1184920 -5976 (Bytes) >
I don't care deeply, but given that --whole-archive is used as a debug feature (we don't actually need the whole archive, but we want to force a linker error if duplicate symbols exist), I don't think it belongs in the RELEASE configuration. _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
