[PATCH] [Polly] Allow to annotate alias scopes in the new SCoP.

Tobias Grosser tobias at grosser.es
Thu Oct 2 03:27:17 PDT 2014


On 02/10/2014 11:33, Johannes Doerfert wrote:
> Dup of http://reviews.llvm.org/D5575
>
> ================
> Comment at: include/polly/CodeGen/CodeGeneration.h:41
> @@ -39,1 +40,3 @@
> +extern bool PollyAnnotateAliasScopes;
> +
>   static inline int getNumberOfIterations(__isl_take isl_set *Domain) {
> ----------------
> grosser wrote:
>> I don't think we need a command line option for this. That seems pure goodness, so why don't we enable it by default?
> This is for example useful for ppl using Polly as an analyzer/annotator with the ignore-aliasing option and without scheduling. In such a case you might don't want the alias scopes.

Are you using Polly in this way? Otherwise, lets only add support for it 
if someone really needs it.

In case we leave this option, we may want to add lines that check that 
we do not emit this information if -polly-annotate-alias-scopes=false is 
set. Also, the test cases have now unnecessary 
-polly-annotate-alias-scopes lines.

Cheers,
Tobias




More information about the llvm-commits mailing list