diff options
author | Iain Buclaw <ibuclaw@gdcproject.org> | 2021-03-13 17:05:52 +0100 |
---|---|---|
committer | Iain Buclaw <ibuclaw@gdcproject.org> | 2021-03-26 16:14:18 +0100 |
commit | 8f5e18db259c8a9790feb1d73bb0348182264f15 (patch) | |
tree | 20b071e888a2c7ea308314634156fe7b95a69919 /gcc/tree-emutls.c | |
parent | 433b6b12dfa5ef109a5d8d40e7b3695dcb749e58 (diff) | |
download | gcc-8f5e18db259c8a9790feb1d73bb0348182264f15.zip gcc-8f5e18db259c8a9790feb1d73bb0348182264f15.tar.gz gcc-8f5e18db259c8a9790feb1d73bb0348182264f15.tar.bz2 |
Fix ICE: in function_and_variable_visibility, at ipa-visibility.c:795 [PR99466]
In get_emutls_init_templ_addr, only thread-local declarations that were
DECL_ONE_ONLY would have a public initializer symbol, ignoring variables
that were declared with __attribute__((weak)).
gcc/ChangeLog:
PR ipa/99466
* tree-emutls.c (get_emutls_init_templ_addr): Mark initializer of weak
TLS declarations as public.
gcc/testsuite/ChangeLog:
PR ipa/99466
* gcc.dg/tls/pr99466-1.c: New test.
* gcc.dg/tls/pr99466-2.c: New test.
Diffstat (limited to 'gcc/tree-emutls.c')
-rw-r--r-- | gcc/tree-emutls.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/gcc/tree-emutls.c b/gcc/tree-emutls.c index f105338..1c9c5d5 100644 --- a/gcc/tree-emutls.c +++ b/gcc/tree-emutls.c @@ -242,16 +242,18 @@ get_emutls_init_templ_addr (tree decl) DECL_PRESERVE_P (to) = DECL_PRESERVE_P (decl); DECL_WEAK (to) = DECL_WEAK (decl); - if (DECL_ONE_ONLY (decl)) + if (DECL_ONE_ONLY (decl) || DECL_WEAK (decl)) { TREE_STATIC (to) = TREE_STATIC (decl); TREE_PUBLIC (to) = TREE_PUBLIC (decl); DECL_VISIBILITY (to) = DECL_VISIBILITY (decl); - make_decl_one_only (to, DECL_ASSEMBLER_NAME (to)); } else TREE_STATIC (to) = 1; + if (DECL_ONE_ONLY (decl)) + make_decl_one_only (to, DECL_ASSEMBLER_NAME (to)); + DECL_VISIBILITY_SPECIFIED (to) = DECL_VISIBILITY_SPECIFIED (decl); DECL_INITIAL (to) = DECL_INITIAL (decl); DECL_INITIAL (decl) = NULL; |