backport: re PR middle-end/89590 (ICE in maybe_emit_free_warning)

	Backported from mainline
	2019-03-05  Jakub Jelinek  <jakub@redhat.com>

	PR middle-end/89590
	* builtins.c (maybe_emit_free_warning): Punt if free doesn't have
	exactly one argument.

	* gcc.dg/pr89590.c: New test.

From-SVN: r275122

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 29a1f92b12698acdb43cef66efa38da8f013a349..ab3d0d650771e80dad0f27c3c2959b117981f091 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,6 +1,12 @@
 2019-08-30  Jakub Jelinek  <jakub@redhat.com>
 
        Backported from mainline
+       2019-03-05  Jakub Jelinek  <jakub@redhat.com>
+
+       PR middle-end/89590
+       * builtins.c (maybe_emit_free_warning): Punt if free doesn't have
+       exactly one argument.
+
        2019-02-28  Jakub Jelinek  <jakub@redhat.com>
 
        PR c/89520

diff --git a/gcc/builtins.c b/gcc/builtins.c
index 767093f539f9cc0436423443aad309122c228e17..fcc8d2294593b62ee4978c20fe31b47d2896b80c 100644 (file)
--- a/gcc/builtins.c
+++ b/gcc/builtins.c
@@ -9774,6 +9774,9 @@ maybe_emit_sprintf_chk_warning (tree exp, enum built_in_function fcode)
 static void
 maybe_emit_free_warning (tree exp)
 {
+  if (call_expr_nargs (exp) != 1)
+    return;
+
   tree arg = CALL_EXPR_ARG (exp, 0);
 
   STRIP_NOPS (arg);

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 17d8ee578ddadfe0649ebe0ae5ff6c634038177e..206a617043e4d4092c21eea38c6180e5f00a391e 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,6 +1,11 @@
 2019-08-30  Jakub Jelinek  <jakub@redhat.com>
 
        Backported from mainline
+       2019-03-05  Jakub Jelinek  <jakub@redhat.com>
+
+       PR middle-end/89590
+       * gcc.dg/pr89590.c: New test.
+
        2019-02-28  Jakub Jelinek  <jakub@redhat.com>
 
        PR c/89521

diff --git a/gcc/testsuite/gcc.dg/pr89590.c b/gcc/testsuite/gcc.dg/pr89590.c
new file mode 100644 (file)
index 0000000..441b0f9
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/pr89590.c
@@ -0,0 +1,11 @@
+/* PR middle-end/89590 */
+/* { dg-do compile } */
+/* { dg-options "-O2 -Wall -w" } */
+
+void free (void *);
+
+void
+foo (void)
+{
+  ((void (*)()) free) ();
+}