[PATCH] D57382: [LV] Move interleave count computation to LVP::plan().

Wed Feb 6 02:45:59 PST 2019

fhahn marked 5 inline comments as done.
fhahn added inline comments.

================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:6147
+      LLVM_DEBUG(printPlans(dbgs()));
+      return {{UserVF, 0}, CM.selectInterleaveCount(OptForSize, UserVF, 0)};
+    }
----------------
hsaito wrote:
> Need to take UserIC into account here as well?
Unfortunately the logic to decide between UserIC and computed IC is currently tightly tied with debug messages/optimization remarks. I think it would make sense to move that to plan() as well, but I would prefer to do it in a follow up patch, if that is OK with.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57382