* [PATCH] ArmPkg/CompilerIntrinsicsLib: use Clang-compatible 'weak' attribute
@ 2016-09-06 14:27 Ard Biesheuvel
2016-09-06 14:38 ` Leif Lindholm
0 siblings, 1 reply; 2+ messages in thread
From: Ard Biesheuvel @ 2016-09-06 14:27 UTC (permalink / raw)
To: edk2-devel, leif.lindholm; +Cc: Ard Biesheuvel
Clang does not like separate definitions for the __alias__ and the
__weak__ attributes, so merge the definitions into one.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
Note that the CI job is currently broken due to this clang issue.
ArmPkg/Library/CompilerIntrinsicsLib/memset.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c b/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
index 0d837fcecf09..7271b4be6f96 100644
--- a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
+++ b/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
@@ -26,20 +26,17 @@ void *__memset(void *s, int c, size_t n)
return s;
}
-__attribute__((__alias__("__memset")))
-void *memset(void *dest, int c, size_t n);
-
-#ifdef __arm__
-
//
// Other modules (such as CryptoPkg/IntrinsicLib) may provide another
// implementation of memset(), which may conflict with this one if this
// object was pulled into the link due to the definitions below. So make
// our memset() 'weak' to let the other implementation take precedence.
//
-__attribute__((__weak__))
+__attribute__((__weak__, __alias__("__memset")))
void *memset(void *dest, int c, size_t n);
+#ifdef __arm__
+
void __aeabi_memset(void *dest, size_t n, int c)
{
__memset(dest, c, n);
--
2.7.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ArmPkg/CompilerIntrinsicsLib: use Clang-compatible 'weak' attribute
2016-09-06 14:27 [PATCH] ArmPkg/CompilerIntrinsicsLib: use Clang-compatible 'weak' attribute Ard Biesheuvel
@ 2016-09-06 14:38 ` Leif Lindholm
0 siblings, 0 replies; 2+ messages in thread
From: Leif Lindholm @ 2016-09-06 14:38 UTC (permalink / raw)
To: Ard Biesheuvel; +Cc: edk2-devel
On Tue, Sep 06, 2016 at 03:27:31PM +0100, Ard Biesheuvel wrote:
> Clang does not like separate definitions for the __alias__ and the
> __weak__ attributes, so merge the definitions into one.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>
> Note that the CI job is currently broken due to this clang issue.
>
> ArmPkg/Library/CompilerIntrinsicsLib/memset.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c b/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
> index 0d837fcecf09..7271b4be6f96 100644
> --- a/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
> +++ b/ArmPkg/Library/CompilerIntrinsicsLib/memset.c
> @@ -26,20 +26,17 @@ void *__memset(void *s, int c, size_t n)
> return s;
> }
>
> -__attribute__((__alias__("__memset")))
> -void *memset(void *dest, int c, size_t n);
> -
> -#ifdef __arm__
> -
> //
> // Other modules (such as CryptoPkg/IntrinsicLib) may provide another
> // implementation of memset(), which may conflict with this one if this
> // object was pulled into the link due to the definitions below. So make
> // our memset() 'weak' to let the other implementation take precedence.
> //
> -__attribute__((__weak__))
> +__attribute__((__weak__, __alias__("__memset")))
> void *memset(void *dest, int c, size_t n);
>
> +#ifdef __arm__
> +
> void __aeabi_memset(void *dest, size_t n, int c)
> {
> __memset(dest, c, n);
> --
> 2.7.4
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-09-06 14:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-06 14:27 [PATCH] ArmPkg/CompilerIntrinsicsLib: use Clang-compatible 'weak' attribute Ard Biesheuvel
2016-09-06 14:38 ` Leif Lindholm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox