From: Marek Polacek <polacek@redhat.com>
Date: Wed, 27 Nov 2013 08:02:48 +0000 (+0000)
Subject: ubsan.c (ubsan_type_descriptor): If varpool_get_node returns NULL for a decl, recreat... 
X-Git-Tag: releases/gcc-4.9.0~2428
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=14a981b922e03f200af99b1db09f599842d8eb7d;p=thirdparty%2Fgcc.git

ubsan.c (ubsan_type_descriptor): If varpool_get_node returns NULL for a decl, recreate that decl.

2013-11-27  Marek Polacek  <polacek@redhat.com>

	* ubsan.c (ubsan_type_descriptor): If varpool_get_node returns NULL
	for a decl, recreate that decl.  Save into the hash table VAR_DECLs
	rather than ADDR_EXPRs.
testsuite/
	* c-c++-common/ubsan/undefined-1.c: New test.

From-SVN: r205433
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 851f746bbfd6..736b2c2b8f91 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2013-11-27  Marek Polacek  <polacek@redhat.com>
+
+	* ubsan.c (ubsan_type_descriptor): If varpool_get_node returns NULL
+	for a decl, recreate that decl.  Save into the hash table VAR_DECLs
+	rather than ADDR_EXPRs.
+
 2013-11-27  Alexander Ivchenko  <alexander.ivchenko@intel.com>
 
 	* config/ia64/hpux.h (TARGET_LIBC_HAS_FUNCTION): Fix typo.
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index a2a81170166e..61fe795127f9 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,7 @@
+2013-11-27  Marek Polacek  <polacek@redhat.com>
+
+	* c-c++-common/ubsan/undefined-1.c: New test.
+
 2013-11-26  Jakub Jelinek  <jakub@redhat.com>
 
 	PR tree-optimization/59014
diff --git a/gcc/testsuite/c-c++-common/ubsan/undefined-1.c b/gcc/testsuite/c-c++-common/ubsan/undefined-1.c
new file mode 100644
index 000000000000..1229b711fe92
--- /dev/null
+++ b/gcc/testsuite/c-c++-common/ubsan/undefined-1.c
@@ -0,0 +1,26 @@
+/* { dg-do run } */
+/* { dg-options "-fsanitize=undefined" } */
+/* { dg-skip-if "" { *-*-* } { "-flto" } { "" } } */
+
+int
+foo (int x, int y)
+{
+  const int z = 2;
+  if (z & 1)
+    return x << y;
+  return 0;
+}
+
+int
+bar (int x, int y)
+{
+  return x + y;
+}
+
+int
+main (void)
+{
+  foo (3, 2);
+  bar (12, 42);
+  return 0;
+}
diff --git a/gcc/ubsan.c b/gcc/ubsan.c
index 468a3b1b1506..fa6b42a868f2 100644
--- a/gcc/ubsan.c
+++ b/gcc/ubsan.c
@@ -272,8 +272,12 @@ ubsan_type_descriptor (tree type, bool want_pointer_type_p)
   type = TYPE_MAIN_VARIANT (type);
 
   tree decl = decl_for_type_lookup (type);
-  if (decl != NULL_TREE)
-    return decl;
+  /* It is possible that some of the earlier created DECLs were found
+     unused, in that case they weren't emitted and varpool_get_node
+     returns NULL node on them.  But now we really need them.  Thus,
+     renew them here.  */
+  if (decl != NULL_TREE && varpool_get_node (decl))
+    return build_fold_addr_expr (decl);
 
   tree dtype = ubsan_type_descriptor_type ();
   tree type2 = type;
@@ -372,11 +376,10 @@ ubsan_type_descriptor (tree type, bool want_pointer_type_p)
   DECL_INITIAL (decl) = ctor;
   rest_of_decl_compilation (decl, 1, 0);
 
-  /* Save the address of the VAR_DECL into the hash table.  */
-  decl = build_fold_addr_expr (decl);
+  /* Save the VAR_DECL into the hash table.  */
   decl_for_type_insert (type, decl);
 
-  return decl;
+  return build_fold_addr_expr (decl);
 }
 
 /* Create a structure for the ubsan library.  NAME is a name of the new