[Bug other/52786] New: double-int.c:231:31: error: comparison between signed and unsigned

[Bug other/52786] New: double-int.c:231:31: error: comparison between signed and unsigned

[danglin at gcc dot gnu.org]

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

             Bug #: 52786
           Summary: double-int.c:231:31: error: comparison between signed
                    and unsigned
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: danglin@gcc.gnu.org
              Host: hppa*-*-*
            Target: hppa*-*-*
             Build: hppa*-*-*


/home/dave/gnu/gcc/objdir/./prev-gcc/g++
-B/home/dave/gnu/gcc/objdir/./prev-gcc/
-B/home/dave/opt/gnu/gcc/gcc-4.7.0/hppa-linux-gnu/bin/ -nostdinc++
-B/home/dave/gnu/gcc/objdir/prev-hppa-linux-gnu/libstdc++-v3/src/.libs
-B/home/dave/gnu/gcc/objdir/prev-hppa-linux-gnu/libstdc++-v3/libsupc++/.libs
-I/home/dave/gnu/gcc/objdir/prev-hppa-linux-gnu/libstdc++-v3/include/hppa-linux-gnu
-I/home/dave/gnu/gcc/objdir/prev-hppa-linux-gnu/libstdc++-v3/include
-I/home/dave/gnu/gcc/gcc/libstdc++-v3/libsupc++
-L/home/dave/gnu/gcc/objdir/prev-hppa-linux-gnu/libstdc++-v3/src/.libs
-L/home/dave/gnu/gcc/objdir/prev-hppa-linux-gnu/libstdc++-v3/libsupc++/.libs -c
  -g -O2 -DIN_GCC   -fno-exceptions -fno-rtti -W -Wall -Wno-narrowing
-Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long
-Wno-variadic-macros -Wno-overlength-strings -Werror -fno-common 
-DHAVE_CONFIG_H 
-I. -I. -I../../gcc/gcc -I../../gcc/gcc/. -I../../gcc/gcc/../include
-I../../gcc
/gcc/../libcpp/include  -I../../gcc/gcc/../libdecnumber
-I../../gcc/gcc/../libde
cnumber/dpd -I../libdecnumber    ../../gcc/gcc/double-int.c -o double-int.o
../../gcc/gcc/double-int.c: In function ‘void rshift_double(long unsigned int,
long int, long unsigned int, unsigned int, long unsigned int*, long int*,
bool)’:
../../gcc/gcc/double-int.c:231:31: error: comparison between signed and
unsigned
 integer expressions [-Werror=sign-compare]

[Bug other/52786] double-int.c:231:31: error: comparison between signed and unsigned

[rguenth at gcc dot gnu.org]

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

--- Comment #1 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-03-30 09:21:00 UTC ---
Author: rguenth
Date: Fri Mar 30 09:20:54 2012
New Revision: 186000

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=186000
Log:
2012-03-30  Richard Guenther  <rguenther@suse.de>

    PR middle-end/52786
    * double-int.c (rshift_double): Remove not needed cast.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/double-int.c

[Bug other/52786] double-int.c:231:31: error: comparison between signed and unsigned

[rguenth at gcc dot gnu.org]

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

Richard Guenther <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-03-30 09:21:26 UTC ---
Fixed.

[Bug other/52786] double-int.c:231:31: error: comparison between signed and unsigned

[rguenth at gcc dot gnu.org]

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

--- Comment #3 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-07-06 09:25:50 UTC ---
Author: rguenth
Date: Fri Jul  6 09:25:46 2012
New Revision: 189314

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=189314
Log:
2012-07-06  Richard Guenther  <rguenther@suse.de>

    Backport from mainline
        2012-03-30  Richard Guenther  <rguenther@suse.de>

        PR middle-end/52786
        * double-int.c (rshift_double): Remove not
        needed cast.

    2012-03-28  Richard Guenther  <rguenther@suse.de>

    PR middle-end/50708
    * double-int.h (rshift_double): Remove.
    * double-int.c (lshift_double): Use absu_hwi to make count
    positive.
    (rshift_double): Make static, take unsigned count argument,
    remove handling of negative count argument.
    (double_int_rshift): Dispatch to lshift_double.

Modified:
    branches/gcc-4_7-branch/gcc/ChangeLog
    branches/gcc-4_7-branch/gcc/double-int.c
    branches/gcc-4_7-branch/gcc/double-int.h