[Bug bootstrap/53279] New: [4.8 regression] error: 'convert_tree_comp_to_rtx' defined but not used [-Werror=unused-function] breaks m68k-linux bootstrap

[Bug bootstrap/53279] New: [4.8 regression] error: 'convert_tree_comp_to_rtx' defined but not used [-Werror=unused-function] breaks m68k-linux bootstrap

[mikpe at it dot uu.se]

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

             Bug #: 53279
           Summary: [4.8 regression] error: 'convert_tree_comp_to_rtx'
                    defined but not used [-Werror=unused-function] breaks
                    m68k-linux bootstrap
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: mikpe@it.uu.se


Bootstrapping gcc-4.8-20120506 on m68k-linux fails with:

/mnt/scratch/objdir48/./prev-gcc/xgcc -B/mnt/scratch/objdir48/./prev-gcc/
-B/mnt/scratch/install48/m68k-unknown-linux-gnu/bin/
-B/mnt/scratch/install48/m68k-unknown-linux-gnu/bin/
-B/mnt/scratch/install48/m68k-unknown-linux-gnu/lib/ -isystem
/mnt/scratch/install48/m68k-unknown-linux-gnu/include -isystem
/mnt/scratch/install48/m68k-unknown-linux-gnu/sys-include    -c   -g -O2
-gtoggle -DIN_GCC   -W -Wall -Wno-narrowing -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   -DHAVE_CONFIG_H -I. -I.
-I/mnt/scratch/gcc-4.8-20120506/gcc -I/mnt/scratch/gcc-4.8-20120506/gcc/.
-I/mnt/scratch/gcc-4.8-20120506/gcc/../include
-I/mnt/scratch/gcc-4.8-20120506/gcc/../libcpp/include 
-I/mnt/scratch/gcc-4.8-20120506/gcc/../libdecnumber
-I/mnt/scratch/gcc-4.8-20120506/gcc/../libdecnumber/dpd -I../libdecnumber   
/mnt/scratch/gcc-4.8-20120506/gcc/expr.c -o expr.o
/mnt/scratch/gcc-4.8-20120506/gcc/expr.c:2358:1: error:
'get_def_for_expr_class' defined but not used [-Werror=unused-function]
 get_def_for_expr_class (tree name, enum tree_code_class tclass)
 ^
/mnt/scratch/gcc-4.8-20120506/gcc/expr.c:7380:1: error:
'convert_tree_comp_to_rtx' defined but not used [-Werror=unused-function]
 convert_tree_comp_to_rtx (enum tree_code tcode, int unsignedp)
 ^
cc1: all warnings being treated as errors
make[3]: *** [expr.o] Error 1
make[3]: Leaving directory `/mnt/scratch/objdir48/gcc'
make[2]: *** [all-stage2-gcc] Error 2
make[2]: Leaving directory `/mnt/scratch/objdir48'
make[1]: *** [stage2-bubble] Error 2
make[1]: Leaving directory `/mnt/scratch/objdir48'
make: *** [bootstrap] Error 2

gcc-4.8-20120429 did bootstrap Ok.

Configuration parameters:
/mnt/scratch/gcc-4.8-20120506/configure --prefix=/mnt/scratch/install48
--enable-bootstrap --enable-shared --enable-threads=posix
--enable-checking=release --with-system-zlib --enable-__cxa_atexit
--disable-libunwind-exceptions --enable-languages=c,c++ --disable-dssi
--with-java-home=/usr/lib/jvm/java-1.5.0-gcj-1.5.0.0/jre
--enable-libgcj-multifile --disable-java-maintainer-mode
--with-ecj-jar=/usr/share/java/eclipse-ecj.jar --disable-libjava-multilib
--disable-sjlj-exceptions --disable-libmudflap --disable-plugin --disable-lto
--disable-multilib --disable-build-poststage1-with-cxx

[Bug bootstrap/53279] [4.8 regression] error: 'convert_tree_comp_to_rtx' defined but not used [-Werror=unused-function] breaks m68k-linux bootstrap

[mikpe at it dot uu.se]

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

--- Comment #1 from Mikael Pettersson <mikpe at it dot uu.se> 2012-05-08 09:56:01 UTC ---
I believe the get_def_for_expr_class and convert_tree_comp_to_rtx functions
should be #ifdef HAVE_conditional_move.

[Bug bootstrap/53279] [4.8 regression] error: 'convert_tree_comp_to_rtx' defined but not used [-Werror=unused-function] breaks m68k-linux bootstrap

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu.org
   Target Milestone|---                         |4.8.0

[Bug middle-end/53279] [4.8 regression] error: 'convert_tree_comp_to_rtx' defined but not used [-Werror=unused-function] breaks m68k-linux bootstrap

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |build
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2012-05-08
          Component|bootstrap                   |middle-end
         AssignedTo|unassigned at gcc dot       |pinskia at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-05-08 16:18:26 UTC ---
Really these #ifdef on the outside of a function call should be removed.  I
will go that route instead.

[Bug middle-end/53279] [4.8 regression] error: 'convert_tree_comp_to_rtx' defined but not used [-Werror=unused-function] breaks m68k-linux bootstrap

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

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

--- Comment #3 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-05-22 05:38:36 UTC ---
Fixed by:
2012-05-21  Andreas Schwab  <schwab@linux-m68k.org>

        * expr.c (get_def_for_expr_class): Define only if
        HAVE_conditional_move.
        (convert_tree_comp_to_rtx): Likewise.