[polly] [patch] [efficiency] Move getAliasSetForPointer under if (!IgnoreAliasing)

Sebastian Pop spop at codeaurora.org
Sun Jun 2 06:32:08 PDT 2013


Hi Dmitry,

The patch looks good to me.  Please allow Tobi to also comment on your patch
before committing.

Dmitry N. Mikushin wrote:
> -static cl::opt<bool>
> +cl::opt<bool>

Why do you remove static here? Your change does not touch any other files.

>  IgnoreAliasing("polly-ignore-aliasing",
>                 cl::desc("Ignore possible aliasing of the array bases"),
>                 cl::Hidden, cl::init(false), cl::cat(PollyCategory));


Also could you prepare a patch to disable AA with -polly-ignore-aliasing

  AA = &getAnalysisID<AliasAnalysis>(AliasAnalysis::getClassPassID());

Please test and fix any other places using AA with -polly-ignore-aliasing.

Also please add [polly] on the subject line on patches for polly.

Thanks,
Sebastian
-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation



More information about the llvm-commits mailing list