Re: Updating frequencies and dominators

2010-09-15 Thread Paulo J. Matos
Jan Hubicka writes: > > TODO_rebuild_frequencies only propagate probabilities into frequencies, so > when you don't > have probabilities set, it won't help you. > If you just create such a simple conrolled functions, I guess it is best to > just > assign frequencies and probabilities by hand. >

Re: Updating frequencies and dominators

2010-09-14 Thread Jan Hubicka
> So I get in stderr: > , > | g (nD.1176) > | { > | : > | Invalid sum of outgoing probabilities 0.0% > | goto ; > | > | Invalid sum of incoming frequencies 0, should be 4600 > | :; > | f (&"1"[0]); > | goto ; > | > | Invalid sum of incoming frequencies 0, should be 5400 > | :; > | f (

Re: Updating frequencies and dominators

2010-09-14 Thread Paulo J. Matos
Sebastian Pop writes: > On Tue, Sep 14, 2010 at 10:52, Paulo J. Matos wrote: >> mark_irreducible_loops is actually failing with a segmentation fault: > > It looks like you don't work at a level where the loops are built. > So don't call mark_irreducible_loops, just use what Richi suggested: > >

Re: Updating frequencies and dominators

2010-09-14 Thread Paulo J. Matos
Richard Guenther writes: > On Tue, Sep 14, 2010 at 5:36 PM, Sebastian Pop wrote: > > Just free them. All following users are required to eventually > compute them anyway. Thus, > >free_dominance_info (CDI_DOMINATORS); > Thanks, it works! :) -- PMatos

Re: Updating frequencies and dominators

2010-09-14 Thread Sebastian Pop
On Tue, Sep 14, 2010 at 10:52, Paulo J. Matos wrote: > mark_irreducible_loops is actually failing with a segmentation fault: It looks like you don't work at a level where the loops are built. So don't call mark_irreducible_loops, just use what Richi suggested: free_dominance_info (CDI_DOMINATORS

Re: Updating frequencies and dominators

2010-09-14 Thread Paulo J. Matos
Sebastian Pop writes: > On Tue, Sep 14, 2010 at 10:19, Paulo J. Matos wrote: >> How can I automatically update dominators? Or do I have to do it for >> each new basic_block I create with recompute_dominator? > > /* Free and compute again all the dominators information. */ > > static inline void

Re: Updating frequencies and dominators

2010-09-14 Thread Richard Guenther
On Tue, Sep 14, 2010 at 5:36 PM, Sebastian Pop wrote: > On Tue, Sep 14, 2010 at 10:19, Paulo J. Matos wrote: >> How can I automatically update dominators? Or do I have to do it for >> each new basic_block I create with recompute_dominator? > > /* Free and compute again all the dominators informat

Re: Updating frequencies and dominators

2010-09-14 Thread Sebastian Pop
On Tue, Sep 14, 2010 at 10:19, Paulo J. Matos wrote: > How can I automatically update dominators? Or do I have to do it for > each new basic_block I create with recompute_dominator? /* Free and compute again all the dominators information. */ static inline void recompute_all_dominators (void) {

Updating frequencies and dominators

2010-09-14 Thread Paulo J. Matos
Hello, My pass is now generating a correct CFG structure (statements are in the right place and edges between bbs are ok), however in the end due TODO, it fails. Here's the pass definition: , | struct tree_opt_pass pass_clusterswt = | { |"clusterswt", /* name */ |