[Bug target/54639] New: mn10300_expand_epilogue: signed / unsigned comparison

[Bug target/54639] New: mn10300_expand_epilogue: signed / unsigned comparison

[amylaar at gcc dot gnu.org]

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

             Bug #: 54639
           Summary: mn10300_expand_epilogue: signed / unsigned comparison
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: amylaar@gcc.gnu.org
            Blocks: 44756
            Target: am33_2.0-linux


g++ -c   -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE  -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/../libdecnumber/dpd
-I../libdecnumber    \
        ../../../gcc/gcc/config/mn10300/mn10300.c -o mn10300.o
../../../gcc/gcc/config/mn10300/mn10300.c: In function ‘void
mn10300_expand_epilogue()’:
../../../gcc/gcc/config/mn10300/mn10300.c:783:64: error: comparison between
signed and unsigned integer expressions [-Werror=sign-compare]
 #define SIZE_ADD_SP(S) ((((S) >= (1 << 15)) || ((S) < -(1 << 15))) ? 6 \
                                                                ^
../../../gcc/gcc/config/mn10300/mn10300.c:1080:29: note: in expansion of macro
'SIZE_ADD_SP'
        this_strategy_size = SIZE_ADD_SP (size + 4 * num_regs_to_save
                             ^
cc1plus: all warnings being treated as errors
make[2]: *** [mn10300.o] Error 1

[Bug target/54639] mn10300_expand_epilogue: signed / unsigned comparison

[amylaar at gcc dot gnu.org]

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

--- Comment #1 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2013-02-26 14:39:24 UTC ---
Author: amylaar
Date: Tue Feb 26 14:39:15 2013
New Revision: 196289

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=196289
Log:
        PR target/54639
        * config/mn10300/mn10300.c (mn10300_expand_epilogue): Avoid offset
        type promotion to unsigned.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/mn10300/mn10300.c

[Bug target/54639] mn10300_expand_epilogue: signed / unsigned comparison

[amylaar at gcc dot gnu.org]

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

--- Comment #2 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2013-02-28 10:09:45 UTC ---
am33_2.0-linux / mn10300-elf build OK now.

[Bug target/54639] mn10300_expand_epilogue: signed / unsigned comparison

[amylaar at gcc dot gnu.org]

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

Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> changed:

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

--- Comment #3 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2013-02-28 10:13:50 UTC ---
See above.