[polly] independent blocks: do not insert stores between phi nodes

Tobias Grosser tobias at grosser.es
Fri May 3 06:04:23 PDT 2013


On 05/02/2013 11:05 PM, Sebastian Pop wrote:
> Hi,
>
> When translateScalarToArray tranlates a phi node to an array, it does not make
> sure that the insertion point is after all the phi nodes of the basic block, and
> that could lead to a mix of phi nodes and translated stmts: this ICEs one of the
> verifiers that expects to see all the phi nodes grouped at the beginning of a
> basic block.  The patch moves the insertion point for the translation of a phi
> node after all the phi nodes.
>
> Ok to commit?

Looks good, but please include a _reduced_ test case.

Tobias




More information about the llvm-commits mailing list