On 10/15/2015 09:42 PM, Trevor Saunders wrote:
Sorry, a little late to the party.. but why is print_insn even in rtl.h?
it seems that sched-vis.c is the only thing that uses it...
Andrew
I'm going to use it in the scheduler...
but then wouldn't something like sched-int.h make more sense? On
the other hand it seems like printing insns is generally useful
functionality, so I'm curious why the scheduler needs its own way of
doing it.
Trev
As for me, I believe sched-int.h is inappropriate place for print_insn
prototype because the function has nothing scheduler specific. And I like
Jeff's idea of removing sched-vis.c and moving everything from it into
print-rtl.[hc]. It would be nice if such code motion resulted also in some
interface and implementation unification for regular and slim dumpers.
Thanks,
Nikolai
