[PATCH] D106279: [llvm] Add enum iteration to Sequence
Guillaume Chatelet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 19 07:43:37 PDT 2021
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
More information about the llvm-commits
mailing list