public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Gerald Pfeifer <gerald@pfeifer.com>
To: Richard Biener <richard.guenther@gmail.com>
Cc: Ramana Radhakrishnan <ramana.radhakrishnan@foss.arm.com>,
	    Andreas Tobler <andreast@gcc.gnu.org>,
	gcc-patches@gcc.gnu.org
Subject: i386-unknown-freebsd -> i586-unknown-freebsd (was: [Patch wwwdocs] Add aarch64-none-linux-gnu as a primary platform for GCC-7)
Date: Sat, 04 Mar 2017 16:40:00 -0000	[thread overview]
Message-ID: <alpine.LSU.2.20.1703041620080.2487@anthias.pfeifer.com> (raw)
In-Reply-To: <CAFiYyc04WYGawMfhqXT+RqZer0XatbJ=gXwabfDT9HDHU=HVvQ@mail.gmail.com>

On Tue, 24 May 2016, Richard Biener wrote:
>> As Jeff noted, i386 actually is the "marketing" name used for the
>> platform, GCC has been defaulting to i486 for ages, and I upgraded
>> to i586 last year:
>>
>>     2015-11-15  Gerald Pfeifer  <gerald@pfeifer.com>
>>
>>         * config/i386/freebsd.h (SUBTARGET32_DEFAULT_CPU): Change to i586.
>>         Remove support for FreeBSD 5 and earlier.
> Can we update to a non-marketing name then, like i586-unknown-freebsd please?
> config.gcc accepts i[34567]86-*-freebsd*.  It at least confused me.

Sooo, I finally got to submitting the patch below to the config.guess 
maintainers.

When/if this has been accepted, is it okay to pull the latest config.guess
into GCC even at this stage of the release process?  (We're only looking
at this change and the addition of nsx-tandem compared to what we have
right now.)

Gerald


2017-03-04  Gerald Pfeifer  <gerald@pfeifer.com>

	* config.guess (*:FreeBSD:*:*): Refactor CPU handling.
	Canonicalize i386-*-freebsd* to i586-*-freebsd*.

diff --git a/config.guess b/config.guess
index 1000e2b..180375c 100755
--- a/config.guess
+++ b/config.guess
@@ -837,10 +837,11 @@ EOF
 	UNAME_PROCESSOR=`/usr/bin/uname -p`
 	case ${UNAME_PROCESSOR} in
 	    amd64)
-		echo x86_64-unknown-freebsd`echo ${UNAME_RELEASE}|sed -e 's/[-(].*//'` ;;
-	    *)
-		echo ${UNAME_PROCESSOR}-unknown-freebsd`echo ${UNAME_RELEASE}|sed -e 's/[-(].*//'` ;;
+	        UNAME_PROCESSOR=x86_64 ;;
+	    i386)
+		UNAME_PROCESSOR=i586 ;;
 	esac
+	echo ${UNAME_PROCESSOR}-unknown-freebsd`echo ${UNAME_RELEASE}|sed -e 's/[-(].*//'`
 	exit ;;
     i*:CYGWIN*:*)
 	echo ${UNAME_MACHINE}-pc-cygwin

  reply	other threads:[~2017-03-04 16:40 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-23  9:17 [Patch wwwdocs] Add aarch64-none-linux-gnu as a primary platform for GCC-7 Ramana Radhakrishnan
2016-05-23  9:26 ` Richard Biener
2016-05-23 15:47   ` Jeff Law
2016-05-23 22:21   ` Gerald Pfeifer
2016-05-24  9:01     ` Richard Biener
2017-03-04 16:40       ` Gerald Pfeifer [this message]
2017-03-07 16:34         ` i386-unknown-freebsd -> i586-unknown-freebsd Jeff Law
2017-03-07 20:01           ` Updating config.guess (was: i386-unknown-freebsd -> i586-unknown-freebsd) Gerald Pfeifer
2017-03-13 15:33             ` Updating config.guess Jeff Law
2017-03-18 18:23               ` Gerald Pfeifer
2017-03-04 16:45       ` i386-unknown-freebsd -> i586-unknown-freebsd (was: [Patch wwwdocs] Add aarch64-none-linux-gnu as a primary platform for GCC-7) Gerald Pfeifer
2017-03-07 16:33         ` i386-unknown-freebsd -> i586-unknown-freebsd Jeff Law

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.LSU.2.20.1703041620080.2487@anthias.pfeifer.com \
    --to=gerald@pfeifer.com \
    --cc=andreast@gcc.gnu.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=ramana.radhakrishnan@foss.arm.com \
    --cc=richard.guenther@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).