[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