From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 106284 invoked by alias); 15 Oct 2015 18:43:00 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 105678 invoked by uid 89); 15 Oct 2015 18:43:00 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.6 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY,T_RP_MATCHES_RCVD autolearn=no version=3.3.2 X-HELO: paperclip.tbsaunde.org Received: from tbsaunde.org (HELO paperclip.tbsaunde.org) (66.228.47.254) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 15 Oct 2015 18:42:59 +0000 Received: from tsaunders-iceball.corp.tor1.mozilla.com (unknown [66.207.208.102]) by paperclip.tbsaunde.org (Postfix) with ESMTPSA id D12B5C06B; Thu, 15 Oct 2015 18:42:56 +0000 (UTC) Date: Thu, 15 Oct 2015 18:43:00 -0000 From: Trevor Saunders To: Nikolai Bozhenov Cc: Andrew MacLeod , Jeff Law , GCC Patches , maltsevm@gmail.com Subject: Re: Fix prototype for print_insn in rtl.h Message-ID: <20151015184249.GA18233@tsaunders-iceball.corp.tor1.mozilla.com> References: <561CBEF8.3040300@samsung.com> <561D241E.6040001@redhat.com> <561FD443.6040309@redhat.com> <561FD8D2.1080207@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <561FD8D2.1080207@samsung.com> User-Agent: Mutt/1.5.24 (2015-08-30) X-SW-Source: 2015-10/txt/msg01506.txt.bz2 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 > >>> > >>> * 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