On 10/23/2017 10:59 AM, Richard Sandiford wrote: > This patch changes the bit size and vector count arguments to the > machmode.h functions from unsigned int to poly_uint64. > > > 2017-10-23 Richard Sandiford <richard.sandif...@linaro.org> > Alan Hayward <alan.hayw...@arm.com> > David Sherwood <david.sherw...@arm.com> > > gcc/ > * machmode.h (mode_for_size, int_mode_for_size, float_mode_for_size) > (smallest_mode_for_size, smallest_int_mode_for_size): Take the mode > size as a poly_uint64. > (mode_for_vector, mode_for_int_vector): Take the number of vector > elements as a poly_uint64. > * stor-layout.c (mode_for_size, smallest_mode_for_size): Take the mode > size as a poly_uint64. > (mode_for_vector, mode_for_int_vector): Take the number of vector > elements as a poly_uint64. OK.
I think that in general a change from a integer to a poly_uint64 should generally be considered OK without the need for review. Ultimately those are highly mechanical changes with little risk for mucking something up badly. Obviously the changes wouldn't go in until we settled the poly_uint64 questions though. Jeff
