[PATCH] D57260: Reapply rL352238: [WarnMissedTransforms] Set default to 1.

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur added a comment.

The check `hasVectorizeTransformation(L)` in front checks whether vectorization is forced by the user. It is not necessary that the user also fixes the simd width/interleave count. Hence, the warning is valid in this case. It is the same behavior as before rL348944 <https://reviews.llvm.org/rL348944> (which I tried to replicate).



================
Comment at: test/Transforms/LoopVectorize/X86/vectorization-remarks-missed.ll:36
 ; CHECK: remark: source.cpp:19:5: loop not vectorized
-; CHECK: warning: source.cpp:19:5: loop not vectorized: the optimizer was unable to perform the requested transformation; the transformation might be disabled or specified as part of an unsupported transformation ordering
 
----------------
Assuming the comment above accurately describes the IR, why do you think the warning should not be emitted?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57260