[llvm-commits] [polly] r149266 - /polly/trunk/lib/ScheduleOptimizer.cpp

[Sebastian Pop]

On Mon, Jan 30, 2012 at 1:38 PM, Tobias Grosser
<grosser at fim.uni-passau.de> wrote:
> Author: grosser
> Date: Mon Jan 30 13:38:54 2012
> New Revision: 149266
>
> URL: http://llvm.org/viewvc/llvm-project?rev=149266&view=rev
> Log:
> Scheduling: Add option to disable schedule_maximise_band_depth
>
> maximise_band_depth does not seem to have any effect for now, but it may help to
> increase the amount of tileable loops. We expose the flag to be able to analyze
> its effects when looking into individual benchmarks.
>
> Modified:
>    polly/trunk/lib/ScheduleOptimizer.cpp
>
> Modified: polly/trunk/lib/ScheduleOptimizer.cpp
> URL: http://llvm.org/viewvc/llvm-project/polly/trunk/lib/ScheduleOptimizer.cpp?rev=149266&r1=149265&r2=149266&view=diff
> ==============================================================================
> --- polly/trunk/lib/ScheduleOptimizer.cpp (original)
> +++ polly/trunk/lib/ScheduleOptimizer.cpp Mon Jan 30 13:38:54 2012
> @@ -61,6 +61,11 @@
>                cl::desc("The fusion strategy to choose (min/max)"),
>                cl::Hidden, cl::init("max"));
>
> +static cl::opt<std::string>
> +MaxizeBandDepth("polly-opt-maximize-bands",

Please replace Maxize with Maximize here and below.
Otherwise the patch looks good to me.

Thanks,
Sebastian
--
Qualcomm Innovation Center, Inc is a member of Code Aurora Forum