[PATCH] D142258: [LV] Ignore runtime checks threshold when vectorization is forced

Kolya Panchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 10:57:53 PST 2023


nikolaypanchenko added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/memcheck_threashold_vec_forced.ll:6
+
+; CHECK-DEBUG-LABEL: LV: Checking a loop in 'vec_forced'
+; CHECK-DEBUG: Executing best plan with VF=4,
----------------
fhahn wrote:
> Given that the test checks the full IR, it would probably be better to remove the check of debug output and run it unconditionally (remove `REQUIRES: asserts`).
Mostly I added it to check that `vec_not_forced` is not vectorized due to the `number of checks exceeded threshold`. Agree that for the `vec_forced` it's not needed though, just tried to keep checks similar


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

https://reviews.llvm.org/D142258



More information about the llvm-commits mailing list