diff options
| author | Corinna Vinschen <corinna@vinschen.de> | 2015-08-27 13:55:22 +0300 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2015-08-27 13:55:22 +0300 |
| commit | e6d9af11f1f21a70a6d947ab452ac4d82caace64 (patch) | |
| tree | beef2ef20e0d71851f7c546d84ae9c4696b035a5 /newlib/ChangeLog | |
| parent | 892cfcb7c2fbdc3b2b09c95a7a6c2065e5e0cfae (diff) | |
strlen-armv7.S: Fix preprocessor check
Hi!
I've got the situation, that the function strlen() occurs twice in libc.a
(building newlib for ARM-V7a and Size-Optimized).
In newlib/libc/machine/arm/strlen.c there are the pre-processor stetements ...
#if defined (__OPTIMIZE_SIZE__) || defined (PREFER_SIZE_OVER_SPEED) || \
(defined (__thumb__) && !defined (__thumb2__))
/*...*/
#else
#if !(defined(_ISA_ARM_7) || defined(__ARM_ARCH_6T2__))
/*...*/
#endif
and in newlib/libc/machine/arm/strlen-armv7.S the "exclude" begins with
/* NOTE: This ifdef MUST match the ones in arm/strlen.c
We fallback to the one in arm/strlen.c for size optimised or
for older architectures. */
#if defined(_ISA_ARM_7) || defined(__ARM_ARCH_6T2__) && \
!(defined (__OPTIMIZE_SIZE__) || defined (PREFER_SIZE_OVER_SPEED) || \
(defined (__thumb__) && !defined (__thumb2__)))
But this is not completely contrary to arm/strlen.c (see above)!
To fix the logical statement in arm/strlen-armv7.S there are parentheses needed
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
Diffstat (limited to 'newlib/ChangeLog')
| -rw-r--r-- | newlib/ChangeLog | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/newlib/ChangeLog b/newlib/ChangeLog index afabb04aa..674ff12be 100644 --- a/newlib/ChangeLog +++ b/newlib/ChangeLog @@ -1,3 +1,8 @@ +2015-08-27 Markus Eisenmann <meisenmann.lba@fh-salzburg.ac.at> + + * libc/machine/arm/strlen-armv7.S: Fix prepocessor check to avoid + double definition of strlen. + 2015-08-27 Andre Vieira <andre.simoesdiasvieira@arm.com> * libc/include/sys/_intsup.h: Defined new __INTxx, __FASTxx and |