[PATCH] D14814: [libcxx] Use __make_integer_seq builtin when available
Eric Fiselier via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 9 12:30:54 PST 2015
EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.
The only issue I have with this is that we should still test both code patchs with clang so we can ensure the fallback implementation remains correct. I would change
#if __has_builtin(__make_integer_seq) && !defined(_LIBCPP_TESTING_FALLBACK_MAKE_INTEGER_SEQUENCE)
Then add a test called `test/std/utilities/intseq/intseq.make/make_integer_seq_fallback.pass.cpp` that contains:
#define _LIBCPP_TESTING_FALLBACK_MAKE_INTEGER_SEQUENCE
#include "make_integer_seq.pass.cpp"
After that these changes LGTM. @K-ballo I'm happy to make the changes as I commit this if your OK with that?
http://reviews.llvm.org/D14814
More information about the cfe-commits
mailing list