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

Reply via email to