This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][RFC][wide-int] Fix some build errors on arm in wide-int branch and report ICE
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Mike Stump <mikestump at comcast dot net>
- Cc: Richard Sandiford <rdsandiford at googlemail dot com>, Kyrill Tkachov <kyrylo dot tkachov at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Kenneth Zadeck <zadeck at naturalbridge dot com>
- Date: Wed, 23 Apr 2014 11:47:12 +0200
- Subject: Re: [PATCH][RFC][wide-int] Fix some build errors on arm in wide-int branch and report ICE
- Authentication-results: sourceware.org; auth=none
- References: <534D11EB dot 9020806 at arm dot com> <53568069 dot 7010503 at arm dot com> <87mwfdtm3y dot fsf at sandifor-thinkpad dot stglab dot manchester dot uk dot ibm dot com> <496FABF5-26F4-42B1-B6C6-267CDD2BE6A3 at comcast dot net>
On Tue, Apr 22, 2014 at 6:04 PM, Mike Stump <mikestump@comcast.net> wrote:
> On Apr 22, 2014, at 8:33 AM, Richard Sandiford <rdsandiford@googlemail.com> wrote:
>> Kyrill Tkachov <kyrylo.tkachov@arm.com> writes:
>>> Ping.
>>> http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00769.html
>>> Any ideas? I recall chatter on IRC that we want to merge wide-int into trunk
>>> soon. Bootstrap failure on arm would prevent that...
>>
>> Sorry for the late reply. I hadn't forgotten, but I wanted to wait
>> until I had chance to look into the ICE before replying, which I haven't
>> had chance to do yet.
>
> They are separable issues, so, I checked in the change.
>
>> It's a shame we can't use C++ style casts,
>> but I suppose that's the price to pay for being able to write
>> "unsigned HOST_WIDE_INTâ.
>
> unsigned_HOST_WIDE_INT isnât horrible, but, yeah, my fingers were expecting a typedef or better. I slightly prefer the int (1) style, but I think we should go the direction of the patch.
Well, on my list of things to try for 4.10 is to kill off HOST_WIDE_* and
require a 64bit integer type on the host and force all targets to use
a 64bit 'hwi'. Thus, s/HOST_WIDE_INT/int64_t/ (and the appropriate
related changes).
Richard.