This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: simple calls.c cleanups


On Thu, Mar 30, 2000 at 01:48:30PM +0200, Jan Hubicka wrote:
> While implementing it I found that we don't support nothrow and malloc
> attributes at the fuction types.  Is that some kind of design decision
> or just omision?

I thought we did support nothrow on types.  Oh, though it's 
in the C++ part of the tree.  Oh well, the CALL_EXPR also has
nothrow marked.

Dunno about the malloc attribute.

>         if (tname[0] == 's')
>   	{
> ! 	  if ((tname[1] == 'e'
> ! 	       && (! strcmp (tname, "setjmp")
> ! 		   || ! strcmp (tname, "setjmp_syscall")))
> ! 	      || (tname[1] == 'i'
> ! 		  && ! strcmp (tname, "sigsetjmp"))
> ! 	      || (tname[1] == 'a'
> ! 		  && ! strcmp (tname, "savectx")))
> ! 	    flags |= ECF_RETURNS_TWICE;

I bet this whole function would get much cleaner if we 
used gprof to recognize the functions.  Not a requirement
for this patch, but a suggestion for future cleanup.

> !       if (flags & (ECF_CONST || ECF_MALLOC))

 "|"

Otherwise ok.


r~

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]