[PATCH] D56551: [LoopVectorizer] give more advice in remark about failure to vectorize call

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 12 07:24:11 PST 2019


spatel added a comment.

In D56551#1354518 <https://reviews.llvm.org/D56551#1354518>, @Ayal wrote:

> This LGTM too, just adding mtcw wondering if these extra checks for more accurate reporting are worth placing under allowExtraAnalysis(); and/or if TLI->isFunctionVectorizable() shouldn't be the one informing the cause of its failure when returning false.


Those are good questions/comments. I'm not too familiar with the code organization here, but I'll add that to the 'TODO' comment for now, so we don't lose it.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56551/new/

https://reviews.llvm.org/D56551





More information about the llvm-commits mailing list