diff --git a/gcc/ChangeLog b/gcc/ChangeLog index dc980a96a0d2c83de40444ab2a5633636d377a67..7189a66ed8395f29406c9356470eab244ff0d0d0 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2017-11-27 Jakub Jelinek <jakub@redhat.com> + + PR target/83100 + * varasm.c (bss_initializer_p): Return true for DECL_COMMON + TREE_READONLY decls. + 2017-11-27 Markus Trippelsdorf <markus@trippelsdorf.de> PR rtl-optimization/82488 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index f750ad7b333db1f5ac8b6f12873871df03bbb070..ff40f3657e34140b56f8307dda4ede4e9f4aa974 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,11 @@ +2017-11-27 Jakub Jelinek <jakub@redhat.com> + + PR target/83100 + * gcc.dg/pr83100-1.c: New test. + * gcc.dg/pr83100-2.c: New test. + * gcc.dg/pr83100-3.c: New test. + * gcc.dg/pr83100-4.c: New test. + 2017-11-26 Julia Koval <julia.koval@intel.com> * gcc.target/i386/funcspec-56.inc: Handle new march. diff --git a/gcc/testsuite/gcc.dg/pr83100-1.c b/gcc/testsuite/gcc.dg/pr83100-1.c new file mode 100644 index 0000000000000000000000000000000000000000..233c1f687b0bbce969e99a3a25e070991f4115aa --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr83100-1.c @@ -0,0 +1,7 @@ +/* PR target/83100 */ +/* { dg-do compile { target *-*-linux* *-*-gnu* } } */ +/* { dg-options "-O2 -fcommon -fdata-sections" } */ + +const int a; + +/* { dg-final { scan-assembler "comm" } } */ diff --git a/gcc/testsuite/gcc.dg/pr83100-2.c b/gcc/testsuite/gcc.dg/pr83100-2.c new file mode 100644 index 0000000000000000000000000000000000000000..6dbe93dcd16029abdef8d177e3d0bfd6b9de5272 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr83100-2.c @@ -0,0 +1,15 @@ +/* PR target/83100 */ +/* { dg-do run } */ +/* { dg-options "-O2 -fcommon -fdata-sections" } */ +/* { dg-additional-sources pr83100-3.c } */ +/* { dg-skip-if "-fdata-sections not supported" { hppa*-*-hpux* nvptx-*-* } } */ + +const int a; + +int +main () +{ + if (a != 7) + __builtin_abort (); + return 0; +} diff --git a/gcc/testsuite/gcc.dg/pr83100-3.c b/gcc/testsuite/gcc.dg/pr83100-3.c new file mode 100644 index 0000000000000000000000000000000000000000..a28e852695f4038cb0484636ee4abccce9a8a80d --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr83100-3.c @@ -0,0 +1,6 @@ +/* PR target/83100 */ +/* { dg-do compile } */ +/* { dg-options "-O2 -fcommon -fdata-sections" } */ +/* { dg-skip-if "-fdata-sections not supported" { hppa*-*-hpux* nvptx-*-* } } */ + +const int a = 7; diff --git a/gcc/testsuite/gcc.dg/pr83100-4.c b/gcc/testsuite/gcc.dg/pr83100-4.c new file mode 100644 index 0000000000000000000000000000000000000000..bb26735dd1b046975fee46fbfa1ea6813f217dbd --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr83100-4.c @@ -0,0 +1,7 @@ +/* PR target/83100 */ +/* { dg-do compile { target *-*-linux* *-*-gnu* } } */ +/* { dg-options "-O2 -fno-common -fdata-sections" } */ + +const int a; + +/* { dg-final { scan-assembler "rodata.a" { target i?86-*-* x86_64-*-* } } } */ diff --git a/gcc/varasm.c b/gcc/varasm.c index 923399d71a46ff6858f4c9d1f62f265af222531e..ff912b7fa578b023cb908ff2da06f71434e340aa 100644 --- a/gcc/varasm.c +++ b/gcc/varasm.c @@ -986,9 +986,9 @@ decode_reg_name (const char *name) bool bss_initializer_p (const_tree decl) { - /* Do not put constants into the .bss section, they belong in a readonly - section. */ - return (!TREE_READONLY (decl) + /* Do not put non-common constants into the .bss section, they belong in + a readonly section. */ + return ((!TREE_READONLY (decl) || DECL_COMMON (decl)) && (DECL_INITIAL (decl) == NULL /* In LTO we have no errors in program; error_mark_node is used to mark offlined constructors. */