This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fix prototype for print_insn in rtl.h


On 10/19/2015 09:14 AM, Jeff Law wrote:
On 10/15/2015 10:28 AM, Andrew MacLeod wrote:
On 10/13/2015 11:32 AM, Jeff Law wrote:
On 10/13/2015 02:21 AM, Nikolai Bozhenov wrote:
2015-10-13  Nikolai Bozhenov<n.bozhenov@samsung.com>

     * gcc/rtl.h (print_insn): fix prototype
Installed on the trunk after bootstrap & regression test.

jeff

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...
Then let's move it to sched-int.h, unless there's some good reason not to.
Because there isn't a sched-vis.h file and sched-vis.c would need to include sched-int.h.

That's all rather silly because sched-vis.c has nothing to do with scheduling. It's just an RTL dumper.

I think moving all that stuff into print-rtl.[ch] is probably the better solution.

jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]