This is the mail archive of the
mailing list for the GCC project.
[PATCH][RFC] Always require a 64bit HWI
- From: Richard Biener <rguenther at suse dot de>
- To: gcc at gcc dot gnu dot org
- Date: Tue, 29 Apr 2014 13:21:53 +0200 (CEST)
- Subject: [PATCH][RFC] Always require a 64bit HWI
- Authentication-results: sourceware.org; auth=none
The following patch forces the availability of a 64bit HWI
(without applying the cleanups that result from this). I propose
this exact patch for a short time to get those that are affected
and do not want to be affected scream.
But honestly I don't see any important host architecture that
not already requires a 64bit HWI.
Another concern is that the host compiler may not provide a
64bit type. I'm not sure that this is an issue nowadays
(even though C++98 doesn't have 'long long', so it's maybe
more an issue now with C++ than it was previously with
requiring C89). But given that it wasn't an issue for
the existing 64bit HWI requiring host archs it shouldn't
be an issue now.
The benefit of this change is obviously the cleanup that
can result from it - especially getting rid of code
generation dependences on the host (!need_64bit_hwi
doesn't mean we force a 32bit hwi). As followup
we can replace HOST_WIDE_INT and its friends with
int64_t variants and appear less confusing to
newcomers (and it's also less characters to type! yay!).
We'd still retain HOST_WIDEST_FAST_INT, and as Kenny
said elsewhere wide-int should internally operate on that,
not on the eventually slow int64_t. But that's a separate
So - any objections?
2014-04-29 Richard Biener <email@example.com>
* configure.ac: Always set need_64bit_hwint to yes.
* configure: Regenerated.
* config.gcc: Always set need_64bit_hwint to yes.
--- libcpp/configure.ac (revision 209890)
+++ libcpp/configure.ac (working copy)
@@ -200,7 +200,7 @@ case $target in
tilegx*-*-* | tilepro*-*-* )
- need_64bit_hwint=no ;;
+ need_64bit_hwint=yes ;;
case $need_64bit_hwint:$ac_cv_sizeof_long in
--- gcc/config.gcc (revision 209890)
+++ gcc/config.gcc (working copy)
@@ -233,7 +233,7 @@ gnu_ld="$gnu_ld_flag"