On 4/17/19 9:35 AM, Peter Bergner wrote: > On 4/16/19 12:47 PM, Peter Bergner wrote: >> The patch below fixes the issue not continuing if the allocno's conflict >> array is null and instead guarding the current conflict prints by that >> test. If the conflict array is null, we instead now print out simple >> empty conflict info. This now gives us what we'd expect to see: >> >> ;; a5(r116,l0) conflicts: >> ;; total conflict hard regs: >> ;; conflict hard regs: >> >> >> cp0:a0(r111)<->a4(r117)@330:move > > > Actually, if we keep the continue, it makes the patch smaller and more > readable. How about this instead which gives the same output as the > previous patch? > > Peter > > * ira-conflicts.c (print_allocno_conflicts): Always print something, > even for allocno's with no conflicts. > (print_conflicts): Print an extra newline. OK. And while it's technically not a regression fix, I think this can safely go in now :-)
jeff