[PATCH] Use getFirstNonPHI to set the insertion point for PHIs
eli.friedman at gmail.com
Thu Sep 26 15:02:52 PDT 2013
On Thu, Sep 26, 2013 at 2:35 PM, Justin Bogner <mail at justinbogner.com>wrote:
> When creating new PHI instructions in SLPVectorizer and LoopVectorize,
> we're setting the insertion point using getFirstInsertionPt. This does
> the wrong thing when landingpads are present, since getFirstInsertionPt
> skips past these.
> The attached patch uses getFirstNonPHI instead, and adds a testcase that
> crashes without the patch and passes with. Can someone please commit?
The patch is missing a testcase for the LoopVectorize change.
(On a side note, you should request commit access; see
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits