[polly] r261620 - Annotation of SIMD loops

[Roman Gareev via llvm-commits]

Hi Tobias,

thank you for the comment! I’ll try to fix this soon.

2016-02-25 21:13 GMT+05:00 Tobias Grosser <tobias at grosser.es>:
> On 02/23/2016 10:00 AM, Roman Gareev via llvm-commits wrote:
>>
>> Author: romangareev
>> Date: Tue Feb 23 03:00:13 2016
>> New Revision: 261620
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=261620&view=rev
>> Log:
>> Annotation of SIMD loops
>>
>> Use 'mark' nodes annotate a SIMD loop during ScheduleTransformation and
>> skip
>> parallelism checks.
>>
>> The buildbot shows the following compile/execution time changes:
>>
>>    Compile time:
>>      Improvements    Δ     Previous  Current  σ
>>      …/gesummv      -6.06% 0.2640    0.2480   0.0055
>>      …/gemver       -4.46% 0.4480    0.4280   0.0044
>>      …/covariance   -4.31% 0.8360    0.8000   0.0065
>>      …/adi          -3.23% 0.9920    0.9600   0.0065
>>      …/doitgen      -2.53% 0.9480    0.9240   0.0090
>>      …/3mm          -2.33% 1.0320    1.0080   0.0087
>>
>>    Execution time:
>>      Regressions     Δ     Previous  Current  σ
>>      …/viterbi       1.70% 5.1840    5.2720   0.0074
>>      …/smallpt       1.06% 12.4920   12.6240  0.0040
>>
>> Reviewed-by: Tobias Grosser <tobias at grosser.es>
>
>
> Hi Roman,
>
> I just looked again at this change and it seems we still call isParallel()
> for these test cases despite us already knowing that
> loops are parallel (as they are marked as SIMD). We can possibly save some
> more compile time by avoiding this call.

-- 
                                    Cheers, Roman Gareev.