On 10/08/12 17:46:03, Easwaran Raman wrote:
> I have attached a revised patch. The updated ChangeLog is given below
> and I have responded to your comments inline:
>
> 2012-10-08 Easwaran Raman
> * optabs.c (emit_cmp_and_jump_insn_1): Add a new parameter to
> specificy the probability of takin
> On Sun, Oct 14, 2012 at 8:09 AM, Jan Hubicka wrote:
> > Hi,
> >
> > Index: optabs.c
> > ===
> > --- optabs.c(revision 191879)
> > +++ optabs.c(working copy)
> > @@ -4249,7 +4249,7 @@ prepare_operand (enum insn_code icode, rt
On Sun, Oct 14, 2012 at 8:09 AM, Jan Hubicka wrote:
> Hi,
>
> Index: optabs.c
> ===
> --- optabs.c(revision 191879)
> +++ optabs.c(working copy)
> @@ -4249,7 +4249,7 @@ prepare_operand (enum insn_code icode, rtx x, int
> w
Hi,
Index: optabs.c
===
--- optabs.c(revision 191879)
+++ optabs.c(working copy)
@@ -4249,7 +4249,7 @@ prepare_operand (enum insn_code icode, rtx x, int
we can do the branch. */
static void
-emit_cmp_and_jump_insn_1 (r
Ping.
On Mon, Oct 8, 2012 at 5:46 PM, Easwaran Raman wrote:
> I have attached a revised patch. The updated ChangeLog is given below
> and I have responded to your comments inline:
>
> 2012-10-08 Easwaran Raman
> * optabs.c (emit_cmp_and_jump_insn_1): Add a new parameter to
> specificy the pr
> >
> >>
> >> > +
> >> > + default_edge->count = default_count;
> >> > + if (count)
> >> > +{
> >> > + edge e;
> >> > + edge_iterator ei;
> >> > + FOR_EACH_EDGE (e, ei, stmt_bb->succs)
> >> > +e->probability = e->count * REG_BR_PROB_BASE / count;
> >> > +}
> >>
> >>
>
>>
>> > +
>> > + default_edge->count = default_count;
>> > + if (count)
>> > +{
>> > + edge e;
>> > + edge_iterator ei;
>> > + FOR_EACH_EDGE (e, ei, stmt_bb->succs)
>> > +e->probability = e->count * REG_BR_PROB_BASE / count;
>> > +}
>>
>> Hmm, this updates origina
Hi Honza,
I am addressing some of the questions you raise here. Will send an
updated patch later.
On Thu, Oct 4, 2012 at 6:19 AM, Jan Hubicka wrote:
> > @@ -560,7 +577,6 @@ compute_outgoing_frequencies (basic_block b)
> >return;
> > }
> > }
> > -
> >if (single_succ_p (b))
> >
> Hi,
> This patch propagates the profile counts during RTL expansion. In
> many cases, there is no way to determine the exact count of an edge
> generated during the expansion. So this patch uses some simple
> heuristics to estimate the edge counts but ensures that the counts of
> the basic block
> thanks for the update!
OK, I will review the patch tomorrow then. It is a good incremental step. I
would certainly like to see the gimple expansion in 4.8 however.
Honza
thanks for the update!
David
On Wed, Oct 3, 2012 at 10:37 AM, Steven Bosscher wrote:
> On Wed, Oct 3, 2012 at 6:12 PM, Xinliang David Li wrote:
>> What is the status of switch expansion GIMPLE rewrite? If it is not
>> planned for 4.8, It will be desirable to include this fix into trunk.
>
> I c
On Wed, Oct 3, 2012 at 6:12 PM, Xinliang David Li wrote:
> What is the status of switch expansion GIMPLE rewrite? If it is not
> planned for 4.8, It will be desirable to include this fix into trunk.
I could work on it for GCC 4.8 (there's not a lot of work left to be
done for it now) but we haven
What is the status of switch expansion GIMPLE rewrite? If it is not
planned for 4.8, It will be desirable to include this fix into trunk.
It also helps set up a good base line to test against regression.
thanks,
David
On Tue, Oct 2, 2012 at 6:09 PM, Easwaran Raman wrote:
> Hi,
> This patch pr
Hi,
This patch propagates the profile counts during RTL expansion. In
many cases, there is no way to determine the exact count of an edge
generated during the expansion. So this patch uses some simple
heuristics to estimate the edge counts but ensures that the counts of
the basic blocks correspond
14 matches
Mail list logo
