public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] use build_function_type_list in the s390 backend
@ 2011-04-20 19:49 Nathan Froyd
  2011-04-21  9:23 ` Richard Guenther
  2011-04-21  9:53 ` Ulrich Weigand
  0 siblings, 2 replies; 3+ messages in thread
From: Nathan Froyd @ 2011-04-20 19:49 UTC (permalink / raw)
  To: gcc-patches; +Cc: uweigand

As $SUBJECT suggests.  Tested with cross to s390-linux-gnu.  OK to
commit?

-Nathan

	* config/s390/s390.c (s390_init_builtins): Call
	build_function_type_list instead of build_function_type.

diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
index caee077..adacfa3 100644
--- a/gcc/config/s390/s390.c
+++ b/gcc/config/s390/s390.c
@@ -9172,7 +9172,7 @@ s390_init_builtins (void)
 {
   tree ftype;
 
-  ftype = build_function_type (ptr_type_node, void_list_node);
+  ftype = build_function_type_list (ptr_type_node, NULL_TREE);
   add_builtin_function ("__builtin_thread_pointer", ftype,
 			S390_BUILTIN_THREAD_POINTER, BUILT_IN_MD,
 			NULL, NULL_TREE);

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

* Re: [PATCH] use build_function_type_list in the s390 backend
  2011-04-20 19:49 [PATCH] use build_function_type_list in the s390 backend Nathan Froyd
@ 2011-04-21  9:23 ` Richard Guenther
  2011-04-21  9:53 ` Ulrich Weigand
  1 sibling, 0 replies; 3+ messages in thread
From: Richard Guenther @ 2011-04-21  9:23 UTC (permalink / raw)
  To: Nathan Froyd; +Cc: gcc-patches, uweigand

On Wed, Apr 20, 2011 at 9:43 PM, Nathan Froyd <froydnj@codesourcery.com> wrote:
> As $SUBJECT suggests.  Tested with cross to s390-linux-gnu.  OK to
> commit?

Those kind of patches are pretty obvious, so if $target maintainers do not
comment within 48h consider them approved.

Thanks,
Richard.

> -Nathan
>
>        * config/s390/s390.c (s390_init_builtins): Call
>        build_function_type_list instead of build_function_type.
>
> diff --git a/gcc/config/s390/s390.c b/gcc/config/s390/s390.c
> index caee077..adacfa3 100644
> --- a/gcc/config/s390/s390.c
> +++ b/gcc/config/s390/s390.c
> @@ -9172,7 +9172,7 @@ s390_init_builtins (void)
>  {
>   tree ftype;
>
> -  ftype = build_function_type (ptr_type_node, void_list_node);
> +  ftype = build_function_type_list (ptr_type_node, NULL_TREE);
>   add_builtin_function ("__builtin_thread_pointer", ftype,
>                        S390_BUILTIN_THREAD_POINTER, BUILT_IN_MD,
>                        NULL, NULL_TREE);
>
>

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

* Re: [PATCH] use build_function_type_list in the s390 backend
  2011-04-20 19:49 [PATCH] use build_function_type_list in the s390 backend Nathan Froyd
  2011-04-21  9:23 ` Richard Guenther
@ 2011-04-21  9:53 ` Ulrich Weigand
  1 sibling, 0 replies; 3+ messages in thread
From: Ulrich Weigand @ 2011-04-21  9:53 UTC (permalink / raw)
  To: Nathan Froyd; +Cc: gcc-patches

Nathan Froyd wrote:

> 	* config/s390/s390.c (s390_init_builtins): Call
> 	build_function_type_list instead of build_function_type.

This is OK.

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com

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

end of thread, other threads:[~2011-04-21  9:27 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-04-20 19:49 [PATCH] use build_function_type_list in the s390 backend Nathan Froyd
2011-04-21  9:23 ` Richard Guenther
2011-04-21  9:53 ` Ulrich Weigand

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).