diff options
| author | Jan Beulich <jbeulich@suse.com> | 2016-07-01 14:23:24 +0000 |
|---|---|---|
| committer | Jan Beulich <jbeulich@gcc.gnu.org> | 2016-07-01 14:23:24 +0000 |
| commit | 059541fd113a360000b08753e355898cdeeaa431 (patch) | |
| tree | d6335a7ae0fa08c73f02972395d93d140ba010c9 /gcc/varasm.c | |
| parent | 51433308bb18319f173f75339e1ac1eb2fe24e80 (diff) | |
| download | gcc-059541fd113a360000b08753e355898cdeeaa431.zip gcc-059541fd113a360000b08753e355898cdeeaa431.tar.gz gcc-059541fd113a360000b08753e355898cdeeaa431.tar.bz2 | |
check initializer to be zero in .bss-like sections
Just like gas, which has recently learned to reject such initializers,
gcc shouldn't accept such either.
gcc/
2016-07-01 Jan Beulich <jbeulich@suse.com>
* varasm.c (get_variable_section): Validate initializer in
named .bss-like sections.
gcc/testsuite/
2016-07-01 Jan Beulich <jbeulich@suse.com>
* gcc.dg/bss.c: New.
From-SVN: r237913
Diffstat (limited to 'gcc/varasm.c')
| -rw-r--r-- | gcc/varasm.c | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/gcc/varasm.c b/gcc/varasm.c index de8bcd6..6a8fb81 100644 --- a/gcc/varasm.c +++ b/gcc/varasm.c @@ -1150,7 +1150,18 @@ get_variable_section (tree decl, bool prefer_noswitch_p) resolve_unique_section (decl, reloc, flag_data_sections); if (IN_NAMED_SECTION (decl)) - return get_named_section (decl, NULL, reloc); + { + section *sect = get_named_section (decl, NULL, reloc); + + if ((sect->common.flags & SECTION_BSS) && !bss_initializer_p (decl)) + { + error_at (DECL_SOURCE_LOCATION (decl), + "only zero initializers are allowed in section %qs", + sect->named.name); + DECL_INITIAL (decl) = error_mark_node; + } + return sect; + } if (ADDR_SPACE_GENERIC_P (as) && !DECL_THREAD_LOCAL_P (decl) |