[PATCH] D152366: [LoopVectorize] Allow inner loop runtime checks to be hoisted above an outer loop

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 12:51:10 PDT 2023


fhahn added a comment.

Does this require the `extra-vectorizer-passes` to hoist the checks out of the loop in the full pipeline? Do you know if we already have sufficient runtime tests in llvm-test-suite for this kind of code? If not, it would be good to add some, perhaps to https://github.com/llvm/llvm-test-suite/blob/main/SingleSource/UnitTests/Vectorizer/runtime-checks.cpp.

Also, could you clean up the block & value names in the tests a bit so they are easier to read and land them separately?


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

https://reviews.llvm.org/D152366



More information about the llvm-commits mailing list