This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: S/390: Fix warnings in "*setmem_long..." patterns.
- From: Andreas Krebbel <krebbel at linux dot vnet dot ibm dot com>
- To: gcc-patches at gcc dot gnu dot org, vogt at linux dot vnet dot ibm dot com
- Date: Fri, 11 Dec 2015 12:19:42 +0100
- Subject: Re: S/390: Fix warnings in "*setmem_long..." patterns.
- Authentication-results: sourceware.org; auth=none
- References: <20151202125141 dot 9BF6CF8BB at oc7340732750 dot ibm dot com> <565EEE13 dot 3010405 at linux dot vnet dot ibm dot com> <565EEFBE dot 3090103 at linux dot vnet dot ibm dot com> <20151204171549 dot GA7839 at linux dot vnet dot ibm dot com>
On 12/04/2015 06:15 PM, Dominik Vogt wrote:
> Version 5 with the latest requested changes. Seems to work now.
> I've dropped the extra patch and rather marked the failing tests
> as "xfail".
>
> Ciao
>
> Dominik ^_^ ^_^
>
Patch applied with minor changes:
> + ; Convert Pmode to BLKmode
> + UNSPEC_REPLICATE_BYTE
That comment did not really fit after changing the name of the unspec.
> -(define_expand "setmem_long"
> +(define_expand "setmem_long_<P:mode>"
> [(parallel
> [(clobber (match_dup 1))
> (set (match_operand:BLK 0 "memory_operand" "")
> - (match_operand 2 "shift_count_or_setmem_operand" ""))
> - (use (match_operand 1 "general_operand" ""))
> + (unspec:BLK [(match_operand:P 2 "shift_count_or_setmem_operand" "Y")
Superfluous constraint removed.
Thanks!
-Andreas-