From ed3689248a072f10638651fc540f8f3d52b9d3c0 Mon Sep 17 00:00:00 2001 From: Yvan Roux Date: Thu, 3 Nov 2016 13:14:39 +0100 Subject: Fix BZ 2575 Change-Id: Ie8f19fcbb851bec5c2dbcab309b412a090e21417 --- gcc/testsuite/g++.dg/opt/pr78201.C | 11 +++++++++++ gcc/varasm.c | 9 +++++---- 2 files changed, 16 insertions(+), 4 deletions(-) create mode 100644 gcc/testsuite/g++.dg/opt/pr78201.C diff --git a/gcc/testsuite/g++.dg/opt/pr78201.C b/gcc/testsuite/g++.dg/opt/pr78201.C new file mode 100644 index 00000000000..a1184d0edfe --- /dev/null +++ b/gcc/testsuite/g++.dg/opt/pr78201.C @@ -0,0 +1,11 @@ +// PR middle-end/78201 +// { dg-do compile } +// { dg-options "-O2" } + +struct B { long d (); } *c; +long e; + +void f() { + char a[e] = ""; + c && c->d(); +} diff --git a/gcc/varasm.c b/gcc/varasm.c index 6a7ffc22e22..1c711b1d8c8 100644 --- a/gcc/varasm.c +++ b/gcc/varasm.c @@ -6804,11 +6804,12 @@ default_use_anchors_for_symbol_p (const_rtx symbol) return false; /* Don't use section anchors for decls that won't fit inside a single - anchor range to reduce the amount of instructions require to refer + anchor range to reduce the amount of instructions required to refer to the entire declaration. */ - if (decl && DECL_SIZE (decl) - && tree_to_shwi (DECL_SIZE (decl)) - >= (targetm.max_anchor_offset * BITS_PER_UNIT)) + if (DECL_SIZE_UNIT (decl) == NULL_TREE + || !tree_fits_uhwi_p (DECL_SIZE_UNIT (decl)) + || (tree_to_uhwi (DECL_SIZE_UNIT (decl)) + >= (unsigned HOST_WIDE_INT) targetm.max_anchor_offset)) return false; } -- cgit v1.2.3