Re: opt-info message change for vectorizer

[Xinliang David Li <davidxl at google dot com>]

Fixed as requested. I don't like the extra newline either, but I will
leave that to Teresa.

basic3.c:8:foo: note: loop vectorized

basic3.c:8:foo: note: loop versioned for vectorization because of
possible aliasing

basic3.c:8:foo: note: loop peeled for vectorization to enhance alignment

basic3.c:8:foo: note: loop with 7 iterations completely unrolled

basic3.c:5:foo: note: loop with 7 iterations completely unrolled


Is this version ok after testing?

thanks,

David

On Wed, Aug 28, 2013 at 2:45 AM, Richard Biener
<richard.guenther@gmail.com> wrote:
> On Tue, Aug 27, 2013 at 10:30 PM, Xinliang David Li <davidxl@google.com> wrote:
>> If this is the convention, we should probably have another patch to
>> fix all the existing opt-info messages.
>
> Yes please.
>
> Also ...
>
>
>>>>> b.c:16:A::foo: note: Loop is vectorized
>
> "loop vectorized"
>
>>>>>
>>>>> b.c:16:A::foo: note: Loop is versioned to remove aliases for vectorization
>
> "loop versioned for vectorization because of possible aliasing"
>
>>>>> b.c:16:A::foo: note: Loop is peeled to enhance alignment for vectorization
>
> "loop peeled for vectorization to enhance alignment"
>
>>>>> b.c:16:A::foo: note: Completely unroll loop 6 times
>
> maybe "loop with 6 iterations completely unrolled"
>
>>>>>
>>>>> b.c:12:A::foo: note: Completely unroll loop 6 times
>>>>>
>
> I hate the excessive vertical spacing as well.
>
> Richard.
>
>>>>> Ok after testing?
>>>>>
>>>>> thanks,
>>>>>
>>>>> David
>>>

Attachment: vec_opt_info.txt
Description: Text document