Paul Brook - Thumb-2 conditional execution fix (original) (raw)

This is the mail archive of the gcc-patches@gcc.gnu.orgmailing list for the GCC project.

• From: Paul Brook
• To: gcc-patches at gcc dot gnu dot org
• Date: Wed, 21 Feb 2007 22:37:01 +0000
• Subject: Thumb-2 conditional execution fix

I'm about to apply the attached patch to sourceryg++ 4.1/4.2.

Ok for the release notes?

  <!-- TARGET: arm*-* -->
  <formalpara>
    <title>Missing Thumb-2 IT instructions</title>
    <para>
  A bug in Thumb-2 code generation has been fixed.  This bug would
  result in missing IT instructions, causing the assembler to
  reject the code.
    </para>
  </formalpara>

Paul

2007-02-21 Paul Brook paul@codesourcery.com

gcc/
* config/arm/arm.c (thumb2_final_prescan_insn): Don't incrememnt
condexec_count when skipping USE and CLOBBER.

Index: gcc/config/arm/arm.c

--- gcc/config/arm/arm.c (revision 163241) +++ gcc/config/arm/arm.c (working copy) @@ -12318,10 +12370,7 @@ thumb2_final_prescan_insn (rtx insn) /* USE and CLOBBER aren't really insns, so just skip them. */ if (GET_CODE (body) == USE || GET_CODE (body) == CLOBBER) - { - arm_condexec_count++; - continue; - } + continue;

   /* ??? Recognise conditional jumps, and combine them with IT blocks.  */
   if (GET_CODE (body) != COND_EXEC)

• Follow-Ups:
  • Re: Thumb-2 conditional execution fix
    * From: Paul Brook