[PATCH] D33021: [libcxx] [test] libc++ test changes for CWG 2094

Marshall Clow via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 10 06:15:35 PDT 2017


mclow.lists added a comment.

In general, I prefer having the space between `static_assert(` and the condition.

That makes it easy (at least for me) to see if there's a `!` there or not.
Other than that, this LGTM.

FWIW, I think this is a language change that core got wrong, and it has already come back to bite us. (https://bugs.llvm.org/show_bug.cgi?id=28527)


https://reviews.llvm.org/D33021





More information about the cfe-commits mailing list