[PATCH] D65197: [LV] Tail-loop Folding

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/tail_loop_folding.ll:1
+; REQUIRES: asserts
+; RUN: opt < %s -loop-vectorize -S | FileCheck %s
----------------
It looks like assertions are not required for the test case.


================
Comment at: llvm/test/Transforms/LoopVectorize/tail_loop_folding.ll:5
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
----------------
If this test relies on the x86 cost model/x86 masked instructions, it should go into the subfolder I think.


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

https://reviews.llvm.org/D65197