On 04/27/2011 06:33 PM, Joseph S. Myers wrote:
This patch, relative to a tree with
<http://gcc.gnu.org/ml/gcc-patches/2011-04/msg02123.html>  applied,
continues the cleanup of toplevel cases relating to disabling Java or
Java libraries by arranging for Java to be disabled (via
unsupported_languages) on targets not supporting libffi, and removing
many cases in the Java-disabling case statement that are no longer
needed, either because they relate to targets not supporting libffi or
because they relate to targets no longer supported in GCC so it
doesn't matter what the Java configuration is for them.

The ideal would be for information about libffi-supported targets to
come from a configure fragment in the libffi/ directory, as per item 4
in<http://gcc.gnu.org/ml/gcc/2011-03/msg00486.html>.  Given that at
present there is some divergence between the libffi repository and the
GCC copy of libffi, however, now may not be the best time to add such
a fragment in GCC; instead, I put a minimal case statement (covering
only targets supported by both libffi and GCC) at toplevel.

Where Java-disabling-related cases did more than just disabling Java
or ${libgcj} (for example, explicitly disabling target-boehm-gc or
target-libffi) I left them in for now even if not strictly needed with
the separate libffi case statement.

This patch is intended to be safe, in that the only way it should
change behavior for targets supported by GCC is to disable Java in
some cases where the libraries would not build because of lack of
libffi support (whether or not the libraries and front end were
previously enabled in such cases).

OK to commit?

2011-04-27  Joseph Myers<jos...@codesourcery.com>

        * configure.ac: Disable Java for targets not supporting libffi.
        (*-*-kaos*, am33_2.0-*-linux*, sh*-*-pe|mips*-*-pe): Remove cases
        in Java-disabling statement.
        (*arm-wince-pe): Change to arm-wince-pe.
        (arc-*-*, arm-*-coff, arm-*-pe*, arm-*-riscix*, avr-*-*): Remove
        cases in Java-disabling statement.
        (bfin-*-*): Don't disable Java again.
        (c4x-*-* | tic4x-*-*, tic54x-*-*, cr16-*-*, d10v-*-*, d30v-*-*,
        fr30-*-elf*, moxie-*-*, h8300*-*-*, h8500-*-*, hppa1.1-*-osf* |
        hppa1.1-*-bsd*, hppa*-*-*elf*  | hppa*-*-lites* | hppa*-*-openbsd*,
        hppa*-*-pro*, i960-*-*, i[[3456789]]86-*-coff,
        i[[3456789]]86-*-pe, i[[3456789]]86-*-sco3.2v5*,
        i[[3456789]]86-*-sco*, i[[3456789]]86-*-sysv4*,
        i[[3456789]]86-*-beos*, i[[3456789]]86-*-rdos*,
        m68hc11-*-*|m6811-*-*|m68hc12-*-*|m6812-*-*): Remove cases in
        Java-disabling statement.
        (mmix-*-*): Don't disable Java again.
        (mt-*-*, powerpc*-*-winnt* | powerpc*-*-pe*, powerpcle-*-solaris*,
        powerpc-*-beos*, rs6000-*-lynxos*, rs6000-*-*, m68k-apollo-*,
        microblaze*, mips*-sde-elf*, mips*-*-irix5*, mips*-*-bsd*,
        sparclet-*-aout* | sparc86x-*-*, sparclite-*-*, sparc-*-sunos4*,
        tic6x-*-*, v810-*-*, vax-*-*): Remove cases in Java-disabling
        statement.
        * configure: Regenerate.


Ok.

Do you plan to finish the switch to unsupported_languages instead of noconfigdirs soon )for Java)?

Paolo

Reply via email to