From b5d5f0d6d03083ab37fc960f3b3f3302922fdd82 Mon Sep 17 00:00:00 2001 From: mkuvyrkov Date: Wed, 12 May 2010 11:05:19 +0000 Subject: [PATCH] * targhooks.c (default_stack_protect_guard): Avoid sharing RTL for __stack_chk_guard. * gcc.target/m68k/20100512-1.c: New. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@159310 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 5 +++++ gcc/targhooks.c | 7 +++++++ gcc/testsuite/ChangeLog | 4 ++++ gcc/testsuite/gcc.target/m68k/20100512-1.c | 16 ++++++++++++++++ 4 files changed, 32 insertions(+) create mode 100644 gcc/testsuite/gcc.target/m68k/20100512-1.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index b9b67952f1d..b63cde5115b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2010-05-12 Maxim Kuvyrkov + + * targhooks.c (default_stack_protect_guard): Avoid sharing RTL + for __stack_chk_guard. + 2010-05-11 Jakub Jelinek * c-opts.c (c_common_parse_file): If start_end_main_source_file, diff --git a/gcc/targhooks.c b/gcc/targhooks.c index 8df0de421f8..2c0a65dc926 100644 --- a/gcc/targhooks.c +++ b/gcc/targhooks.c @@ -499,6 +499,8 @@ default_stack_protect_guard (void) if (t == NULL) { + rtx x; + t = build_decl (UNKNOWN_LOCATION, VAR_DECL, get_identifier ("__stack_chk_guard"), ptr_type_node); @@ -510,6 +512,11 @@ default_stack_protect_guard (void) DECL_ARTIFICIAL (t) = 1; DECL_IGNORED_P (t) = 1; + /* Do not share RTL as the declaration is visible outside of + current function. */ + x = DECL_RTL (t); + RTX_FLAG (x, used) = 1; + stack_chk_guard_decl = t; } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index a1275c845c7..4daeaa6b6b7 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2010-05-12 Maxim Kuvyrkov + + * gcc.target/m68k/20100512-1.c: New. + 2010-05-12 Eric Botcazou * gnat.dg/lto8.adb: New test. diff --git a/gcc/testsuite/gcc.target/m68k/20100512-1.c b/gcc/testsuite/gcc.target/m68k/20100512-1.c new file mode 100644 index 00000000000..d07bb519abc --- /dev/null +++ b/gcc/testsuite/gcc.target/m68k/20100512-1.c @@ -0,0 +1,16 @@ +/* { dg-do compile } */ +/* { dg-options "-O2 -fstack-protector" } */ +/* { dg-require-effective-target fstack_protector } */ +/* There should be 2 references to __stack_chk_guard in every function. */ +/* { dg-final { scan-assembler-times "__stack_chk_guard" 4 } } */ + +#include +#include +void doTest1(void) { + volatile char foo[10]; + memset((void *)foo, 1, 100); +} +void doTest2(void) { + volatile char foo[10]; + memset((void *)foo, 1, 100); +} -- 2.11.0