[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