[Patch ARM] Fix dwarf2cfi_asm properly.

Ramana Radhakrishnan ramana.radhakrishnan@arm.com
Mon Oct 5 09:32:00 GMT 2009


Hi,

I just realized that I'd (sent out and) committed the wrong version of
the patch for working around the dwarf2cfi_asm problem, in which I was
checking for TARGET_AAPCS_BASED before arm_abi was initialized !  

Thanks to Kirrill for pointing it out that it was not working correctly.

I've now committed the following fix as obvious r152446 and will
backport this to the 4.4 branch as well.

[I also took the opportunity of fixing a problem with the previous
Changelog entry on trunk.]


cheers
Ramana


2009-10-05  Ramana Radhakrishnan  <ramana.radhakrishnan@arm.com>

        * config/arm/arm.c (arm_override_options): Really initialize 
        flag_dwarf2_cfi_asm to 0.

	 
Index: arm.c
===================================================================
--- arm.c       (revision 152442)
+++ arm.c       (working copy)
@@ -1298,13 +1298,6 @@
   enum processor_type target_arch_cpu = arm_none;
   enum processor_type selected_cpu = arm_none;
 
-  /* Ideally we would want to use CFI directives to generate
-     debug info.  However this also creates the .eh_frame
-     section, so disable them until GAS can handle
-     this properly.  See PR40521. */
-  if (TARGET_AAPCS_BASED)
-    flag_dwarf2_cfi_asm = 0;
-
   /* Set up the flags based on the cpu/architecture selected by the
user.  */
   for (i = ARRAY_SIZE (arm_select); i--;)
     {
@@ -1871,6 +1864,13 @@
         max_insns_skipped = 3;
     }
 
+  /* Ideally we would want to use CFI directives to generate
+     debug info.  However this also creates the .eh_frame
+     section, so disable them until GAS can handle
+     this properly.  See PR40521. */
+  if (TARGET_AAPCS_BASED)
+    flag_dwarf2_cfi_asm = 0;
+
   /* Register global variables with the garbage collector.  */
   arm_add_gc_roots ();
 }




More information about the Gcc-patches mailing list