This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On Tue, Aug 31, 2010 at 1:50 PM, Joseph S. Myers <joseph@codesourcery.com> wrote: >> Attached is the final (but untested) patch. The difference is, that > > This patch does not add the %inherit and empty GCC_4.6.0 version to > libgcc-std.ver that should be there. > >> .ver file (and corresponding t-freebsd file) have been moved to >> libgcc. I modified existing solaris .ver file (as suggested by Joseph) >> and sprinkled it with %if[n]defs (IMO, solaris .ver file also needs >> these). > > They are not needed as symbols not present on a given target are quietly > ignored there - although having them - very carefully reviewed as being > exactly right - may avoid some problems arising accidentally in future if > symbols are added to more targets (e.g. if TImode support is added for > 32-bit x86 - which would require care to ensure no TImode symbols are > accidentally added to old versions for any affected target). Thanks for the review! Attached is a patch that adds missing %inherit. gcc/ChangeLog: 2010-08-31 Uros Bizjak <ubizjak@gmail.com> * config.gcc (i[34567]86-*-freebsd*, x86_64-*-freebsd*): Add i386/t-fprules-softfp and soft-fp/t-softfp to tmake_file. * libgcc-std.ver (GCC_4.6.0): Define version. libgcc/ChangeLog: 2010-08-31 Uros Bizjak <ubizjak@gmail.com> * config.host (i[34567]86-*-freebsd*, x86_64-*-freebsd*): Add i386/t-freebsd to tmake_file. * config/i386/t-freebsd: New file. * config/i386/libgcc-bsd.ver: New file. OK for mainline? Uros.
Attachment:
c.diff.txt
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |