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

[Guillaume Chatelet via Phabricator via llvm-commits]

gchatelet added a comment.

This is a different take on D103900 <https://reviews.llvm.org/D103900> which I had to revert because of the following UB operation.

  [  DEATH   ] llvm-project/llvm/include/llvm/ADT/Sequence.h:198:68: runtime error: signed integer overflow: -9223372036854775808 - 1 cannot be represented in type 'long'

I think the new design is easier to reason about as all the casts and arithmetic operations are isolated and controlled.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106279