------- Additional Comments From dj at redhat dot com  2005-09-19 19:57 -------
Subject: Re:  [4.1 regression] ICE: no-op convert from 8 to 4 bytes in 
initializer


> DJ, apparently you caused this one. 

Yup, and I had reservations about that portion of the patch at the
time, too, which have turned out to be justified (the reservations,
not the patch).

I think the best option at this point is to just take the check out
(leaving in the other part of the patch, which is needed for m32c).

The code will still be buggy if the no-op converts to a larger size,
though, because it doesn't take endianness into account when padding.
I'm rather surprised that the test case works for sizeof(long) >
sizeof(int), I'd expect the bytes to be truncated in amusing ways.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=23799

Reply via email to