[llvm-commits] Update to latest versions of config.guess and config.sub from GNU mainline

Roman Divacky rdivacky at freebsd.org
Thu Aug 18 00:10:37 PDT 2011


Nathan (the PPC64/FreeBSD guy)

should this change:

Index: autoconf/config.guess
===================================================================
--- autoconf/config.guess       (revision 132626)
+++ autoconf/config.guess       (revision 132627)
@@ -789,13 +789,12 @@
        echo ${UNAME_MACHINE}-unknown-bsdi${UNAME_RELEASE}
        exit ;;
     *:FreeBSD:*:*)
+        UNAME_PROCESSOR=`/usr/bin/uname -p`
        case ${UNAME_MACHINE} in
-           pc98)
-               echo i386-unknown-freebsd`echo ${UNAME_RELEASE}|sed -e 's/[-(].*//'` ;;
            amd64)
                echo x86_64-unknown-freebsd`echo ${UNAME_RELEASE}|sed -e 's/[-(].*//'` ;;
            *)
-               echo ${UNAME_MACHINE}-unknown-freebsd`echo ${UNAME_RELEASE}|sed -e 's/[-(].*//'` ;;
+               echo ${UNAME_PROCESSOR}-unknown-freebsd`echo ${UNAME_RELEASE}|sed -e 's/[-(].*//'` ;;
        esac
        exit ;;
     i*:CYGWIN*:*)

go upstream to the mainline GNU config.guess?

On Wed, Aug 17, 2011 at 09:13:33PM -0700, Ivan Krasin wrote:
> Nick, thanks for the catch.
> 
> Please, find the updated patch attached. Also, I've updated the patch
> at http://codereview.chromium.org/7676009/
> 
> Roman, do you think it makes sense to push into the mainline GNU config.guess?
> 
> Ivan
> 
> 
> On Wed, Aug 17, 2011 at 8:30 PM, Nick Lewycky <nicholas at mxc.ca> wrote:
> > Ivan Krasin wrote:
> >>
> >> Err: missed the patch.
> >> Please, find it attached to this message.
> >>
> >> Also, you can take a look at the patch online:
> >> http://codereview.chromium.org/7676009/
> >
> > This drops r132627 to add support for PPC64 on FreeBSD. Roman, why isn't
> > that upstream? Should Ivan merge that in to this integrate?
> >
> > Nick
> >





More information about the llvm-commits mailing list