[PATCH] D22036: [polly] Add accelerator code generation pass sceleton
Michael Kruse via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 6 05:04:50 PDT 2016
Meinersbur added a comment.
In http://reviews.llvm.org/D22036#475026, @grosser wrote:
> In http://reviews.llvm.org/D22036#475020, @Meinersbur wrote:
> > `PPCGCodeGenerationPass` sounds like it would invoke PPCG. I'd remove PPCG from the name, keeping only ACC.
> It will, internally. If you prefer, I can still rename this pass to AcceleratorCodeGenerationPass.
I didn't even know that PPCG has an interface for this. I'll wait further comments until I see the larger code base. Rename at your own discretion.
Comment at: lib/Support/RegisterPasses.cpp:90
@@ +89,3 @@
+static cl::opt<bool> PollyACC("polly-acc",
+ cl::desc("Enable accelerator code generation"),
+ cl::Hidden, cl::init(false), cl::ZeroOrMore,
"Enable" sounds like this would add accelerator code generation to the other optimizations, but effectively replaces them. Wouldn't an equivalent to -polly-vectorizer/-polly-code-generator be more appropriate?
More information about the llvm-commits