[PATCH] D26873: [LV] Do not vectorize loops with a low dynamic tripcount, as determined by profile information

[Michael Kuperstein via llvm-commits]

mkuper added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:7203-7206
       ORE->emit(createMissedAnalysis(Hints.vectorizeAnalysisPassName(),
                                      "NotBeneficial", L)
                 << "vectorization is not beneficial "
                    "and is not explicitly forced");
----------------
anemet wrote:
> While you're here, can you please improve this message to actually mention low-trip count?
Sure.


================
Comment at: test/Transforms/LoopVectorize/X86/runtime-trip-count.ll:2
+; RUN: opt < %s  -loop-vectorize -force-vector-interleave=1 -force-vector-width=4 -S -debug-only=loop-vectorize 2>&1 | FileCheck %s
+; REQUIRES: asserts
+
----------------
anemet wrote:
> We usually try to formulate these tests without relying on asserts so that we get coverage with a no-assert build as well.
I realize that, but from a testing perspective, I actually want to verify the reason it didn't get vectorized, not only that it's not vectorized.
Do you think it would be better to duplicate the test and have an asserts and a non-asserts version? Do you know if "UNSUPPORTED: asserts" works?

(Is there a way to require asserts only for a specific run line, as opposed to the whole test? That would solve the problem.)


https://reviews.llvm.org/D26873