On 17/07/12 16:17, Ulrich Weigand wrote: > Richard Earnshaw wrote: > >> Hmm, I wonder if we should just unconditionally call split_all_insns() >> at the start of md_reorg when -O0. This would address your problem, but >> have the added benefit that the length calculations would be more >> accurate. We're going to have to split the insns anyway during output, >> so why not get it over and done with... > > Yes, that's already done on various other targets (including s390). > [ Note that you need split_all_insns_noflow at this stage. ] > > I've noticed many other forced-split insn patterns that also carry > a length attribute -- if we do the change you suggest, I guess we > should then also remove those attributes? > > Bye, > Ulrich >
There's certainly no harm in that, but it's not a priority. R.
