public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] Don't use priority {cd}tors if not supported by a target (PR, gcov-profile/78086)
@ 2016-10-31  9:16 Martin Liška
  2016-10-31 10:07 ` Rainer Orth
  2016-11-25 10:48 ` Jan Hubicka
  0 siblings, 2 replies; 6+ messages in thread
From: Martin Liška @ 2016-10-31  9:16 UTC (permalink / raw)
  To: GCC Patches; +Cc: danglin, Nathan Sidwell

[-- Attachment #1: Type: text/plain, Size: 384 bytes --]

Hi.

Using priority {cd}tors on a target that does not support that can cause failures (see the PR).
Apart from that, I decided to use priority 100 for both gcov_init and gcov_exit functions as
the reserved range includes priority 100. Moreover, I enhanced test-cases we have.

Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.

Ready to be installed?
Martin

[-- Attachment #2: 0001-Don-t-use-priority-cd-tors-if-not-supported-by-a-tar.patch --]
[-- Type: text/x-patch, Size: 6049 bytes --]

From 05a0dcb13d608facdd1c85f4101cd821634d07cd Mon Sep 17 00:00:00 2001
From: marxin <mliska@suse.cz>
Date: Wed, 26 Oct 2016 12:50:35 +0200
Subject: [PATCH] Don't use priority {cd}tors if not supported by a target (PR
 gcov-profile/78086)

gcc/testsuite/ChangeLog:

2016-10-26  Martin Liska  <mliska@suse.cz>

	* g++.dg/gcov/pr16855.C: Clean up the test case.
	* g++.dg/gcov/pr16855-priority.C: New test.

gcc/ChangeLog:

2016-10-26  Martin Liska  <mliska@suse.cz>

	* coverage.c (build_init_ctor): Don't use priority {cd}tors if
	not supported by a target.  Set priority to 100 if possible.
	(build_gcov_exit_decl): Likewise.
---
 gcc/coverage.c                               | 13 +++--
 gcc/testsuite/g++.dg/gcov/pr16855-priority.C | 79 ++++++++++++++++++++++++++++
 gcc/testsuite/g++.dg/gcov/pr16855.C          | 55 +++++++++----------
 3 files changed, 116 insertions(+), 31 deletions(-)
 create mode 100644 gcc/testsuite/g++.dg/gcov/pr16855-priority.C

diff --git a/gcc/coverage.c b/gcc/coverage.c
index 8810710..4167e26 100644
--- a/gcc/coverage.c
+++ b/gcc/coverage.c
@@ -1056,8 +1056,10 @@ build_init_ctor (tree gcov_info_type)
   stmt = build_call_expr (init_fn, 1, stmt);
   append_to_statement_list (stmt, &ctor);
 
-  /* Generate a constructor to run it (with priority 99).  */
-  cgraph_build_static_cdtor ('I', ctor, DEFAULT_INIT_PRIORITY - 1);
+  /* Generate a constructor to run it.  */
+  int priority = SUPPORTS_INIT_PRIORITY
+    ? MAX_RESERVED_INIT_PRIORITY: DEFAULT_INIT_PRIORITY;
+  cgraph_build_static_cdtor ('I', ctor, priority);
 }
 
 /* Generate the destructor function to call __gcov_exit.  */
@@ -1078,8 +1080,11 @@ build_gcov_exit_decl (void)
   tree stmt = build_call_expr (init_fn, 0);
   append_to_statement_list (stmt, &dtor);
 
-  /* Generate a destructor to run it (with priority 99).  */
-  cgraph_build_static_cdtor ('D', dtor, MAX_RESERVED_INIT_PRIORITY - 1);
+  /* Generate a destructor to run it.  */
+  int priority = SUPPORTS_INIT_PRIORITY
+    ? MAX_RESERVED_INIT_PRIORITY: DEFAULT_INIT_PRIORITY;
+
+  cgraph_build_static_cdtor ('D', dtor, priority);
 }
 
 /* Create the gcov_info types and object.  Generate the constructor
diff --git a/gcc/testsuite/g++.dg/gcov/pr16855-priority.C b/gcc/testsuite/g++.dg/gcov/pr16855-priority.C
new file mode 100644
index 0000000..7e39565
--- /dev/null
+++ b/gcc/testsuite/g++.dg/gcov/pr16855-priority.C
@@ -0,0 +1,79 @@
+/* { dg-options "-fprofile-arcs -ftest-coverage" } */
+/* { dg-do run { target native } } */
+/* { dg-require-effective-target init_priority } */
+
+#include <stdlib.h>
+#include <stdio.h>
+
+int a;
+
+void
+foo ()
+{
+  fprintf (stderr, "In foo\n");
+  a = 123; /* count(1) */
+}
+
+using namespace std;
+class Test
+{
+public:
+  Test (void) { fprintf (stderr, "In Test::Test\n"); /* count(1) */ }
+  ~Test (void) { fprintf (stderr, "In Test::~Test\n"); /* count(1) */ }
+} T1;
+
+void
+uncalled (void)
+{
+  fprintf (stderr, "In uncalled\n"); /* count(#####) */
+}
+
+int
+main (void)
+{
+  atexit (&foo);
+  fprintf (stderr, "In main\n"); /* count(1) */
+  return 0;
+}
+
+static void __attribute__ ((constructor)) ctor_default ()
+{
+  fprintf (stderr, "in constructor(())\n"); /* count(1) */
+}
+
+static void __attribute__ ((constructor ((101)))) ctor_100 ()
+{
+  fprintf (stderr, "in constructor((101))\n"); /* count(1) */
+}
+
+static void __attribute__ ((constructor ((500)))) ctor_500 ()
+{
+  fprintf (stderr, "in constructor((500))\n"); /* count(1) */
+}
+
+static void __attribute__ ((constructor ((65535)))) ctor_65535 ()
+{
+  fprintf (stderr, "in constructor((65535))\n"); /* count(1) */
+}
+
+static void __attribute__ ((destructor)) dtor_default ()
+{
+  fprintf (stderr, "in destructor(())\n"); /* count(1) */
+}
+
+static void __attribute__ ((destructor ((101)))) dtor_100 ()
+{
+  fprintf (stderr, "in destructor((101))\n"); /* count(1) */
+}
+
+static void __attribute__ ((destructor ((500)))) dtor_500 ()
+{
+  fprintf (stderr, "in destructor((500))\n"); /* count(1) */
+}
+
+static void __attribute__ ((destructor ((65535)))) dtor_65535 ()
+{
+  fprintf (stderr, "in destructor((65535))\n"); /* count(1) */
+}
+
+/* { dg-final { run-gcov branches { -b pr16855-priority.C } } } */
diff --git a/gcc/testsuite/g++.dg/gcov/pr16855.C b/gcc/testsuite/g++.dg/gcov/pr16855.C
index 91801d4..d7aa8a4 100644
--- a/gcc/testsuite/g++.dg/gcov/pr16855.C
+++ b/gcc/testsuite/g++.dg/gcov/pr16855.C
@@ -2,46 +2,47 @@
 /* { dg-do run { target native } } */
 
 #include <stdlib.h>
+#include <stdio.h>
 
 int a;
 
-void foo()
+void
+foo ()
 {
-  a = 123;						  /* count(1) */
+  fprintf (stderr, "In foo\n");
+  a = 123; /* count(1) */
 }
 
-#include <iostream>
 using namespace std;
-class Test {
+class Test
+{
 public:
-	Test(void){
-	cout<< "In Test ctor" << endl;			  /* count(1) */
-	}
-	~Test(void){
-	cout<< "In Test dtor" << endl;			  /* count(1) */
-	}
-}T1;
-
-void uncalled(void){
-	cout<< "In uncalled" << endl;			  /* count(#####) */
-}
-int main(void){
-atexit (&foo);
-// Test T2;
-cout<< "In main" << endl;				  /* count(1) */
-return 0;
+  Test (void) { fprintf (stderr, "In Test::Test\n"); /* count(1) */ }
+  ~Test (void) { fprintf (stderr, "In Test::~Test\n"); /* count(1) */ }
+} T1;
+
+void
+uncalled (void)
+{
+  fprintf (stderr, "In uncalled\n"); /* count(#####) */
 }
 
-#include <stdio.h>
+int
+main (void)
+{
+  atexit (&foo);
+  fprintf (stderr, "In main\n"); /* count(1) */
+  return 0;
+}
 
-__attribute__((constructor))
-static void construct_navigationBarImages() {
-  fprintf (stderr,  "((construct_navigationBarImages))"); /* count(1) */
+static void __attribute__ ((constructor)) ctor_default ()
+{
+  fprintf (stderr, "in constructor(())\n"); /* count(1) */
 }
 
-__attribute__((destructor))
-static void destroy_navigationBarImages() {
-  fprintf (stderr,  "((destroy_navigationBarImages))");	  /* count(1) */
+static void __attribute__ ((destructor)) dtor_default ()
+{
+  fprintf (stderr, "in destructor(())\n"); /* count(1) */
 }
 
 /* { dg-final { run-gcov branches { -b pr16855.C } } } */
-- 
2.10.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-11-25 10:48 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-10-31  9:16 [PATCH] Don't use priority {cd}tors if not supported by a target (PR, gcov-profile/78086) Martin Liška
2016-10-31 10:07 ` Rainer Orth
2016-10-31 12:13   ` Martin Liška
2016-11-14 12:12     ` Martin Liška
2016-11-23 14:06       ` Martin Liška
2016-11-25 10:48 ` Jan Hubicka

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).