[PATCH] D107378: Make enum iteration with seq safe by default
Aaron Ballman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 2 08:40:45 PDT 2021
aaron.ballman accepted this revision.
aaron.ballman added a comment.
LGTM!
In D107378#3102694 <https://reviews.llvm.org/D107378#3102694>, @nikic wrote:
> For what it's worth, I really don't like this change. Seems like unnecessary complexity for very little gain.
I think we have a need in this area, but I wish we didn't. So I wish we didn't need this change, but I think it's an improvement over the previous interface because it makes it very explicit that you're forming a sequence from a type that may or may not be a contiguous sequence.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107378/new/
https://reviews.llvm.org/D107378
More information about the llvm-commits
mailing list