[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