Re: Move misplaced assignment in num_sign_bit_copies1

2016-11-15 Thread Richard Sandiford
Eric Botcazou writes: >> 2016-11-15 Richard Sandiford >> Alan Hayward >> David Sherwood >> >> * rtlanal.c (num_sign_bit_copies1): Calculate bitwidth after >> handling VOIDmode. > > OK, thanks, but please change the comment too, I think "For a smaller mode"

Re: Move misplaced assignment in num_sign_bit_copies1

2016-11-15 Thread Eric Botcazou
> 2016-11-15 Richard Sandiford > Alan Hayward > David Sherwood > > * rtlanal.c (num_sign_bit_copies1): Calculate bitwidth after > handling VOIDmode. OK, thanks, but please change the comment too, I think "For a smaller mode" would be less confusing. --

Move misplaced assignment in num_sign_bit_copies1

2016-11-15 Thread rsandifo
[ This patch is part of the SVE series posted here: https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] The old assignment to bitwidth was before we handled VOIDmode with: if (mode == VOIDmode) mode = GET_MODE (x); so when VOIDmode was specified we would always use: if (bitwidth < GET_