From: Ian Lance Taylor Date: Tue, 25 Aug 2015 03:48:35 +0000 (+0000) Subject: compiler: Type check params in sink function decl. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e3552bf9f0a78afb9d13351b05c3adf306d847d0;p=gcc.git compiler: Type check params in sink function decl. When a function is declared and named with the blank identifier, only the syntax is checked. This patch modifies the parser to add a dummy node for each function declaration with a blank identifier name that will be type checked like any function declaration. Fixes golang/go#11535. Reviewed-on: https://go-review.googlesource.com/13792 From-SVN: r227160 --- diff --git a/gcc/go/gofrontend/MERGE b/gcc/go/gofrontend/MERGE index 853cb4acc00..e499d7f3c01 100644 --- a/gcc/go/gofrontend/MERGE +++ b/gcc/go/gofrontend/MERGE @@ -1,4 +1,4 @@ -cc7303c97b232ea979cab950d95aaf76c4e0f5b5 +81810917af7ba19e1f9f8efc8b1989f7d6419d30 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/parse.cc b/gcc/go/gofrontend/parse.cc index 211fd73b3c5..922473ac810 100644 --- a/gcc/go/gofrontend/parse.cc +++ b/gcc/go/gofrontend/parse.cc @@ -2305,8 +2305,20 @@ Parse::function_decl(bool saw_nointerface) if (!this->peek_token()->is_op(OPERATOR_LCURLY)) { - if (named_object == NULL && !Gogo::is_sink_name(name)) + if (named_object == NULL) { + // Function declarations with the blank identifier as a name are + // mostly ignored since they cannot be called. We make an object + // for this declaration for type-checking purposes. + if (Gogo::is_sink_name(name)) + { + static int count; + char buf[30]; + snprintf(buf, sizeof buf, ".$sinkfndecl%d", count); + ++count; + name = std::string(buf); + } + if (fntype == NULL || (expected_receiver && rec == NULL)) this->gogo_->add_erroneous_name(name);