[PATCH] D107378: Make enum iteration with seq safe by default

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 14 08:01:51 PDT 2021


gchatelet accepted this revision.
gchatelet added a comment.
This revision is now accepted and ready to land.

Please wait for @dblaikie approval as well.



================
Comment at: llvm/include/llvm/ADT/Sequence.h:333
+    static_assert(
+        enum_traits::is_iterable,
+        "Enum not iterable. Use DECLARE_ITERABLE_ENUM or override this "
----------------
I think I already asked the question but can't you inline this? I mean removing the `using` above,


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