RE: suggest assert wide_int larger than hashval_t

2010-07-19 Thread Jay K
It's "just" a warning, no "real" affects seen. I patched my copy to say   hwi = ((hwi >> (shift - 1)) >> 1); Thanks,  - Jay > From: i...@google.com > To: jay.kr...@cornell.edu > CC: gcc@gcc.gnu.org >

Re: suggest assert wide_int larger than hashval_t

2010-07-19 Thread Ian Lance Taylor
Jay K writes: > I get this in 4.3.5: > > ../../gcc/gcc/varasm.c: In function `const_rtx_hash_1': > ../../gcc/gcc/varasm.c:3387: warning: right shift count >= width of type > > ./include/hashtab.h:typedef unsigned int hashval_t; > >   unsigned HOST_WIDE_INT hwi; >   hashval_t h, *hp; >  ... >    

RE: suggest assert wide_int larger than hashval_t (32bit hwint?)

2010-07-19 Thread Jay K
Hm, it seems on some 32bit systems, where there is no -m64, wideint can be a mere 32bits. In which case the code should probably say: hwi = ((hwi >> (shift - 1)) >> 1); This was targeting OpenBSD/x86. Maybe I should just stick need_64bit_hwint = yes on config.gcc for that and move along? Assu