Thumb2 overestimates size of COND_EXEC insns

Bug #645279 reported by Andrew Stubbs
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro GCC
Triaged
Low
Unassigned

Bug Description

This piece of code

/* Add two bytes to the length of conditionally executed Thumb-2
   instructions for the IT instruction. */
#define ADJUST_INSN_LENGTH(insn, length) \
  if (TARGET_THUMB2 && GET_CODE (PATTERN (insn)) == COND_EXEC) \
    length += 2;

causes us to overestimate the length of COND_EXEC blocks with more than one conditional instruction. It would be better to add a pass to arm_reorg that adds unspec_volatile IT insns which have length 2.

[CodeSourcery Tracker ID #8452]

Michael Hope (michaelh1)
tags: added: speed task
Changed in gcc-linaro:
importance: Undecided → Low
Changed in gcc-linaro:
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.