[polly] r287263 - [DBG] Collect statistics about statically infeasible SCoPs

[Tobias Grosser via llvm-commits]

On Thu, Nov 17, 2016, at 10:44 PM, Johannes Doerfert via llvm-commits
wrote:
> Author: jdoerfert
> Date: Thu Nov 17 15:44:47 2016
> New Revision: 287263
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=287263&view=rev
> Log:
> [DBG] Collect statistics about statically infeasible SCoPs
> 
> Modified:
>     polly/trunk/lib/Analysis/ScopBuilder.cpp
> 
> Modified: polly/trunk/lib/Analysis/ScopBuilder.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/polly/trunk/lib/Analysis/ScopBuilder.cpp?rev=287263&r1=287262&r2=287263&view=diff
> ==============================================================================
> --- polly/trunk/lib/Analysis/ScopBuilder.cpp (original)
> +++ polly/trunk/lib/Analysis/ScopBuilder.cpp Thu Nov 17 15:44:47 2016
> @@ -28,6 +28,8 @@ using namespace polly;
>  
>  STATISTIC(ScopFound, "Number of valid Scops");
>  STATISTIC(RichScopFound, "Number of Scops containing a loop");
> +STATISTIC(INFEASIBLE_SCOPS,
> +          "Number of SCoPs with statically infeasible context.");

Hi Johannes,

thank you for upstreaming these changes!

One minor comment: Why do you use UPPERCASE for variable names? This
seems to be contrary to the local coding style and the style used to
name STATISTIC variables in llvm as well?

Best,
Tobias