[polly] r261620 - Annotation of SIMD loops

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 08:13:36 PST 2016


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.

Best,
Tobias



More information about the llvm-commits mailing list