> 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

Reply via email to