[PATCH] D38566: [SimplifyCFG] don't sink common insts too soon (PR34603)

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 14 12:17:00 PST 2017


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D38566#955625, @spatel wrote:

> Patch updated:
>  This version just moves what was known as 'latesimplifycfg' before the SLP vectorizer (no extra passes are added as in the last rev). 
>  SLP preserves the CFG, so there's no reason to wait to run SimplifyCFG after that.


Thanks. LGTM.


https://reviews.llvm.org/D38566





More information about the llvm-commits mailing list