On 11/23/2013 04:36 PM, Andrew Pinski wrote:
On Sat, Nov 23, 2013 at 11:19 AM, Mike Stump <mikest...@comcast.net> wrote:
Richi has asked the we break the wide-int patch so that the individual port and
front end maintainers can review their parts without have to go through the
entire patch. This patch covers the aarch64 port.
+ wide_int w = real_to_integer (&m, &fail, HOST_BITS_PER_WIDE_INT * 2);
Should we have a gcc_assert (!fail); after the real_to_integer? Other
than that this seems obvious.
It actually is not obvious. In general we did not add checking where
there was none before. the fact that this interface allows the
checking the trunk interface did not does not mean that we are obligated
to do that.
Thanks,
Andrew
Ok?
