[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