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: [PATCH] Fix compile time regression caused by fix to PR64111

From: Jeff Law <law at redhat dot com>
To: Caroline Tice <cmtice at google dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: Richard Biener <rguenther at suse dot de>, Luis Lozano <llozano at google dot com>
Date: Fri, 23 Oct 2015 16:38:27 -0600
Subject: Re: [PATCH] Fix compile time regression caused by fix to PR64111
Authentication-results: sourceware.org; auth=none
References: <CABtf2+RxvHchi=k9z9HPoaUFVY3Mx89rGgh8Sx2HLXMfQVqANQ at mail dot gmail dot com>

On 10/23/2015 04:22 PM, Caroline Tice wrote:

This patch fixes a compile-time regression that was originally
introduced by the fix
for PR64111, in GCC 4.9.3.    One of our user's encountered this problem with a
particular file, where the compile time (on arm) went from 20 seconds
to 150 seconds.

The fix in this patch was suggested by Richard Biener, who wrote the
original fix for
PR64111.  I have verified that this patch fixes the compile time
regression; I have bootstrapped
the compiler with this patch; and I have run the regression testsuite
(no regressions).
Is this ok to commit to ToT?   (I am also working on backports for
gcc-5_branch and gcc-4_9-branch).

-- Caroline Tice
cmtice@google.com

gcc/ChangeLog:

2015-10-23  Caroline Tice  <cmtice@google.com>

         (from Richard Biener)
         * tree.c (int_cst_hasher::hash):  Replace XOR with more efficient
         call to iterative_hash_host_wide_int.

OK.
jeff

References:
- [PATCH] Fix compile time regression caused by fix to PR64111
  - From: Caroline Tice

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