Hi,
In the aarch64 backend we would need VARIANT field in AARCH64_CORE to specify
and detect
variants of xgene.
I found this patch of Andrew Pinski
https://gcc.gnu.org/ml/gcc-patches/2015-11/msg02148.html
However it did not find its way to trunk.
Why was that the case?
What is the correct way t
ping
patch is in: https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00417.html
Thank you
> On 07 Sep 2015, at 12:40, Benedikt Huber
> wrote:
>
> This fifth revision of the patch:
> * Moves a function declaration to a header.
> * Adds comments to functions.
>
> Ok for
100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,22 @@
+2015-10-02 Benedikt Huber
+ Philipp Tomsich
+
+ * config/aarch64/aarch64-builtins.c: Builtins for rsqrt and
+ rsqrtf.
+ * config/aarch64/aarch64-protos.h: Declare.
+ * config/aarch64/aarch64-simd.md
This sixth revision of the patch:
* Cleans up style issues.
* Makes test conform to standards.
Ok for check in.
Benedikt Huber (1):
2015-10-02 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 19
gcc/config/aarch64/aarch64
This second revision of the patch:
* incorporated the review
* added vector types
* added commandline options
-mrecip -mlow-precision-recip-sqrt
* separated functional and assembly parse tests
Ok for check in.
Benedikt Huber (1):
2015-07-14 Benedikt Huber
Philipp
-check.c
create mode 100644 gcc/testsuite/gcc.target/aarch64/rsqrt.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 3432adb..f4b7407 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,20 @@
+2015-07-14 Benedikt Huber
+ Philipp Tomsich
+
+ * config/aarch64/aarch64-builtins.c
This third revision of the patch:
* makes -mrecip default value specified per core.
* disables rsqrt when -Os is given.
Ok for check in.
Benedikt Huber (1):
2015-07-29 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 19
gcc
..ac63f70 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,22 @@
+2015-07-29 Benedikt Huber
+ Philipp Tomsich
+
+ * config/aarch64/aarch64-builtins.c: Builtins for rsqrt and
+ rsqrtf.
+ * config/aarch64/aarch64-opts.h: -mrecip has a default value
+ depending
/gcc.target/aarch64/rsqrt-asm-check.c
create mode 100644 gcc/testsuite/gcc.target/aarch64/rsqrt.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 3432adb..3bf3098 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,24 @@
+2015-07-31 Benedikt Huber
+ Philipp Tomsich
+
+ * config
This fourth revision of the patch:
* fixes an error in the docs.
* uses gen_rsqrt* functions.
* uses extra_tuning_flags.
Ok for check in.
Benedikt Huber (1):
2015-07-31 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 21
gcc
This seventh revision of the patch:
* Cleans up style issues.
* Correct bug in condition.
* Improved testing code.
Ok for check in.
Benedikt Huber (1):
2015-10-15 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 20
gcc
/testsuite/gcc.target/aarch64/rsqrt-asm-check_1.c
create mode 100644 gcc/testsuite/gcc.target/aarch64/rsqrt_1.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index a865043..be096c6 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,23 @@
+2015-10-15 Benedikt Huber
+ Philipp Tomsich
This eighth revision of the patch:
* Style improvements.
Ok for check in.
Benedikt Huber (1):
2015-10-15 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 20
gcc/config/aarch64/aarch64-builtins.c | 115
/rsqrt_asm_check_common.h
create mode 100644
gcc/testsuite/gcc.target/aarch64/rsqrt_asm_check_negative_1.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 30860c4..2abe832 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,23 @@
+2015-10-15 Benedikt Huber
+ Philipp Tomsich
rid of the typedef and leave the struct without
it?
Or should I completely drop the struct?
> On 16 Oct 2015, at 14:37, Oleg Endo wrote:
>
> On Thu, 2015-10-15 at 22:03 +, Benedikt Huber wrote:
>>
>> +/* Add builtins for reciprocal square root. */
>> +
>>
This ninth revision of the patch:
* Removes unnecessary typedef.
Ok for check in.
Benedikt Huber (1):
2015-10-15 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 20
gcc/config/aarch64/aarch64-builtins.c | 115
/rsqrt_asm_check_common.h
create mode 100644
gcc/testsuite/gcc.target/aarch64/rsqrt_asm_check_negative_1.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 30860c4..2abe832 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,23 @@
+2015-10-15 Benedikt Huber
+ Philipp Tomsich
This tenth revision of the patch:
* Removes unnecessary enum.
Ok for check in.
Benedikt Huber (1):
2015-10-19 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 20
gcc/config/aarch64/aarch64-builtins.c | 115
/rsqrt_asm_check_common.h
create mode 100644
gcc/testsuite/gcc.target/aarch64/rsqrt_asm_check_negative_1.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index f39753d..596c9c3 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,23 @@
+2015-10-19 Benedikt Huber
+ Philipp Tomsich
ccess? And if so, who would approve that?
> On 20 Oct 2015, at 15:34, Marcus Shawcroft wrote:
>
> On 4 January 1970 at 00:02, Benedikt Huber
> wrote:
>>* config/aarch64/aarch64-builtins.c: Builtins for rsqrt and rsqrtf.
>>* config/aarch64/aarch64-protos.h
> I guess the first question would be, have you done the copyright assignment?
Yes, I have already done that.
signature.asc
Description: Message signed with OpenPGP using GPGMail
behavior
with -ffast-math.
The patch applies to commit:
svn+ssh://gcc.gnu.org/svn/gcc/trunk@224470
Please consider including this patch.
Thank you and best regards,
Benedikt Huber
Benedikt Huber (1):
2015-06-15 Benedikt Huber
gcc/ChangeLog| 9 +++
gcc/config
(+)
create mode 100644 gcc/testsuite/gcc.target/aarch64/rsqrt.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index c9b156f..690ebba 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,12 @@
+2015-06-15 Benedikt Huber
+
+ * config/aarch64/aarch64-builtins.c: Builtins for rsqrt and
;> this or not depending on the data type. BTW, do you have some tests showing
>> the speed up?
>>
>> Thank you,
>>
>> --
>> Evandro Menezes Austin, TX
>>
>>> -Original Message-
>>> From: gcc-patches-ow...@
Andrew,
> This is NOT a win on thunderX at least for single precision because you have
> to do the divide and sqrt in the same time as it takes 5 multiples (estimate
> and step are multiplies in the thunderX pipeline). Doubles is 10 multiplies
> which is just the same as what the patch does (b
ch
>> Sent: Wednesday, June 24, 2015 15:08
>> To: Evandro Menezes
>> Cc: Benedikt Huber; gcc-patches@gcc.gnu.org
>> Subject: Re: [PATCH] [aarch64] Implemented reciprocal square root (rsqrt)
>> estimation in -ffast-math
>>
>> Evandro,
>>
>> Sh
> Hmm, Reducing the iterations to "1 step for float" and "2 steps for double"
>
> I got VE (miscompares) on following benchmarks
> 416.gamess
> 453.povray
> 454.calculix
> 459.GemsFDTD
>
> Benedikt , I have ICE for 444.namd with your patch, not sure if something
> wrong in my local tree.
I co
>
> Benedikt , I have ICE for 444.namd with your patch, not sure if something
> wrong in my local tree.
Venkat, now I could reproduce it.
Strangely it does not happen with -flto.
I will try to find out the reason for that.
Thank you for the catch,
Benedikt
signature.asc
Description: Message
ping
[PATCH v4][aarch64] Implemented reciprocal square root (rsqrt) estimation in
-ffast-math
https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02698.html
> On 31 Jul 2015, at 19:05, Benedikt Huber
> wrote:
>
> * config/aarch64/aarch64-builtins.c: Builtins for rsqrt and
&g
/aarch64/rsqrt-asm-check.c
create mode 100644 gcc/testsuite/gcc.target/aarch64/rsqrt.c
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 77fb2c1..382f6b3 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,24 @@
+2015-09-03 Benedikt Huber
+ Philipp Tomsich
+
+ * config
This fifth revision of the patch:
* Moves a function declaration to a header.
* Adds comments to functions.
Ok for check in.
Benedikt Huber (1):
2015-09-03 Benedikt Huber
Philipp Tomsich
gcc/ChangeLog | 21
gcc/config/aarch64
31 matches
Mail list logo
