[Bug tree-optimization/47234] New: ipa-split is executed before profile feedback is read

[Bug tree-optimization/47234] New: ipa-split is executed before profile feedback is read

[hubicka at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47234

           Summary: ipa-split is executed before profile feedback is read
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: hubicka@gcc.gnu.org


This happens since tree-profile SSA reorg. Obviously ipa-split it profile
driven pass and should be done afterwards.  Working on patch.

This also makes it ipossible to profile w/o LTO and use it for LTO build

[Bug tree-optimization/47234] ipa-split is executed before profile feedback is read

[hubicka at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47234

Jan Hubicka <hubicka at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2011.01.09 16:07:49
             Blocks|                            |45375
         AssignedTo|unassigned at gcc dot       |hubicka at gcc dot gnu.org
                   |gnu.org                     |
     Ever Confirmed|0                           |1

[Bug tree-optimization/47234] ipa-split is executed before profile feedback is read

[hubicka at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47234

--- Comment #1 from Jan Hubicka <hubicka at gcc dot gnu.org> 2011-01-09 23:05:05 UTC ---
Patch posted http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00509.html

[Bug tree-optimization/47234] ipa-split is executed before profile feedback is read

[hubicka at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47234

--- Comment #2 from Jan Hubicka <hubicka at gcc dot gnu.org> 2011-01-10 14:33:08 UTC ---
Author: hubicka
Date: Mon Jan 10 14:33:04 2011
New Revision: 168632

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=168632
Log:

    PR tree-optimization/47234 
    * tree-pass.h (TODO_rebuild_cgraph_edges): New TODO.
    (pass_feedback_split_functions): Declare.
    * passes.c (init_optimization_passes): Add ipa-split as subpass of
    tree-profile.
    * ipa-split.c (gate_split_functions): Update comments; disable
    split-functions for profile_arc_flag and branch_probabilities.
    (gate_feedback_split_functions): New function.
    (execute_feedback_split_functions): New function.
    (pass_feedback_split_functions): New global var.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ipa-split.c
    trunk/gcc/passes.c
    trunk/gcc/tree-pass.h

[Bug tree-optimization/47234] ipa-split is executed before profile feedback is read

[hubicka at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=47234

Jan Hubicka <hubicka at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Jan Hubicka <hubicka at gcc dot gnu.org> 2011-01-10 14:37:07 UTC ---
Fixed.