Hello,
On 16 Sep 16:36, Uros Bizjak wrote:
> The patch with a fixed comment is OK otherwise.
Checked into main trunk: http://gcc.gnu.org/ml/gcc-cvs/2013-09/msg00512.html
--
Thanks, K
Here is a final patch with fixed commentary.
2013/9/16 Uros Bizjak :
> On Mon, Sep 16, 2013 at 5:01 PM, Yuri Rumyantsev wrote:
>
>> Does this comment looks good to you:
>>
>> if (start != NULL_RTX)
>> {
>> bb = BLOCK_FOR_INSN (start);
>> if (start != BB_HEAD (bb))
>> /* Initiali
On Mon, Sep 16, 2013 at 4:26 PM, Yuri Rumyantsev wrote:
> Thanks for your review.
>
> I attached modified patch - is it OK for you now?
I can't understand the comment, so perhaps a native English speaking
person could help here.
The patch with a fixed comment is OK otherwise.
Thanks,
Uros.
Uros,
Thanks for your review.
I attached modified patch - is it OK for you now?
2013/9/16 Uros Bizjak :
> On Mon, Sep 16, 2013 at 2:50 PM, Yuri Rumyantsev wrote:
>> Hi All,
>>
>> Here is 1-line fix which improves Atom performance by better distance
>> estimation. We got ~20% speedup on one of b
On Mon, Sep 16, 2013 at 2:50 PM, Yuri Rumyantsev wrote:
> Hi All,
>
> Here is 1-line fix which improves Atom performance by better distance
> estimation. We got ~20% speedup on one of bench from eembc2.0 with
> this fix.
>
> Bootstrapping and regression testing were successful for x86-64.
>
> Is i
Hi All,
Here is 1-line fix which improves Atom performance by better distance
estimation. We got ~20% speedup on one of bench from eembc2.0 with
this fix.
Bootstrapping and regression testing were successful for x86-64.
Is it OK for trunk?
ChangeLog:
2013-09-16 Yuri Rumyantsev
* config/i38
