This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix prototype for print_insn in rtl.h
- From: Trevor Saunders <tbsaunde at tbsaunde dot org>
- To: Nikolai Bozhenov <n dot bozhenov at samsung dot com>
- Cc: Andrew MacLeod <amacleod at redhat dot com>, Jeff Law <law at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, maltsevm at gmail dot com
- Date: Thu, 15 Oct 2015 14:42:49 -0400
- Subject: Re: Fix prototype for print_insn in rtl.h
- Authentication-results: sourceware.org; auth=none
- References: <561CBEF8 dot 3040300 at samsung dot com> <561D241E dot 6040001 at redhat dot com> <561FD443 dot 6040309 at redhat dot com> <561FD8D2 dot 1080207 at samsung dot com>
On Thu, Oct 15, 2015 at 07:48:18PM +0300, Nikolai Bozhenov wrote:
>
> On 10/15/2015 07:28 PM, 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...
> >
> >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
>
> Thanks,
> Nikolai