[PATCH] D20639: SLPVectorizer: Move propagateMetadata to VectorUtils

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed May 25 13:18:18 PDT 2016


arsenm added a comment.

In http://reviews.llvm.org/D20639#439764, @mssimpso wrote:

> Hi Matt,
>
> There's a similar static function in the loop vectorizer. Do you think it would be beneficial to refactor that one to also use the common interface?


I think so. It looks like that one might be buggy since this version seems to be trying to pick the most conservative set of metadata?


http://reviews.llvm.org/D20639





More information about the llvm-commits mailing list