[x86,PATCH] Simple fix for Atom LEA splitting.

Yuri Rumyantsev ysrumyan@gmail.com
Tue Sep 17 08:54:00 GMT 2013


Here is a final patch with fixed commentary.

2013/9/16 Uros Bizjak <ubizjak@gmail.com>:
> On Mon, Sep 16, 2013 at 5:01 PM, Yuri Rumyantsev <ysrumyan@gmail.com> wrote:
>
>> Does this comment looks good to you:
>>
>>   if (start != NULL_RTX)
>>     {
>>       bb = BLOCK_FOR_INSN (start);
>>       if (start != BB_HEAD (bb))
>> /* Initialize prev to insn if insn and start belong to the same bb;
>>   in this case increase_distance can increment distance to 1.  */
>> prev = insn;
>
> I'd say something in the lines of:
>
> If insn and start belong to the same bb, set prev to insn, so the call
> to increase_distance will increase the distance between insns by 1.
>
> Best regards,
> Uros.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fixed_patch
Type: application/octet-stream
Size: 805 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20130917/82a6d2a3/attachment.obj>


More information about the Gcc-patches mailing list