From: Ian Lance Taylor <ian@gcc.gnu.org>
Date: Thu, 20 Aug 2015 17:43:55 +0000 (+0000)
Subject: compiler: Don't crash on invalid print call.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=f4cd6a8d567ba14f7c936e53bf6c65724c6d42b7;p=gcc.git

compiler: Don't crash on invalid print call.


    When the print builtins are called with no arguments, the compiler
    issues a warning and crashes when trying to produce the backend
    representation for the arguments.

    Fixes golang/go#11526.

    Reviewed-on: https://go-review.googlesource.com/13131

From-SVN: r227039
---

diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE
index b2f305b354c..853cb4acc00 100644
--- a/gcc/go/gofrontend/MERGE
+++ b/gcc/go/gofrontend/MERGE
@@ -1,4 +1,4 @@
-ec34cfb0b148ff461df12c8f5270a06e2f438b7c
+cc7303c97b232ea979cab950d95aaf76c4e0f5b5
 
 The first line of this file holds the git revision number of the last
 merge done from the gofrontend repository.
diff --git a/gcc/go/gofrontend/expressions.cc b/gcc/go/gofrontend/expressions.cc
index 9f757a2aa26..c0e5fc53a82 100644
--- a/gcc/go/gofrontend/expressions.cc
+++ b/gcc/go/gofrontend/expressions.cc
@@ -8177,6 +8177,12 @@ Builtin_call_expression::do_get_backend(Translate_context* context)
                                                       location);
 	  }
 
+        // There aren't any arguments to the print builtin.  The compiler
+        // issues a warning for this so we should avoid getting the backend
+        // representation for this call.  Instead, perform a no-op.
+        if (print_stmts == NULL)
+          return context->backend()->boolean_constant_expression(false);
+
         return print_stmts->get_backend(context);
       }