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

Sebastian Pop spop at codeaurora.org
Thu May 2 14:05:36 PDT 2013


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?

Thanks,
Sebastian
-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
hosted by The Linux Foundation
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-independent-blocks-do-not-insert-stores-between-phi-.patch
Type: text/x-diff
Size: 1321 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130502/caabe891/attachment.patch>


More information about the llvm-commits mailing list