From: James Greenhalgh <james.greenhalgh@arm.com>
To: Evandro Menezes <e.menezes@samsung.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>,
Wilco Dijkstra <Wilco.Dijkstra@arm.com>,
Andrew Pinski <pinskia@gmail.com>,
"philipp.tomsich@theobroma-systems.com"
<philipp.tomsich@theobroma-systems.com>,
Benedikt Huber <benedikt.huber@theobroma-systems.com>,
<nd@arm.com>
Subject: Re: [PATCH 1/3][AArch64] Add more choices for the reciprocal square root approximation
Date: Mon, 13 Jun 2016 10:01:00 -0000 [thread overview]
Message-ID: <20160613100031.GA30504@arm.com> (raw)
In-Reply-To: <5751FB88.4050906@samsung.com>
On Fri, Jun 03, 2016 at 04:50:00PM -0500, Evandro Menezes wrote:
> From 763562f829d4fec54d21555b2bfd6478d449294f Mon Sep 17 00:00:00 2001
> From: Evandro Menezes <e.menezes@samsung.com>
> Date: Thu, 3 Mar 2016 18:13:46 -0600
> Subject: [PATCH 1/3] [AArch64] Add more choices for the reciprocal square root
> approximation
>
> Allow a target to prefer such operation depending on the operation mode.
This is OK for trunk.
Thanks for the patch, and your work getting it ready for trunk.
Thanks,
James
>
> 2016-03-03 Evandro Menezes <e.menezes@samsung.com>
>
> gcc/
> * config/aarch64/aarch64-protos.h
> (AARCH64_APPROX_MODE): New macro.
> (AARCH64_APPROX_{NONE,ALL}): Likewise.
> (cpu_approx_modes): New structure.
> (tune_params): New member "approx_modes".
> * config/aarch64/aarch64-tuning-flags.def
> (AARCH64_EXTRA_TUNE_APPROX_RSQRT): Remove macro.
> * config/aarch64/aarch64.c
> ({generic,exynosm1,xgene1}_approx_modes): New core
> "cpu_approx_modes" structures.
> (generic_tunings): New member "approx_modes".
> (cortexa35_tunings): Likewise.
> (cortexa53_tunings): Likewise.
> (cortexa57_tunings): Likewise.
> (cortexa72_tunings): Likewise.
> (exynosm1_tunings): Likewise.
> (thunderx_tunings): Likewise.
> (xgene1_tunings): Likewise.
> (use_rsqrt_p): New argument for the mode and use new member from
> "tune_params".
> (aarch64_builtin_reciprocal): Devise mode from builtin.
> (aarch64_optab_supported_p): New argument for the mode.
> * doc/invoke.texi (-mlow-precision-recip-sqrt): Reword description.
prev parent reply other threads:[~2016-06-13 10:01 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-27 21:13 Evandro Menezes
2016-05-25 12:29 ` James Greenhalgh
2016-05-28 13:44 ` Evandro Menezes
2016-06-01 8:36 ` James Greenhalgh
2016-06-03 21:50 ` Evandro Menezes
2016-06-13 10:01 ` James Greenhalgh [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=20160613100031.GA30504@arm.com \
--to=james.greenhalgh@arm.com \
--cc=Wilco.Dijkstra@arm.com \
--cc=benedikt.huber@theobroma-systems.com \
--cc=e.menezes@samsung.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=nd@arm.com \
--cc=philipp.tomsich@theobroma-systems.com \
--cc=pinskia@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).