[PATCH] D59832: [LoopRotation] Allow loop header duplication if vectorization is forced

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 27 08:34:39 PDT 2019


fhahn added inline comments.


================
Comment at: test/Transforms/LoopVectorize/AArch64/Oz-and-forced-vectorize.ll:1
+; RUN: opt -Oz -S < %s | FileCheck %s
+
----------------
anemet wrote:
> fhahn wrote:
> > Could you limit the test case to running loop-rotate? The tests for LV should already be sufficient to check the transformed case. (same for second test)
> I don't think so.  There is no other way to get sizelevel=2 activated for opt.  You either run the entire -Oz pipeline or you run specific tests with sizelevel=0.
Argh, that's unfortunate :( . I thought it just respects the minsize attribute.......


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59832





More information about the llvm-commits mailing list