diff mbox

[AArch64,4.8] Wire up TARGET_SIMD and TARGET_FLOAT properly

Message ID 53467ADC.5030500@arm.com
State New
Headers show

Commit Message

Kyrylo Tkachov April 10, 2014, 11:05 a.m. UTC
Hi all,

This is the 4.8 version of the patch posted at:
http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00315.html

TARGET_CRYPTO was not defined in 4.8 therefore that hunk is removed.
Ok for the 4.8 branch?

Thanks,
Kyrill

2014-04-10  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>

     * config/aarch64/aarch64.h (TARGET_SIMD): Take AARCH64_ISA_SIMD
     into account.
     (TARGET_FLOAT): Take AARCH64_ISA_FP into account.

Comments

Marcus Shawcroft April 24, 2014, 8:18 a.m. UTC | #1
On 10 April 2014 12:05, Kyrill Tkachov <kyrylo.tkachov@arm.com> wrote:
> Hi all,
>
> This is the 4.8 version of the patch posted at:
> http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00315.html
>
> TARGET_CRYPTO was not defined in 4.8 therefore that hunk is removed.
> Ok for the 4.8 branch?
>
> Thanks,
> Kyrill
>
> 2014-04-10  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
>
>     * config/aarch64/aarch64.h (TARGET_SIMD): Take AARCH64_ISA_SIMD
>
>     into account.
>     (TARGET_FLOAT): Take AARCH64_ISA_FP into account.

This looks fine but wait until the 4.9 back port is committed before
applying this one.

/Marcus
diff mbox

Patch

diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h
index c3efd2a..19ac5eb 100644
--- a/gcc/config/aarch64/aarch64.h
+++ b/gcc/config/aarch64/aarch64.h
@@ -73,9 +73,9 @@ 
 #define WORDS_BIG_ENDIAN (BYTES_BIG_ENDIAN)
 
 /* AdvSIMD is supported in the default configuration, unless disabled by
-   -mgeneral-regs-only.  */
-#define TARGET_SIMD !TARGET_GENERAL_REGS_ONLY
-#define TARGET_FLOAT !TARGET_GENERAL_REGS_ONLY
+   -mgeneral-regs-only or the +nosimd extension.  */
+#define TARGET_SIMD (!TARGET_GENERAL_REGS_ONLY && AARCH64_ISA_SIMD)
+#define TARGET_FLOAT (!TARGET_GENERAL_REGS_ONLY && AARCH64_ISA_FP)
 
 #define UNITS_PER_WORD		8