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  <mli...@suse.cz>
            Honza Hubicka  <hubi...@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 also simple and mechanical and thus trivial to review. Approved, but obviously don't install until the rest of your patch has been approved.

Similar changes for recently added tests or cases where you might improve ICF requiring similar tweaks to existing tests are pre-approved as well.

jeff

Reply via email to