Re: [PATCH, ARM] Split all insns before pool placement (Re: [PATCH, ARM] Fix length attributes for sync.md patterns)

2012-07-23 Thread Richard Earnshaw
On 23/07/12 14:57, 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 ac

[PATCH, ARM] Split all insns before pool placement (Re: [PATCH, ARM] Fix length attributes for sync.md patterns)

2012-07-23 Thread Ulrich Weigand
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 i

Re: [PATCH, ARM] Fix length attributes for sync.md patterns

2012-07-17 Thread Ulrich Weigand
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 ca

Re: [PATCH, ARM] Fix length attributes for sync.md patterns

2012-07-17 Thread Richard Earnshaw
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 >> accurat

Re: [PATCH, ARM] Fix length attributes for sync.md patterns

2012-07-17 Thread Ulrich Weigand
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

Re: [PATCH, ARM] Fix length attributes for sync.md patterns

2012-07-17 Thread Richard Earnshaw
On 16/07/12 14:45, Ulrich Weigand wrote: > Hello, > > when testing an out-of-tree patch I ran into a latent bug. > The symptom is error messages along the lines of > /tmp/cc6q0E3x.s:38: Error: co-processor offset out of range > caused by an out-of-range reference to a literal pool constant. > This