From f524b013bbac559d91842545e747997180cb1067 Mon Sep 17 00:00:00 2001 From: midipix Date: Sat, 31 May 2025 13:51:57 +0000 Subject: code base: utilize the visibility attribute also on supporting PE targets. --- src/internal/slibtool_visibility_impl.h | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/internal/slibtool_visibility_impl.h b/src/internal/slibtool_visibility_impl.h index 2bdded0..44ab2dd 100644 --- a/src/internal/slibtool_visibility_impl.h +++ b/src/internal/slibtool_visibility_impl.h @@ -7,20 +7,20 @@ /* attribute not only redundant, but also tricky if not properly */ /* supported by the toolchain. */ /* */ -/* When targeting Midipix, __PE__, __dllexport and __dllimport are */ -/* always defined by the toolchain. Otherwise, the absnece of these */ -/* macros has been detected by sofort's ccenv.sh during ./configure, */ -/* and they have accordingly been added to CFLAGS_OS. */ +/* When targeting Midipix hosts, where elf-like visibility is fully */ +/* supported and may be detected via the __PE_VISIBILITY__ macro, */ +/* we utilize the attribute to render private symbols invisibile */ +/* to dlsym(), as well as reduce the size of the .gotstrs section. */ /**********************************************************************/ -#ifdef __PE__ +#if defined(__PE_VISIBILITY__) +#define slbt_hidden _ATTR_VISIBILITY_HIDDEN +#elif defined(__PE__) #define slbt_hidden -#else -#ifdef _ATTR_VISIBILITY_HIDDEN +#elif defined(_ATTR_VISIBILITY_HIDDEN) #define slbt_hidden _ATTR_VISIBILITY_HIDDEN #else #define slbt_hidden #endif -#endif #endif -- cgit v1.2.3