[PATCH] D85794: [WIP][llvm][LV] Replace `unsigned VF` with `ElementCount VF` [NFCI]
Francesco Petrogalli via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 13 14:08:52 PDT 2020
fpetrogalli updated this revision to Diff 285493.
fpetrogalli marked 3 inline comments as done.
fpetrogalli added a comment.
Thank you for the reviews.
@vkmr, I have addressed your comments.
I have replaced the initializer lists used for bnuilding ECs instances
with the explicit constructor. The reasoning that led to this makes me
thing that maybe we shoudl make `Min` and `scalable` private in the
`ElementCount` class... but that definitely for a separate patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85794/new/
https://reviews.llvm.org/D85794
Files:
llvm/include/llvm/Analysis/TargetTransformInfo.h
llvm/include/llvm/Analysis/VectorUtils.h
llvm/include/llvm/IR/DiagnosticInfo.h
llvm/include/llvm/Support/TypeSize.h
llvm/lib/IR/DiagnosticInfo.cpp
llvm/lib/Transforms/Vectorize/LoopVectorizationPlanner.h
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
llvm/lib/Transforms/Vectorize/VPlan.cpp
llvm/lib/Transforms/Vectorize/VPlan.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85794.285493.patch
Type: text/x-patch
Size: 104158 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200813/db161e4f/attachment-0001.bin>
More information about the llvm-commits
mailing list