This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR63148 - two choices
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 4 Sep 2014 14:24:11 +0200
- Subject: Re: [PATCH] Fix PR63148 - two choices
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot LSU dot 2 dot 11 dot 1409041401060 dot 20733 at zhemvz dot fhfr dot qr>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Sep 04, 2014 at 02:11:26PM +0200, Richard Biener wrote:
> Sofar I have only fully tested removing try_move_mult_to_index
> on trunk which has some fallout that I have fixed and some
> fallout that should be addressed as followup. The patch
Doesn't that regress e.g. testcases where users would expect
such expressions to be folded in initializers, __builtin_constant_p and
similar? If only one case in fold_plusminus_mult_expr is problematic,
removing the whole function altogether sounds too risky to me.
Jakub