hfinkel added a comment. Here's what I suggest:
1. We commit this patch (I think it logically makes sense; the builtins should be constexpr evaluatable, even if they're currently not) 2. We follow up by fixing Clang to make the builtins constexpr evaluatable (since I think that makes sense regardless of this) 3. We then go back and add some tests here (which would need to be predicated on the newer version of Clang) @mclow.lists , does that make sense to you? https://reviews.llvm.org/D25403 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits