tbaeder marked an inline comment as done. tbaeder added inline comments.
================ Comment at: clang/test/AST/Interp/loops.cpp:278 + +namespace RangeForLoop { + constexpr int localArray() { ---------------- aaron.ballman wrote: > You should also add failure tests where the range-based for loop is not valid > in a constant expression. Especially interesting cases would be ones > involving lifetime problems, such as: https://godbolt.org/z/3EE7f8rdE Oof, that reproducer doesn't work right now for several reasons :/ Can you come up with something simpler? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D140803/new/ https://reviews.llvm.org/D140803 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits