Skip to content

Commit

Permalink
StdLib/LibC: Fix ARM symbol problems
Browse files Browse the repository at this point in the history
StdLibPrivateInternalFiles: Add AArch64 support

Disable 'memcpy' symbol for ARM in LibC. It is already provided by the ARM compiler intrinsics library.
Add missing 'strtold' symbol. Use the same fallback as IPF for now.
Use Include definitions as provided by ARM version.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Harry Liebel <[email protected]>
Reviewed-By: Olivier Martin <[email protected]>
Reviewed-By: Daryl McDaniel <[email protected]>


git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15858 6f19259b-4bc3-4df7-8a09-765794883524
  • Loading branch information
hliebel authored and darylm503 committed Aug 20, 2014
1 parent df294f7 commit 1bc81f4
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
5 changes: 3 additions & 2 deletions StdLib/LibC/String/Copying.c
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,12 @@
#include <stdlib.h>
#include <string.h>

/** Do not define memcpy for IPF+GCC builds.
/** Do not define memcpy for IPF+GCC or ARM+GCC builds.
For IPF, using a GCC compiler, the memcpy function is converted to
CopyMem by objcpy during build.
For ARM, the memcpy function is provided by the CompilerIntrinsics library.
**/
#if !(defined(MDE_CPU_IPF) && defined(__GNUC__))
#if !((defined(MDE_CPU_IPF) || defined(MDE_CPU_ARM)) && defined(__GNUC__))
/** The memcpy function copies n characters from the object pointed to by s2
into the object pointed to by s1.
Expand Down
1 change: 1 addition & 0 deletions StdLib/LibC/gdtoa/gdtoa.inf
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@

[Sources.ARM]
strtof.c
Ipf/strtold.c

[Sources]
strtod.c # Public interfaces
Expand Down
4 changes: 4 additions & 0 deletions StdLibPrivateInternalFiles/DoNotUse.dec
Original file line number Diff line number Diff line change
Expand Up @@ -33,3 +33,7 @@

[Includes.ARM]
Include/Arm

[Includes.AARCH64]
# Use the ARM headers for now.
Include/Arm

0 comments on commit 1bc81f4

Please sign in to comment.