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 4/5] Existing tests fix

From: Jeff Law <law at redhat dot com>
To: mliska <mliska at suse dot cz>, gcc-patches at gcc dot gnu dot org
Cc: hubicka at ucw dot cz
Date: Tue, 17 Jun 2014 13:52:26 -0600
Subject: Re: [PATCH 4/5] Existing tests fix
Authentication-results: sourceware.org; auth=none
References: <c5c2463c07186b4ba35b10f3063ecdd8f8d46d63 dot 1402913001 dot git dot mliska at suse dot cz> <6c7ae6475c2a98900d53a52392dc18c12f986347 dot 1402913001 dot git dot mliska at suse dot cz>

On 06/13/14 04:48, mliska wrote:

Hi,
   many tests rely on a precise number of scanned functions in a dump file. If IPA ICF decides to merge some function and(or) read-only variables, counts do not match.

Martin

Changelog:

2014-06-13  Martin Liska  <mliska@suse.cz>
	    Honza Hubicka  <hubicka@ucw.cz>

	* c-c++-common/rotate-1.c: Text
	* c-c++-common/rotate-2.c: New test.
	* c-c++-common/rotate-3.c: Likewise.
	* c-c++-common/rotate-4.c: Likewise.
	* g++.dg/cpp0x/rv-return.C: Likewise.
	* g++.dg/cpp0x/rv1n.C: Likewise.
	* g++.dg/cpp0x/rv1p.C: Likewise.
	* g++.dg/cpp0x/rv2n.C: Likewise.
	* g++.dg/cpp0x/rv3n.C: Likewise.
	* g++.dg/cpp0x/rv4n.C: Likewise.
	* g++.dg/cpp0x/rv5n.C: Likewise.
	* g++.dg/cpp0x/rv6n.C: Likewise.
	* g++.dg/cpp0x/rv7n.C: Likewise.
	* gcc.dg/ipa/ipacost-1.c: Likewise.
	* gcc.dg/ipa/ipacost-2.c: Likewise.
	* gcc.dg/ipa/ipcp-agg-6.c: Likewise.
	* gcc.dg/ipa/remref-2a.c: Likewise.
	* gcc.dg/ipa/remref-2b.c: Likewise.
	* gcc.dg/pr46309-2.c: Likewise.
	* gcc.dg/torture/ipa-pta-1.c: Likewise.
	* gcc.dg/tree-ssa/andor-3.c: Likewise.
	* gcc.dg/tree-ssa/andor-4.c: Likewise.
	* gcc.dg/tree-ssa/andor-5.c: Likewise.
	* gcc.dg/vect/no-vfa-pr29145.c: Likewise.
	* gcc.dg/vect/vect-cond-10.c: Likewise.
	* gcc.dg/vect/vect-cond-9.c: Likewise.
	* gcc.dg/vect/vect-widen-mult-const-s16.c: Likewise.
	* gcc.dg/vect/vect-widen-mult-const-u16.c: Likewise.
	* gcc.dg/vect/vect-widen-mult-half-u8.c: Likewise.
	* gcc.target/i386/bmi-1.c: Likewise.
	* gcc.target/i386/bmi-2.c: Likewise.
	* gcc.target/i386/pr56564-2.c: Likewise.
	* g++.dg/opt/pr30965.C: Likewise.
	* g++.dg/tree-ssa/pr19637.C: Likewise.
	* gcc.dg/guality/csttest.c: Likewise.
	* gcc.dg/ipa/iinline-4.c: Likewise.
	* gcc.dg/ipa/iinline-7.c: Likewise.
	* gcc.dg/ipa/ipa-pta-13.c: Likewise.

I know this is the least interesting part of your changes, but it's alsosimple and mechanical and thus trivial to review. Approved, butobviously don't install until the rest of your patch has been approved.

Similar changes for recently added tests or cases where you mightimprove ICF requiring similar tweaks to existing tests are pre-approvedas well.


jeff

Follow-Ups:
- Re: [PATCH 4/5] Existing tests fix
  - From: Rainer Orth
- Re: [PATCH 4/5] Existing tests fix
  - From: Martin Liška

References:
- [PATCH 1/5] New Identical Code Folding IPA pass
  - From: mliska
- [PATCH 4/5] Existing tests fix
  - From: mliska

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