[PATCH] D22036: [polly] Add accelerator code generation pass sceleton

[Michael Kruse via llvm-commits]

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?


http://reviews.llvm.org/D22036