> For AARCH64, that would be sufficient, but for ARM there are plenty of > other compiler intrinsics, so you should always include the > CompilerIntrinsicsLib.inf anyway (which the upstream DSCs are already > doing)
since there already seems to be AArch64 support in ArmPkg/Library/CompilerIntrinsicsLib which supports memcpy, the fix for this bug probably should be to add this lib to the platform's dsc instead of editing the compiler options. On Thu, May 12, 2016 at 2:21 PM, Ard Biesheuvel <ard.biesheu...@linaro.org> wrote: > On 12 May 2016 at 14:17, Michael Zimmermann <sigmaepsilo...@gmail.com> > wrote: > >> The EDK2 build system already does that for IPF under GCC> > >> > >> DEFINE GCC_IPF_SYMRENAME_FLAGS = --redefine-sym memcpy=CopyMem > > > > this probably should be moved to the global GCC flags then, also GCC can > > generate other symbols too, i.e. "struct x = {0};" results in memset. > (this > > one would need a wrapper though because SetMem uses a different argument > > order). > > > > For AARCH64, that would be sufficient, but for ARM there are plenty of > other compiler intrinsics, so you should always include the > CompilerIntrinsicsLib.inf anyway (which the upstream DSCs are already > doing) > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel