On Tue, Jul 03, 2018 at 12:15:48PM +0200, Martin Liška wrote: > > toplev.c already has (in init_alignments): > > > > if (align_jumps_max_skip > align_jumps) > > align_jumps_max_skip = align_jumps - 1; > > I'm rewriting this logic in the patch set. Issue is that > checking for value of align_jumps_max_skip is done > in rs6000_option_override_internal, which is place before > align_jumps_max_skip is parsed. > > That said, 'align_jumps_max_skip <= 0' is always true.
It's not clear to me what you want me to do. You should write your patch so that the end result behaves the same as before, on all targets. If that requires changing (or at least checking) all targets, then you have a lot of work to do. If you think the rs6000 backend is doing something wrong, please say what exactly? I don't see it. Still confused, Segher
