public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Matthew Gretton-Dann <matthew.gretton-dann@linaro.org>
To: Richard Earnshaw <rearnsha@arm.com>
Cc: Matthias Klose <doko@ubuntu.com>,
	"gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>,
		"patches@linaro.org" <patches@linaro.org>,
	Richard Biener <rguenther@suse.de>
Subject: Re: [RFA/4.7/ARM] Backport arm-*-linux-gnueabihf triplet support to 4.7
Date: Fri, 21 Dec 2012 10:02:00 -0000	[thread overview]
Message-ID: <CADSXKXremSKKoS_hSC07KH9mqKcknLb5fB-o-rTMEWjiBP9=aw@mail.gmail.com> (raw)
In-Reply-To: <50CF2C0A.3000505@arm.com>

[-- Attachment #1: Type: text/plain, Size: 1625 bytes --]

On 17 December 2012 14:28, Richard Earnshaw <rearnsha@arm.com> wrote:
> On 21/11/12 11:48, Matthew Gretton-Dann wrote:
>>
>> On 21 November 2012 00:05, Matthias Klose <doko@ubuntu.com> wrote:
>>>
>>>
>>> looks fine, except one missing chunk from my original patch. maybe left
>>> out
>>> intentionally.
>>>
>>>    Matthias
>>>
>>> Index: b/src/gcc/config.gcc
>>> ===================================================================
>>> --- a/src/gcc/config.gcc
>>> +++ b/src/gcc/config.gcc
>>> @@ -934,7 +934,7 @@
>>>          tm_file="dbxelf.h elfos.h arm/unknown-elf.h arm/elf.h
>>> arm/linux-gas.h
>>> arm/uclinux-elf.h glibc-stdint.h"
>>>          tmake_file="arm/t-arm arm/t-arm-elf"
>>>          case ${target} in
>>> -       arm*-*-uclinux*eabi)
>>> +       arm*-*-uclinux*eabi*)
>>>              tm_file="$tm_file arm/bpabi.h arm/uclinux-eabi.h"
>>>              tmake_file="$tmake_file arm/t-bpabi"
>>>              # The BPABI long long divmod functions return a 128-bit
>>> value in
>>
>>
>> This change isn't in your commit to trunk of 2012-10-15 which is what
>> I backported.  This is because Richard Earnshaw effectively made this
>> change when he removed FPA support (SVN rev 188510).
>>
>> I'm happy to do a patch that makes this change - but I think it should
>> be a separate patch to this backport one.
>>
>
> I would have thought this ought to be done for consistency.

Committed above as obvious (after discussions off-list with Richard
Earnshaw).  Attached patch shows what was actually committed.

Thanks,

Matt


--
Matthew Gretton-Dann
Linaro Toolchain Working Group
matthew.gretton-dann@linaro.org

[-- Attachment #2: uclinux-eabi.txt --]
[-- Type: text/plain, Size: 969 bytes --]

Index: gcc/config.gcc
===================================================================
--- gcc/config.gcc	(revision 194655)
+++ gcc/config.gcc	(working copy)
@@ -882,7 +882,7 @@
 	tm_file="dbxelf.h elfos.h arm/unknown-elf.h arm/elf.h arm/linux-gas.h arm/uclinux-elf.h glibc-stdint.h"
 	tmake_file="arm/t-arm arm/t-arm-elf"
 	case ${target} in
-	arm*-*-uclinux*eabi)
+	arm*-*-uclinux*eabi*)
 	    tm_file="$tm_file arm/bpabi.h arm/uclinux-eabi.h"
 	    tmake_file="$tmake_file arm/t-bpabi"
   	    # The BPABI long long divmod functions return a 128-bit value in
Index: gcc/ChangeLog
===================================================================
--- gcc/ChangeLog	(revision 194655)
+++ gcc/ChangeLog	(working copy)
@@ -1,3 +1,7 @@
+2012-12-21  Matthew Gretton-Dann  <matthew.gretton-dann@linaro.org>
+
+	* config.gcc: Match arm*-*-uclinux*eabi* for EABI uCLinux.
+
 2012-12-18  Matthew Gretton-Dann  <matthew.gretton-dann@linaro.org>
 
 	Backport from mainline

  reply	other threads:[~2012-12-21 10:02 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-20 20:35 Matthew Gretton-Dann
2012-11-21  0:05 ` Matthias Klose
2012-11-21 11:49   ` Matthew Gretton-Dann
2012-12-17 14:35     ` Richard Earnshaw
2012-12-21 10:02       ` Matthew Gretton-Dann [this message]
2012-11-26 12:13 ` Richard Biener
2012-11-29 14:45 ` Matthew Gretton-Dann
2012-12-07 13:52   ` Matthew Gretton-Dann
2012-12-17 14:28 ` Richard Earnshaw

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='CADSXKXremSKKoS_hSC07KH9mqKcknLb5fB-o-rTMEWjiBP9=aw@mail.gmail.com' \
    --to=matthew.gretton-dann@linaro.org \
    --cc=doko@ubuntu.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=patches@linaro.org \
    --cc=rearnsha@arm.com \
    --cc=rguenther@suse.de \
    /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).