[PATCH] D67764: [LV] Forced vectorization with runtime checks and OptForSize

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:2737
+                     "unless forced to vectorize.");
+    if (Forced)
+      ORE->emit([&]() {
----------------
Check is redundant, Forced is asserted.

Probably need to have a dummy use of Forced, or avoid defining it, to avoid warning of unused variable in release mode.


================
Comment at: llvm/test/Transforms/LoopVectorize/runtime-check.ll:185
+
 ; CHECK: !9 = !DILocation(line: 101, column: 1, scope: !{{.*}})
 
----------------
This check line belongs to previous test


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

https://reviews.llvm.org/D67764