Hi, On 10/10/2012 12:53 PM, Jakub Jelinek wrote:
Yeah, clearly a different version of the patch has been posted vs. what has been checked in. The difference is removal of the (define_cpu_unit "bdver1-mult" "bdver1_mult") line (present in the posted patch, not in the checked in patch).Also, the ChangeLog entry is wrong: PR 51109 * gcc/config/i386/bdver1.md (bdver1_int): Automaton has been split to reduce state transitions. - should start with: PR target/51109 * config/i386/bdver1.md and really document the changes it has done, or just not contain (bdver1_int) and document the file changes. And Copyright year hasn't been updated (but this wasn't first commit in 2012, others made the same mistake, no changes in 2011 btw).
Shall we (I) just revert the change for now? Thanks, Paolo.
