[PATCH] [Polly] Put the parallel context into the function entry block

Tobias Grosser tobias at grosser.es
Thu Oct 2 10:45:28 PDT 2014


No test case changes?

Otherwise LGTM

================
Comment at: lib/CodeGen/LoopGenerators.cpp:296
@@ +295,3 @@
+  //      For now we assume the annotations suffice to sink the allocas to the
+  //      most recent dominating basic block which is not part of a loop.
+  BasicBlock &EntryBB = Builder.GetInsertBlock()->getParent()->getEntryBlock();
----------------
I don't think this XXX is even needed. AFAIK clang will always allocate all stack space in the entry block.

http://reviews.llvm.org/D5582






More information about the llvm-commits mailing list