Hello!

> 2014-06-10  Kugan Vivekanandarajah  <kug...@linaro.org>
>
> * config/arm/arm.c (arm_atomic_assign_expand_fenv): call
> default_atomic_assign_expand_fenv for TARGET_THUMB1.
> (arm_init_builtins) : Initialize builtins __builtins_arm_set_fpscr and
> __builtins_arm_get_fpscr only when !TARGET_THUMB1.
> * config/arm/vfp.md (set_fpscr): Make pattern conditional on
> !TARGERT_THUMB1.
> (get_fpscr) : Likewise.

-  if (!TARGET_VFP)
-    return;
+  if (!TARGET_VFP || TARGET_THUMB1)
+    return default_atomic_assign_expand_fenv (hold, clear, update);

You don't need to call default function here. It is empty, the
documentation says:

"The default implementation leaves all three expressions as @code{NULL_TREE}."

Also, the function is declared as void, so returning something looks
strange to me.

Uros.

Reply via email to