git: f2ef124469ae - stable/12 - linker_set: fix globl/weak symbol redefinitions to work on clang 12
Mateusz Guzik
mjg at FreeBSD.org
Sat Jun 26 16:28:22 UTC 2021
The branch stable/12 has been updated by mjg:
URL: https://cgit.FreeBSD.org/src/commit/?id=f2ef124469aef4ba17fb201d5412b04cc298bd2a
commit f2ef124469aef4ba17fb201d5412b04cc298bd2a
Author: Greg V <greg at unrelenting.technology>
AuthorDate: 2021-04-20 00:47:15 +0000
Commit: Mateusz Guzik <mjg at FreeBSD.org>
CommitDate: 2021-06-26 15:52:06 +0000
linker_set: fix globl/weak symbol redefinitions to work on clang 12
In clang 12.0.0.rc2, going from weak to global is now a hard error:
```
/usr/src/stand/libsa/amd64/_setjmp.S:67:25: error: _longjmp changed binding to STB_GLOBAL
.text; .p2align 4,0x90; .globl _longjmp; .type _longjmp, at function; _longjmp:; .cfi_startproc
```
And the other way is a warning, but we have -Werror:
```
error: __start_set_Xcommand_set changed binding to STB_WEAK [-Werror,-Winline-asm]
error: __stop_set_Xcommand_set changed binding to STB_WEAK [-Werror,-Winline-asm]
```
ref: https://reviews.llvm.org/D90108
Reviewed By: arichardson
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D29159
(cherry picked from commit 32231805fbe2b9438c2de50c229b43c016207a08)
---
sys/sys/cdefs.h | 4 ++--
sys/sys/linker_set.h | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/sys/sys/cdefs.h b/sys/sys/cdefs.h
index a2625a1ed5b4..c8485fa58f54 100644
--- a/sys/sys/cdefs.h
+++ b/sys/sys/cdefs.h
@@ -569,8 +569,8 @@
#endif /* __STDC__ */
#endif /* __GNUC__ || __INTEL_COMPILER */
-#define __GLOBL1(sym) __asm__(".globl " #sym)
-#define __GLOBL(sym) __GLOBL1(sym)
+#define __GLOBL(sym) __asm__(".globl " __XSTRING(sym))
+#define __WEAK(sym) __asm__(".weak " __XSTRING(sym))
#if defined(__GNUC__) || defined(__INTEL_COMPILER)
#define __IDSTRING(name,string) __asm__(".ident\t\"" string "\"")
diff --git a/sys/sys/linker_set.h b/sys/sys/linker_set.h
index 9f49b802887b..b67ed51551a6 100644
--- a/sys/sys/linker_set.h
+++ b/sys/sys/linker_set.h
@@ -57,8 +57,8 @@
*/
#ifdef __GNUCLIKE___SECTION
#define __MAKE_SET_QV(set, sym, qv) \
- __GLOBL(__CONCAT(__start_set_,set)); \
- __GLOBL(__CONCAT(__stop_set_,set)); \
+ __WEAK(__CONCAT(__start_set_,set)); \
+ __WEAK(__CONCAT(__stop_set_,set)); \
static void const * qv \
__set_##set##_sym_##sym __section("set_" #set) \
__used = &(sym)
More information about the dev-commits-src-all
mailing list