[PATCH 3/4] [ARM] Add attribute/pragma target fpu=

Kyrill Tkachov kyrylo.tkachov@arm.com
Fri Sep 18 08:42:00 GMT 2015


Hi Christian,
(going through the patches...)

On 14/09/15 12:39, Christian Bruel wrote:
> This patch splits the neon_builtins initialization into 2 internals
> functions. One for NEON and one for CRYPTO, each one guarded by its own
> predicate. arm_init_neon_builtins is now global to be called from
> arm_valid_target_attribute_tree if needed.
>
  
diff -ruN gnu_trunk.p2/gcc/gcc/config/arm/arm-protos.h gnu_trunk.p3/gcc/gcc/config/arm/arm-protos.h
--- gnu_trunk.p2/gcc/gcc/config/arm/arm-protos.h	2015-09-11 15:23:51.852687891 +0200
+++ gnu_trunk.p3/gcc/gcc/config/arm/arm-protos.h	2015-09-11 16:30:15.833511559 +0200
@@ -213,7 +213,10 @@
  extern bool arm_change_mode_p (tree);
  #endif
  
-extern tree arm_valid_target_attribute_tree (tree, struct gcc_options *,
+extern void arm_init_neon_builtins (void);
+
+extern tree arm_valid_target_attribute_tree (tree,
+					     struct gcc_options *,
  					     struct gcc_options *);

Why the arm_valid_target_attribute_tree declaration here?
I don't see its relevance in this patch, and it's not mentioned in the ChangeLog.

Kyrill



More information about the Gcc-patches mailing list