This is the mail archive of the
mailing list for the GCC project.
Re: Patch for builtin fputs (first stdio opt ready for install)
- To: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- Subject: Re: Patch for builtin fputs (first stdio opt ready for install)
- From: Jeffrey A Law <law at cygnus dot com>
- Date: Thu, 07 Sep 2000 16:07:28 -0600
- cc: jason at redhat dot com, gcc-patches at gcc dot gnu dot org
- Reply-To: law at cygnus dot com
In message <200009072106.RAA08081@caip.rutgers.edu>you write:
> > From: Jason Merrill <firstname.lastname@example.org>
> > You could save a pointer to the _DECL for __builtin_fputc in
> > c_common_nodes_and_builtins.
> > Jason
> Here's what I came up with. Bootstrapped on solaris2.7, no testsuite
> regressions. Okay to install?
> 2000-09-06 Kaveh R. Ghazi <email@example.com>
> * builtins.c (built_in_decls): New array.
> (expand_builtin_fputs): New function.
> (expand_builtin): Handle BUILT_IN_FPUTC and BUILT_IN_FPUTS.
> * builtins.def (BUILT_IN_FPUTC, BUILT_IN_FPUTS): New members.
> * c-common.c (c_common_nodes_and_builtins): Handle fputc/fputs.
> * tree.h (built_in_decls): New array.
I might have missed something -- you appear to be registering a builtin
fputc too, but no expander for it. What's the point behind that?
I also think you should kill the debugging #ifdefs.