[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 mailing list