[PATCH] D25403: [CUDA] Mark __libcpp_{isnan, isinf, isfinite} as constexpr.

Hal Finkel via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 26 16:20:23 PDT 2016


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





More information about the cfe-commits mailing list