Patchwork [AARCH64] : Remove vpadd_f64 from arm_neon.h.

login
register
mail settings
Submitter Tejas Belagod
Date June 14, 2012, 10:15 a.m.
Message ID <4FD9B9AB.9090602@arm.com>
Download mbox | patch
Permalink /patch/164877/
State New
Headers show

Comments

Tejas Belagod - June 14, 2012, 10:15 a.m.
Hi,

This patch removes vpadd_f64 from arm_neon.h because the definition is incorrect 
and it should be vpaddq_f64 which is defined elsewhere in the same header. OK?

Thanks,
Tejas.

Changelog:

2012-06-14  Tejas Belagod  <tejas.belagod@arm.com>

gcc/
         * config/aarch64/arm_neon.h (vpadd_f64): Remove.
Marcus Shawcroft - June 14, 2012, 10:25 a.m.
On 14/06/12 11:15, Tejas Belagod wrote:
>
> Hi,
>
> This patch removes vpadd_f64 from arm_neon.h because the definition is incorrect
> and it should be vpaddq_f64 which is defined elsewhere in the same header. OK?
>
> Thanks,
> Tejas.
>
> Changelog:
>
> 2012-06-14  Tejas Belagod<tejas.belagod@arm.com>
>
> gcc/
>           * config/aarch64/arm_neon.h (vpadd_f64): Remove.

OK

Patch

diff --git a/gcc/config/aarch64/arm_neon.h b/gcc/config/aarch64/arm_neon.h
index 58976cc..c6ffd27 100644
--- a/gcc/config/aarch64/arm_neon.h
+++ b/gcc/config/aarch64/arm_neon.h
@@ -13175,17 +13175,6 @@  vpadd_f32 (float32x2_t a, float32x2_t b)
   return result;
 }
 
-__extension__ static __inline float64x2_t __attribute__ ((__always_inline__))
-vpadd_f64 (float64x2_t a, float64x2_t b)
-{
-  float64x2_t result;
-  __asm__ ("faddp %0.2d,%1.2d,%2.2d"
-           : "=w"(result)
-           : "w"(a), "w"(b)
-           : /* No clobbers */);
-  return result;
-}
-
 __extension__ static __inline int8x8_t __attribute__ ((__always_inline__))
 vpadd_s8 (int8x8_t __a, int8x8_t __b)
 {