[Bug target/46623] New: microblaze --enable-werror-always build fails

[Bug target/46623] New: microblaze --enable-werror-always build fails

[amylaar at gcc dot gnu.org]

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

           Summary: microblaze --enable-werror-always build fails
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Keywords: build
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: amylaar@gcc.gnu.org
            Blocks: 44756
            Target: microblaze-elf


gcc -c   -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE  -W -Wall -Wwrite-strings
-Wcast-qual -Wstrict-prototypes -Wmissing-prototypes -Wmissing-format-attribute
-pedantic -Wno-long-long -Wno-variadic-macros -Wno-overlength-strings -Werror
-Wold-style-definition -Wc++-compat -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    insn-preds.c -o
insn-preds.o
../../../gcc/gcc/config/microblaze/predicates.md: In function
‘arith_operand32’:
../../../gcc/gcc/config/microblaze/predicates.md:31:2: error: comparison
between signed and unsigned integer expressions [-Werror=sign-compare]
../../../gcc/gcc/config/microblaze/predicates.md:31:2: error: comparison
between signed and unsigned integer expressions [-Werror=sign-compare]

[Bug target/46623] microblaze --enable-werror-always build fails

[amylaar at gcc dot gnu.org]

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

--- Comment #1 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2010-11-26 17:38:24 UTC ---
Author: amylaar
Date: Fri Nov 26 17:38:20 2010
New Revision: 167186

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=167186
Log:
    PR target/46623
    * config/microblaze/microblaze.c (microblaze_block_move_straight):
    Use XALLOCAVEC.
    (microblaze_option_override): Don't use C++ style comments.
    (save_restore_insns): Remove unused variable base_offset.
    (microblaze_expand_prologue): Remove unused variable insn.
    (microblaze_secondary_reload): Adjust type to match target.h .
    (microblaze_elf_in_small_data_p): Move declarations to start of block.
    (microblaze_expand_move): Likewise.
    * config/microblaze/microblaze.h (LARGE_INT):
    Avoid signed / unsigned comparisons.
    (ASM_OUTPUT_ALIGNED_COMMON, ASM_OUTPUT_ALIGNED_LOCAL): Likewise.
    (ASM_FORMAT_PRIVATE_NAME): Make format specifier match printed data.
    (ASM_FINISH_DECLARE_OBJECT): Likewise.  Constify name.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/config/microblaze/microblaze.c
    trunk/gcc/config/microblaze/microblaze.h

[Bug target/46623] microblaze --enable-werror-always build fails

[amylaar at gcc dot gnu.org]

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

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

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

--- Comment #2 from Jorn Wolfgang Rennecke <amylaar at gcc dot gnu.org> 2010-11-26 17:42:10 UTC ---
Patch has been applied to trunk.