[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