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

Nathan Whitehorn nwhitehorn at freebsd.org
Thu Aug 18 07:53:55 PDT 2011


Yes, I believe so. This is also important for our various ARM and MIPS 
architectures (and simplifies the PC98 case in the patch).
-Nathan

On 08/18/11 02:10, Roman Divacky wrote:
> 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