This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR60896
- From: Richard Biener <rguenther at suse dot de>
- To: Cong Hou <congh at google dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 24 Apr 2014 14:21:57 +0200 (CEST)
- Subject: Re: [PATCH] Fix PR60896
- Authentication-results: sourceware.org; auth=none
- References: <CAK=A3=1UT4RAEsPR29W8OR+zP0j2R2iD5M7a87CpeRuAqBqXRg at mail dot gmail dot com>
On Wed, 23 Apr 2014, Cong Hou wrote:
> See http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60896 for bug report.
>
> The cause of PR60896 is that those statements in PATTERN_DEF_SEQ in
> pre-recognized widen-mult pattern are not forwarded to later
> recognized dot-product pattern.
>
> Another issue is that the def types of statements in PATTERN_DEF_SEQ
> are assigned with the def type of the pattern statement. This is
> incorrect for reduction pattern statement, in which case all
> statements in PATTERN_DEF_SEQ will all be vect_reduction_def, and none
> of them will be vectorized later. The def type of statement in
> PATTERN_DEF_SEQ should always be vect_internal_def.
>
> The patch is attached. Bootstrapped and tested on a x86_64 machine.
>
> OK for trunk?
Ok.
Thanks,
Richard.