[PATCH] D98850: [LV] Switch to using pre-computed VF ranges instead clamping.

[Florian Hahn via Phabricator via llvm-commits]

fhahn created this revision.
fhahn added reviewers: Ayal, gilr, rengolin.
Herald added subscribers: bmahjour, rogfer01, bollu, hiraditya.
fhahn requested review of this revision.
Herald added a subscriber: vkmr.
Herald added a project: LLVM.

This patch replaces the current logic to clamp the vectorization factors
during recipe creation with using pre-computed VF ranges. The clamp is
replaced with a assertion checking that the same decision is taken for
all VFs in the current range.

The main motivation is untangling the VF range clamping from recipe
creation. Computing the ranges of VFs with matching decisions up-front
will enable us to break down VPlan creation into more modular steps.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D98850

Files:
  llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/lib/Transforms/Vectorize/VPRecipeBuilder.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98850.331502.patch
Type: text/x-patch
Size: 14668 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210318/945e6aa0/attachment.bin>