Patch to create builtin bzero

Richard Henderson rth@cygnus.com
Thu Mar 23 16:36:00 GMT 2000


On Thu, Mar 23, 2000 at 06:42:26PM -0500, Kaveh R. Ghazi wrote:
> I didn't see any C++ problems... but here's a revised patch anyway
> which builds a new chain of memset parameters.

You didn't play with the right combinations of inline functions.
I just ran into this when the sibcall code mucked with the trees
it received.  ;-)

> Okay to install?

Not quite.

> +      TREE_OPERAND (exp, 1) = newarglist;
> +
> +      result = expand_builtin_memset(exp);
> +      
> +      /* If the above call fails, delete the zero parameter.  */
> +      if (result == 0)
> +	TREE_OPERAND (exp, 1) = arglist;

exp must always look *exactly* like it did before you got it.
Always put arglist back.


r~


More information about the Gcc-patches mailing list