> Is GET_MODE_UNIT_SIZE (mode) always equal to TYPE_ALIGN_UNIT (type) at > this point?
I don't think TYPE_ALIGN_UNIT is necessarily equal to GET_MODE_UNIT_SIZE but it's likely(?) still ok due the way it is/was used. is_packed is now always defined (set to !is_size_aligned). So comparing to the size of the access mode's unit is at least consistent. If we had a type with TYPE_ALIGN_UNIT (type) == 1 (and TYPE_SIZE (type) > 1) we might have a problem. The opposite, TYPE_ALIGN_UNIT (type) > TYPE_SIZE_UNIT (type) doesn't matter for the is_packed check but might for the last check. That one, in turn, is very likely redundant as the access isn't packed. In short, if you notice something odd after the fix, please report back :) -- Regards Robin
