From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16427 invoked by alias); 9 Nov 2007 22:47:05 -0000 Received: (qmail 16416 invoked by uid 22791); 9 Nov 2007 22:47:05 -0000 X-Spam-Check-By: sourceware.org Received: from nz-out-0506.google.com (HELO nz-out-0506.google.com) (64.233.162.232) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 09 Nov 2007 22:47:02 +0000 Received: by nz-out-0506.google.com with SMTP id s1so724973nze for ; Fri, 09 Nov 2007 14:47:00 -0800 (PST) Received: by 10.142.48.14 with SMTP id v14mr930396wfv.1194648419554; Fri, 09 Nov 2007 14:46:59 -0800 (PST) Received: by 10.142.84.21 with HTTP; Fri, 9 Nov 2007 14:46:59 -0800 (PST) Message-ID: <571f6b510711091446oe4ee8a1sa7e28fa1ca9da675@mail.gmail.com> Date: Sat, 10 Nov 2007 01:09:00 -0000 From: "Steven Bosscher" To: gcc-patches Subject: Re: [PATCH] gcse.c: don't compute a mapping from CUID to INSN In-Reply-To: <571f6b510711091446r16bff2fbu2f720d16367452e6@mail.gmail.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_2426_14154481.1194648419549" References: <571f6b510711091446r16bff2fbu2f720d16367452e6@mail.gmail.com> 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 X-SW-Source: 2007-11/txt/msg00552.txt.bz2 ------=_Part_2426_14154481.1194648419549 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline Content-length: 854 And with patch this time. On Nov 9, 2007 11:46 PM, Steven Bosscher wrote: > Hi, > > I've long lost the illusion that anyone here cares one serious bit > about compile time. But having loops over all insns just for the sake > of it is surely too much even for the most stubborn pass-adding, > compile time increasing gcc hacker around here? > > Thus. Don't do useless work in gcse.c. This saves 2 passes over all > insns with -fgcse, and even 3 with -Os. I doubt this will have any > measurable impact, but one has to start somewhere. > > Patch is untested beyond "it compiles", but that's really sufficient > in this case anyway :-) > > Gr. > Steven > > * gcse.c (CUID_INSN): Remove. > (cuid_insn): Ditto. > (alloc_gcse_mem): Don't allocate cuid_insn. > (free_gcse_mem): Don't free cuid_insn. > ------=_Part_2426_14154481.1194648419549 Content-Type: text/plain; name=gcse_no_cuid_to_insn_mapping.diff.txt Content-Transfer-Encoding: base64 X-Attachment-Id: f_f8tatb310 Content-Disposition: attachment; filename=gcse_no_cuid_to_insn_mapping.diff.txt Content-length: 1452 SW5kZXg6IGdjc2UuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBnY3Nl LmMJKHJldmlzaW9uIDEzMDA1NSkKKysrIGdjc2UuYwkod29ya2luZyBjb3B5 KQpAQCAtMzgwLDEyICszODAsNiBAQAogLyogTnVtYmVyIG9mIGN1aWRzLiAg Ki8KIHN0YXRpYyBpbnQgbWF4X2N1aWQ7CiAKLS8qIE1hcHBpbmcgb2YgY3Vp ZHMgdG8gaW5zbnMuICAqLwotc3RhdGljIHJ0eCAqY3VpZF9pbnNuOwotCi0v KiBHZXQgaW5zbiBmcm9tIGN1aWQuICAqLwotI2RlZmluZSBDVUlEX0lOU04o Q1VJRCkgKGN1aWRfaW5zbltDVUlEXSkKLQogLyogTWF4aW11bSByZWdpc3Rl ciBudW1iZXIgaW4gZnVuY3Rpb24gcHJpb3IgdG8gZG9pbmcgZ2NzZSArIDEu CiAgICBSZWdpc3RlcnMgY3JlYXRlZCBkdXJpbmcgdGhpcyBwYXNzIGhhdmUg cmVnbm8gPj0gbWF4X2djc2VfcmVnbm8uCiAgICBUaGlzIGlzIG5hbWVkIHdp dGggImdjc2UiIHRvIG5vdCBjb2xsaWRlIHdpdGggZ2xvYmFsIG9mIHNhbWUg bmFtZS4gICovCkBAIC05NDEsMTYgKzkzNSw3IEBACiAJZWxzZQogCSAgdWlk X2N1aWRbSU5TTl9VSUQgKGluc24pXSA9IGk7CiAgICAgICB9Ci0KLSAgLyog Q3JlYXRlIGEgdGFibGUgbWFwcGluZyBjdWlkcyB0byBpbnNucy4gICovCi0K ICAgbWF4X2N1aWQgPSBpOwotICBjdWlkX2luc24gPSBnY2FsbG9jIChtYXhf Y3VpZCArIDEsIHNpemVvZiAocnR4KSk7Ci0gIGkgPSAwOwotICBGT1JfRUFD SF9CQiAoYmIpCi0gICAgRk9SX0JCX0lOU05TIChiYiwgaW5zbikKLSAgICAg IGlmIChJTlNOX1AgKGluc24pKQotCUNVSURfSU5TTiAoaSsrKSA9IGluc247 CiAKICAgLyogQWxsb2NhdGUgdmFycyB0byB0cmFjayBzZXRzIG9mIHJlZ3Mu ICAqLwogICByZWdfc2V0X2JpdG1hcCA9IEJJVE1BUF9BTExPQyAoTlVMTCk7 CkBAIC05NzEsNyArOTU2LDYgQEAKIGZyZWVfZ2NzZV9tZW0gKHZvaWQpCiB7 CiAgIGZyZWUgKHVpZF9jdWlkKTsKLSAgZnJlZSAoY3VpZF9pbnNuKTsKIAog ICBCSVRNQVBfRlJFRSAocmVnX3NldF9iaXRtYXApOwogCg== ------=_Part_2426_14154481.1194648419549--