[PATCH] D103900: [llvm] Add enum iteration to Sequence

[Guillaume Chatelet via Phabricator via llvm-commits]

gchatelet updated this revision to Diff 353885.
gchatelet marked 3 inline comments as done.
gchatelet added a comment.

- Trying a different approach
- rebase and use seq_inclusive in a few places
- rename StorageType to iota_range_iterator_storage_type
- Use std::conditional instead of custom specialization
- Address comments
- Address comments
- Prevent overflow of PastEndValue + tests
- Address comments
- Fix clang-tidy warning
- Fix another clang-tidy warning
- Fix UB in reverse iteration with signed type


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103900

Files:
  llvm/include/llvm/ADT/Sequence.h
  llvm/include/llvm/Support/MachineValueType.h
  llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp
  llvm/tools/llvm-exegesis/lib/X86/Target.cpp
  llvm/unittests/ADT/SequenceTest.cpp
  llvm/unittests/CodeGen/ScalableVectorMVTsTest.cpp
  llvm/unittests/IR/ConstantRangeTest.cpp
  mlir/lib/Dialect/Linalg/IR/LinalgOps.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103900.353885.patch
Type: text/x-patch
Size: 22458 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210623/0620e059/attachment-0001.bin>