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 (new standard pattern setmem)


On Thu, Jun 23, 2005 at 06:36:51PM +0200, Adrian Straetling wrote:
> Actually, I already have a patch on my disk that does so by
> simply removing "clear_storage_via_clrmem" and the 'clrmem' pattern
> itself. On the back-end side it should IMHO suffice to rename the
> relevant expander patterns, add a new operand x which holds the byte
> value and add:
>       if (operands[x] != const0_rtx) FAIL;
> as their first line. They should then behave as 'clrmem' pattern do now.
> I tested that on s390x and it behaved as desired.

I would prefer this approach.  It does sound very much like your
patch does the right thing; I'd like to see it before I approve
it though.


r~


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