public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Marcus Shawcroft <marcus.shawcroft@arm.com>
To: James Greenhalgh <James.Greenhalgh@arm.com>
Cc: "gcc-patches@gcc.gnu.org" <gcc-patches@gcc.gnu.org>
Subject: Re: [Patch AArch64-4.7] Backport - Refactor Advanced SIMD builtin initialisation.
Date: Thu, 29 Nov 2012 15:11:00 -0000	[thread overview]
Message-ID: <50B77AF9.3080702@arm.com> (raw)
In-Reply-To: <1354100826-14884-1-git-send-email-james.greenhalgh@arm.com>

On 28/11/12 11:07, James Greenhalgh wrote:
>
> Hi,
>
> The backport for this patch was not entirely clean,
> __builtin_thread_pointer was made a front end builtin on the
> 4.8 branch, but remains a back-end builtin on aarch64-4.7.
>
> Otherwise, there are no differences between this patch and the
> patch which was committed to mainline last week.
>
> OK to commit to AArch64-4.7?
>
> Thanks,
> James
>
> ---
> gcc/ChangeLog.aarch64
>
> 2012-11-28  James Greenhalgh  <james.greenhalgh@arm.com>
>
> 	Backport from mainline.
> 	2012-11-20  James Greenhalgh  <james.greenhalgh@arm.com>
> 		    Tejas Belagod  <tejas.belagod@arm.com>
>
> 	* config/aarch64/aarch64-builtins.c
> 	(aarch64_simd_builtin_type_bits): Rename to...
> 	(aarch64_simd_builtin_type_mode): ...this, make sequential.
> 	(aarch64_simd_builtin_datum): Refactor members.
> 	(VAR1, VAR2, ..., VAR12): Update accordingly.
> 	(aarch64_simd_builtin_data): Include from aarch64-simd-builtins.def.
> 	(aarch64_builtins): Update accordingly.
> 	(init_aarch64_simd_builtins): Refactor, rename to...
> 	(aarch64_init_simd_builtins): ...this.
> 	(aarch64_simd_builtin_compare): Remove.
> 	(locate_simd_builtin_icode): Likewise.
> 	* config/aarch64/aarch64-protos.h (aarch64_init_builtins): New.
> 	(aarch64_expand_builtin): Likewise.
> 	(aarch64_load_tp): Likewise.
> 	* config/aarch64/aarch64-simd-builtins.def: New file.
> 	* config/aarch64/aarch64.c (aarch64_init_builtins):
> 	Move to aarch64-builtins.c.
> 	(aarch64_expand_builtin): Likewise.
> 	(aarch64_load_tp): Remove static designation.
> 	* config/aarch64/aarch64.h
> 	(aarch64_builtins): Move to aarch64-builtins.c.
>


OK

/Marcus

      reply	other threads:[~2012-11-29 15:11 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-05 15:53 [PATCH] [AArch64] " James Greenhalgh
2012-10-15 11:35 ` Marcus Shawcroft
2012-10-26 10:17   ` James Greenhalgh
2012-11-12 12:00   ` James Greenhalgh
2012-11-19  9:51     ` *ping* " James Greenhalgh
2012-11-19 18:36     ` Marcus Shawcroft
2012-11-28 11:07       ` [Patch AArch64-4.7] Backport - " James Greenhalgh
2012-11-29 15:11         ` Marcus Shawcroft [this message]

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=50B77AF9.3080702@arm.com \
    --to=marcus.shawcroft@arm.com \
    --cc=James.Greenhalgh@arm.com \
    --cc=gcc-patches@gcc.gnu.org \
    /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).