Richard Earnshaw wrote: > 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? > > There's certainly no harm in that, but it's not a priority.
OK, I'll prepare a patch doing just the split_all_insns then ... Bye, Ulrich -- Dr. Ulrich Weigand GNU Toolchain for Linux on System z and Cell BE ulrich.weig...@de.ibm.com