[PATCH] D107378: Make enum iteration with seq safe by default
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 30 12:58:02 PDT 2021
dblaikie added a comment.
> A couple of weeks later, I learned that seq was enhanced to support enums. But when I looked at the implementation, I learned that it blindly accepts all enum types, even though it does not make sense to iterate over all of them.
Do you have a link to the patch that made that change?
Any references to problems created (bugs introduced/patches that fix those bugs, etc) by that more liberal change compared to the direction in this patch?
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