aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
> I think the patch itself is mostly obvious, but I was wondering if it is the > right time to add such RUN lines to existing test cases like I do here for > test/SemaCXX/constexpr-nqueens.cpp. If the tests are (mostly) passing without a bunch of FIXME comments, then yes, I think it's a great step to start running more of these tests with both constant expression engines. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141772/new/ https://reviews.llvm.org/D141772 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
