[PATCH] D33396: [LV] Report multiple reasons for not vectorizing under allowExtraAnalysis

[Ayal Zaks via Phabricator via llvm-commits]

Ayal created this revision.
Herald added subscribers: fhahn, mzolotukhin, sanjoy.

Report multiple reasons for not vectorizing a loop instead of stopping and reporting the first reason only, under ORE->allowExtraAnalysis() - via Clang's -fsave-optimization-record or opt's -pass-remarks-missed.

Removed checking and reporting if we CantComputeNumberOfIterations from LoopVectorizationLegality::canVectorize(), as  
LAI::canAnalyzeLoop() is doing so, called by canVectorizeMemory(). This redundancy is caught by a lit test once multiple reasons are reported. Can be committed separately.

Patch initially developed by Dror Barak.


https://reviews.llvm.org/D33396

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/X86/vectorization-remarks-missed.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33396.99724.patch
Type: text/x-patch
Size: 8687 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170522/e777cda0/attachment.bin>