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]
Other format: [Raw text]

Re: [patch]: builtin_memset support (replace clrmem by setmem)


Adrian Straetling <straetling@de.ibm.com> writes:

> This updated patch replaces the 'clrmem' pattern by a new 'setmem'
> pattern which handles arbitrary byte values to initialize the storage.
> In a follow-up patch S/390 will make use of that.
>
> [...]
>
> Index: gcc/config/alpha/alpha.md
> [...]
> +  /* If value to set is not zero, use the library routine.  */
> +  if (operands[2] != const0_rtx)
> +    FAIL;
> +

The comment is somewhat confusing, since OTS$ZERO is a library
routine, too.

Also, Tru64 has _OtsFill; I suppose OpenVMS has OTS$FILL then, which
could be used here. If calling conventions are the same, then the byte
needs to be zero-extended in register 18. I have no idea whether it's
worth bothering, though.

-- 
	Falk


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