[PATCH] D35725: [LV] Avoid redundant operations manipulating masks

Matthew Simpson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 07:20:07 PDT 2017


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

In https://reviews.llvm.org/D35725#817619, @Ayal wrote:

> As we provide patches of VPlan modelling, we're also trying to cleanup ILV's existing functionality, as in the ValueMap case. It's also possible to model ILV's existing functionality as-is, including redundant operations. But such cleanups seem desirable by themselves, and also help simplify the new modelling being introduced.


I agree.


https://reviews.llvm.org/D35725





More information about the llvm-commits mailing list