On 10/4/22 11:52, Aldy Hernandez wrote:
The assert removed by this patch was there to keep users from passing
masks of incompatible types. The self tests are passing host wide
ints down (set_nonzero_bits (-1)), which seem to be 32 bits, whereas
some embedded targets have integer_type_node's of 16-bits. This is
causing problems in m32c-elf, among others.
I suppose there's no harm in passing a 32-bit mask, because
set_nonzero_bits calls wide_int::from() to convert the mask to the
appropriate type. So we can remove the assert.
Sorry for the pain Jeff.
gcc/ChangeLog:
* value-range.cc (irange::set_nonzero_bits): Remove assert.
Thanks. I'll respin everything that failed this AM and see where we are.
jeff
