From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28422 invoked by alias); 7 May 2012 12:28:06 -0000 Received: (qmail 28411 invoked by uid 22791); 7 May 2012 12:28:04 -0000 X-SWARE-Spam-Status: No, hits=-4.3 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00,KHOP_THREADED X-Spam-Check-By: sourceware.org Received: from localhost (HELO gcc.gnu.org) (127.0.0.1) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 07 May 2012 12:27:52 +0000 From: "matz at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug other/29442] insn-attrtab has grown too large Date: Mon, 07 May 2012 12:29:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: other X-Bugzilla-Keywords: compile-time-hog, memory-hog, patch X-Bugzilla-Severity: enhancement X-Bugzilla-Who: matz at gcc dot gnu.org X-Bugzilla-Status: RESOLVED X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2012-05/txt/msg00732.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=29442 --- Comment #18 from Michael Matz 2012-05-07 12:27:18 UTC --- (In reply to comment #15) > While looking for ways to speed up genattrtab itself, I found this patch, which > doesn't speed up genattrtab, but would make the insn-*tab.c files smaller: > > http://gcc.gnu.org/ml/gcc-patches/2003-09/msg00316.html > > Micha, this appears to have been dropped. Any reason why you never asked for an > OK to commit this patch? I did on multiple occasions (last was http://gcc.gnu.org/ml/gcc-patches/2010-06/msg01521.html) , never got an okay. Meanwhile Jakub implemented a different mean which doesn't call the get_attr_ functions dynamically more often (which my approach potentially does), but which as result isn't nearly as effective in reducing size and compile and generation time. Some of the hunks of my patch still are useful cleanups/speedups, lemme submit them.