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