Hi Christian,

Sorry for the delay.

On 14/09/15 11:47, Christian Bruel wrote:
This patch defines and uses accessors for the current fpu type fields,
based on switchable arm_fpu_index rather than defuncted arm_fpu_desc.

Christian

+  if (TARGET_SOFT_FLOAT)
+    arm_fpu_attr = FPU_NONE;
+  else if (TARGET_FPU_MODEL == ARM_FP_MODEL_VFP)
+    arm_fpu_attr = FPU_VFP;
+  else
+    gcc_unreachable();
Instead of "TARGET_FPU_MODEL == ARM_FP_MODEL_VFP" you can just use the new TARGET_VFP definition, right?


@@ -25679,7 +25667,7 @@
       if (print_tune_info)
        arm_print_tune_info ();
- if (! TARGET_SOFT_FLOAT && arm_fpu_desc->model == ARM_FP_MODEL_VFP)
+      if (! TARGET_SOFT_FLOAT && TARGET_FPU_MODEL == ARM_FP_MODEL_VFP)
        {
          if (TARGET_HARD_FLOAT && TARGET_VFP_SINGLE)
            arm_emit_eabi_attribute ("Tag_ABI_HardFP_use", 27, 1);

Likewise.

This is ok with those changes, but please wait until all patches in the series 
have been approved before
committing.

Thanks,
Kyrill

Reply via email to